[PATCH] gitk errors out if launched in a subdirectory

[PATCH] gitk errors out if launched in a subdirectory

[Peter Baumann]

Make gitk use git-rev-parse --git-dir to find the repository.

Signed-off-by: Peter Baumann <siprbaum@stud.informatik.uni-erlangen.de>
---
 gitk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gitk b/gitk
index 3dabc69..ec4f1bd 100755
--- a/gitk
+++ b/gitk
@@ -12,7 +12,7 @@ proc gitdir {} {
     if {[info exists env(GIT_DIR)]} {
 	return $env(GIT_DIR)
     } else {
-	return ".git"
+	return [exec git rev-parse --git-dir]
     }
 }
 
-- 
1.4.4.2

Re: [PATCH] gitk errors out if launched in a subdirectory

[Peter Baumann]

Is there something obviously wrong with this patch? It is the second
time I send this (or a slight variant; last time I even removed the
enviroment checking) and nobody answered.

Please say so if it is NAKed.

-Peter


On Tue, Jan 09, 2007 at 03:30:19PM +0100, Peter Baumann wrote:
> Make gitk use git-rev-parse --git-dir to find the repository.
> 
> Signed-off-by: Peter Baumann <siprbaum@stud.informatik.uni-erlangen.de>
> ---
>  gitk |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/gitk b/gitk
> index 3dabc69..ec4f1bd 100755
> --- a/gitk
> +++ b/gitk
> @@ -12,7 +12,7 @@ proc gitdir {} {
>      if {[info exists env(GIT_DIR)]} {
>  	return $env(GIT_DIR)
>      } else {
> -	return ".git"
> +	return [exec git rev-parse --git-dir]
>      }
>  }
>  
> -- 
> 1.4.4.2
> 

Re: [PATCH] gitk errors out if launched in a subdirectory

[Junio C Hamano]

Peter Baumann <siprbaum@stud.informatik.uni-erlangen.de> writes:

> Is there something obviously wrong with this patch? It is the second
> time I send this (or a slight variant; last time I even removed the
> enviroment checking) and nobody answered.
>
> Please say so if it is NAKed.

I am Ok with it if Paul is.  I think it makes sense.

Re: [PATCH] gitk errors out if launched in a subdirectory

[Paul Mackerras]

Peter Baumann writes:

> Is there something obviously wrong with this patch? It is the second
> time I send this (or a slight variant; last time I even removed the
> enviroment checking) and nobody answered.
> 
> Please say so if it is NAKed.

No, it looks fine.  It's just that gitk takes a lower priority than
some other things in my life.

Paul.

Re: [PATCH] gitk errors out if launched in a subdirectory

[Peter Baumann]

On Sat, Jan 13, 2007 at 02:13:58PM +1100, Paul Mackerras wrote:
> Peter Baumann writes:
> 
> > Is there something obviously wrong with this patch? It is the second
> > time I send this (or a slight variant; last time I even removed the
> > enviroment checking) and nobody answered.
> > 
> > Please say so if it is NAKed.
> 
> No, it looks fine.  It's just that gitk takes a lower priority than
> some other things in my life.
> 
> Paul.
> 

Ok. No problem then. I just felt a little bit stupid after posting the
this simple fix the second time and nobody carred.

-Peter