diff options
author | Jeff King <peff@peff.net> | 2011-06-15 18:31:28 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-06-15 15:56:28 -0700 |
commit | 2321286298bf956a3fa8e91dd361b8cf4e81e6a0 (patch) | |
tree | 8879ff6c7d193e9ea5760bff2f32bdd2444890f5 /object.h | |
parent | 2c162b56f370f5c33e6a945e6922d598006c5ec4 (diff) | |
download | git-2321286298bf956a3fa8e91dd361b8cf4e81e6a0.tar.gz git-2321286298bf956a3fa8e91dd361b8cf4e81e6a0.tar.xz |
archive: reorder option parsing and config reading
The archive command does three things during its
initialization phase:
1. parse command-line options
2. setup the git directory
3. read config
During phase (1), if we see any options that do not require
a git directory (like "--list"), we handle them immediately
and exit, making it safe to abort step (2) if we are not in
a git directory.
Step (3) must come after step (2), since the git directory
may influence configuration. However, this leaves no
possibility of configuration from step (3) impacting the
command-line options in step (1) (which is useful, for
example, for supporting user-configurable output formats).
Instead, let's reorder this to:
1. setup the git directory, if it exists
2. read config
3. parse command-line options
4. if we are not in a git repository, die
This should have the same external behavior, but puts
configuration before command-line parsing.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'object.h')
0 files changed, 0 insertions, 0 deletions