Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-28 Thread Jeff King
On Tue, Feb 28, 2017 at 12:21:57PM -0800, Stefan Beller wrote: > > I'm still open to the idea that we simply improve the documentation to > > make it clear that per-repo config really is per-repo, and is not shared > > between super-projects and submodules. And then something like Duy's > > propos

Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-28 Thread Stefan Beller
On Tue, Feb 28, 2017 at 12:08 PM, Jeff King wrote: > On Tue, Feb 28, 2017 at 10:05:24AM -0800, Stefan Beller wrote: > >> > I have a feeling that something like this would create unwelcome corner >> > cases in the config-writer, which is otherwise does not have to care >> > about which existing sec

Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-28 Thread Jeff King
On Tue, Feb 28, 2017 at 10:05:24AM -0800, Stefan Beller wrote: > > I have a feeling that something like this would create unwelcome corner > > cases in the config-writer, which is otherwise does not have to care > > about which existing section of a file it adds a key to. > > Yeah the writer woul

Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-28 Thread Stefan Beller
On Tue, Feb 28, 2017 at 6:37 AM, Jeff King wrote: >> >> This would change the semantics of a config file as the attribute for >> each setting depends on the location (was attribute.FOO.read = >> {true, false} read before). > > I'm not enthused by this, just because there is a hidden dependency > b

Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-28 Thread Jeff King
On Mon, Feb 27, 2017 at 11:09:12AM -0800, Stefan Beller wrote: > For worktrees these multiple config files sounded like > the obvious solution, but I wonder if there was also > some bike shedding about other solutions? > > I could imagine that we would want to have attributes > for specific confi

Re: 'git submodules update' ignores credential.helper config of the parent repository

2017-02-27 Thread Stefan Beller
On Mon, Feb 27, 2017 at 5:33 AM, Dmitry Neverov wrote:> > git -c credential.helper= submodule update > > Is it by design? A similar question came up w.r.t. submodule configuration recently. It is about url..insteadOf[1] that is set in the super project and is expected to work in the submodules.

'git submodules update' ignores credential.helper config of the parent repository

2017-02-27 Thread Dmitry Neverov
I'm checking out a repository in a non-interactive environment and would like to disable interactive credential helpers. According to [1] it can be done by specifying an empty helper in a local config: [credential] helper = But the submodule update command ignores the helper specified in th