chiark / gitweb /
dgit: Postpone call to git_slurp_config
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Wed, 28 Jun 2017 15:39:59 +0000 (16:39 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Sat, 8 Jul 2017 17:53:59 +0000 (18:53 +0100)
commit3c25f09fe6a6787011c2a16e12e54da264a881e0
tree9c8bea361216d1007f82cb29e5b34e6d97935711
parent8efcda8d67ec2934f259e4ce951570fef8f37a34
dgit: Postpone call to git_slurp_config

This is needed to fix #865863: we need to know what our operation is,
before we can decide whether to look for --local git config.

The code which now runs earlier is:
 * The messages about $dryrun_level (which is set only by
   the command line, and not by configuration - verified by
   searching for $dryrun_level).
 * Usage failure if @ARGV empty.  This is not affected by
   configuration.  (parseopts does the argument parsing and
   already runs before git_slurp_config.)
 * Extracting the $cmd from @ARGV.
 * Calling $pre_fn.  There is only one pre_* sub, which is
   pre_gbp_build.  It provides the default for $quilt_mode.
   $quilt_mode is indeed somewhat entangled with the git config,
   but this takes place in parseopts_late_defaults, which is
   called much later.

Therefore there is no functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
dgit