# [PATCH 1/2] chdir after computing new cwd in check_repository_format_gently

2 messages from 2008-05-14 to 2008-05-14. Participants: David Reiss, Johannes Schindelin.
Thread: https://gitlist.dev/t/13507

## David Reiss, 2008-05-14 05:45

Subject: [PATCH 1/2] chdir after computing new cwd in check_repository_format_gently
Message-ID: <482A7C71.6050509@facebook.com>
URL: https://gitlist.dev/e/482A7C71.6050509%40facebook.com

```
In preparation for adding a check before the chdir.
This should be a no-op because the cwd is not read in the interim
and any nonlocal exits either chdir to an absolute path or die.

Signed-off-by: David Reiss <dreiss@facebook.com>
---
I sent this in a few weeks ago, but got no response.  Probably because I
messed up the formatting.  Hopefully I'm getting it right this time.

 setup.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/setup.c b/setup.c
index b8fd476..c54f2b6 100644
--- a/setup.c
+++ b/setup.c
@@ -443,7 +443,6 @@ const char *setup_git_directory_gently(int *nongit_ok)
 			check_repository_format_gently(nongit_ok);
 			return NULL;
 		}
-		chdir("..");
 		do {
 			if (!offset) {
 				if (nongit_ok) {
@@ -455,6 +454,7 @@ const char *setup_git_directory_gently(int *nongit_ok)
 				die("Not a git repository");
 			}
 		} while (cwd[--offset] != '/');
+		chdir("..");
 	}
 
 	inside_git_dir = 0;
-- 
1.5.4

```

## Johannes Schindelin, 2008-05-14 08:02

Subject: Re: [PATCH 1/2] chdir after computing new cwd in check_repository_format_gently
Message-ID: <alpine.DEB.1.00.0805140901420.30431@racer>
URL: https://gitlist.dev/e/alpine.DEB.1.00.0805140901420.30431%40racer
In-Reply-To: <482A7C71.6050509@facebook.com>

```
Hi,

On Tue, 13 May 2008, David Reiss wrote:

> In preparation for adding a check before the chdir. This should be a 
> no-op because the cwd is not read in the interim and any nonlocal exits 
> either chdir to an absolute path or die.

I think it would be clearer if you squashed that commit into the other 
one.

Ciao,
Dscho

```