On Thu, 12 Apr 2018 16:55:15 -0700 Stefan Beller <sbel...@google.com> wrote:
> Hi Antonio, > > the subject line could also be > fetch: fix custom submodule location > as I was not expecting this patch from the subject line. > OK. > On Thu, Apr 12, 2018 at 3:20 PM, Antonio Ospite <a...@ao2.it> wrote: > > Import the default git configuration before accessing the gitmodules > > file. > > > > This is important when a value different from the default one is set via > > the 'core.submodulesfile' config. > > > > Signed-off-by: Antonio Ospite <a...@ao2.it> > > --- > > builtin/fetch.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/builtin/fetch.c b/builtin/fetch.c > > index dcdfc66f0..d56636f74 100644 > > --- a/builtin/fetch.c > > +++ b/builtin/fetch.c > > @@ -1428,8 +1428,8 @@ int cmd_fetch(int argc, const char **argv, const char > > *prefix) > > for (i = 1; i < argc; i++) > > strbuf_addf(&default_rla, " %s", argv[i]); > > > > - config_from_gitmodules(gitmodules_fetch_config, NULL); > > git_config(git_fetch_config, NULL); > > + config_from_gitmodules(gitmodules_fetch_config, NULL); > > Wouldn't this break the overwriting behavior? > e.g. you configure a submodule URL in .gitmodules and then override it > in .git/config, > then we'd currently first load config from the modules file and then override > it > in git_config(git_fetch_config,..), whereas swapping them might have > unintended consequences? Do the tests pass with this patch? The patch changes the current behavior indeed, but it does not break any of the existing tests. Anyway, to be on the safe side, I could load only the core.submodulesFile option from the global configuration, maybe even in config_from_gitmodules() itself, before accessing the gitmodules file, but I still don't know how to do that. Is there an API to just load one config setting? Ciao, Antonio -- Antonio Ospite https://ao2.it https://twitter.com/ao2it A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing?