Thread (12 messages) flat view 12 messages, 4 authors, 2016-06-15

Re: [PATCH v2] git: submodule honor -c credential.* from command line

From: Jacob Keller <hidden>
Date: 2016-06-15 23:08:27

On Wed, Feb 24, 2016 at 11:11 PM, Jeff King [off-list ref] wrote:
quoted hunk ↗ jump to hunk
 static int clone_submodule(const char *path, const char *gitdir, const char *url,
                           const char *depth, const char *reference, int quiet)
 {
@@ -145,7 +166,7 @@ static int clone_submodule(const char *path, const char *gitdir, const char *url
        argv_array_push(&cp.args, path);

        cp.git_cmd = 1;
-       cp.env = local_repo_env;
+       add_submodule_repo_env(&cp.env_array);
        cp.no_stdin = 1;

        return run_command(&cp);

Ignore my previous comment. The cp.env API is *very* subtle. If the
line is just a name, it removes the environment variable, while
"name=value" adds it. That is definitely not what I was expecting
here, so I misread how it works.

I am sending a v3 with an extended test similar to the one that Jeff suggested.

Thanks,
Jake
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help