Re: [PATCH] git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS

2016-03-23 Thread Junio C Hamano
Jonathan Nieder writes: > I should have mentioned this is > > Reported-by: Shin Fan > >> Signed-off-by: Jeff King >> --- >> I just did this on master, and it is standalone. But for the reasons >> above I think it would also be fine to stick on the tip of the >> jk/submodule-c-credential topic.

Re: [PATCH] git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS

2016-03-22 Thread Jacob Keller
On Tue, Mar 22, 2016 at 2:43 PM, Jonathan Nieder wrote: > Jeff King wrote[1]: > >> Subject: git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS >> >> The "git -c var=value" option stuffs the config value into >> $GIT_CONFIG_PARAMETERS, so that sub-processes can see it. >> When the config

Re: [PATCH] git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS

2016-03-22 Thread Jonathan Nieder
Jeff King wrote[1]: > Subject: git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS > > The "git -c var=value" option stuffs the config value into > $GIT_CONFIG_PARAMETERS, so that sub-processes can see it. > When the config is later read via git_config() or similar, > we parse it back ou

Re: [PATCH] git_config_push_parameter: handle empty GIT_CONFIG_PARAMETERS

2016-03-22 Thread Junio C Hamano
Jeff King writes: > Obviously another option would be to make the parsing side more liberal > (which has the added effect that if anybody _else_ ever wants to > generate $GIT_CONFIG_PARAMETERS, they will not get annoyed). But I took > this route for now because it's the simplest way to fix the re