On 03/03, Nguyễn Thái Ngọc Duy wrote:
> This variable was added because the repo_set_gitdir() was created to
> cover both submodule and main repos, but these two are initialized a
> bit differently so ignore_env == 0 means main repo, while ignore_env
> != 0 is submodules.
>
> Since the difference part (env variables) has been moved out of
> repo_set_gitdir(), this function works the same way for both repo
> types and ignore_env is not needed anymore.
>
> Signed-off-by: Nguyễn Thái Ngọc Duy <[email protected]>
Sweet! Thanks for getting rid of this.
> ---
> repository.c | 2 --
> repository.h | 9 ---------
> 2 files changed, 11 deletions(-)
>
> diff --git a/repository.c b/repository.c
> index 04d85a2869..62f52f47fc 100644
> --- a/repository.c
> +++ b/repository.c
> @@ -140,8 +140,6 @@ static int repo_init(struct repository *repo,
> struct repository_format format;
> memset(repo, 0, sizeof(*repo));
>
> - repo->ignore_env = 1;
> -
> if (repo_init_gitdir(repo, gitdir))
> goto error;
>
> diff --git a/repository.h b/repository.h
> index 2bfbf762f3..e7127baffb 100644
> --- a/repository.h
> +++ b/repository.h
> @@ -75,15 +75,6 @@ struct repository {
> const struct git_hash_algo *hash_algo;
>
> /* Configurations */
> - /*
> - * Bit used during initialization to indicate if repository state (like
> - * the location of the 'objectdir') should be read from the
> - * environment. By default this bit will be set at the begining of
> - * 'repo_init()' so that all repositories will ignore the environment.
> - * The exception to this is 'the_repository', which doesn't go through
> - * the normal 'repo_init()' process.
> - */
> - unsigned ignore_env:1;
>
> /* Indicate if a repository has a different 'commondir' from 'gitdir' */
> unsigned different_commondir:1;
> --
> 2.16.1.435.g8f24da2e1a
>
--
Brandon Williams
