Re: [PATCH 10/22] run-command: optionally clear git environment

> Alex Riesen <raa.lkml@gmail.com> wrote:
> > On 5/24/07, skimo@liacs.nl <skimo@liacs.nl> wrote:
> > >+               if (cmd->clear_git_env) {
> > >+                       unsetenv(ALTERNATE_DB_ENVIRONMENT);
> > >+                       unsetenv(DB_ENVIRONMENT);
> > >+                       unsetenv(CONFIG_ENVIRONMENT);
> > >+                       unsetenv(GIT_DIR_ENVIRONMENT);
> > >+                       unsetenv(GRAFT_ENVIRONMENT);
> > >+                       unsetenv(INDEX_ENVIRONMENT);
> > >+               }
> >
> > You might want to try the alternative approach from the recently
> > proposed patches to do the same, but more generic. Would
> > be less code, too.
>
> Unfortunately Alex's approach means the caller must know the list of
> "special Git envvars" that should be cleared when entering into a
> subproject Git repository to execute a command.  That's horrible code
> duplication in the callers of run_command, and is just asking for
> trouble later when/if another magic environment variable is added.