Re: [PATCH] Add missing inside_work_tree setting in setup_git_directory_gently
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Nov 3, 2007, 11:38 UTC
- Message-ID
- <Pine.LNX.4.64.0711031138150.4362@racer.site>
- In-Reply-To
- <20071103100323.GA25305@laptop>
Hi,
On Sat, 3 Nov 2007, Nguyễn Thái Ngọc Duy wrote:
Show 20 quoted lines
> Without this, work_tree handling code in setup_git_directory
> will be activated. If you stay in root work tree (no prefix),
> it does not harm. It does if you work from a subdirectory though.
>
> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
> ---
> Turns out my patch on NEED_WORK_TREE is fixing a wrong place.
>
> setup.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/setup.c b/setup.c
> index 145eca5..6f8f769 100644
> --- a/setup.c
> +++ b/setup.c
> @@ -240,6 +240,7 @@ const char *setup_git_directory_gently(int *nongit_ok)
> if (chdir(work_tree_env) < 0)
> die ("Could not chdir to %s", work_tree_env);
> strcat(buffer, "/");
> + inside_work_tree = 1;I really have to wonder why this is needed, as it should be deduced (correctly!) when you ask is_inside_work_tree().
Ciao, Dscho