git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: Relating to some syntax error in the gitk script
@ 2007-09-27 17:00 Brett Schwarz
  2007-09-28 20:57 ` [PATCH] gitk: add check for required tcl version >= 8.4 Steffen Prohaska
  0 siblings, 1 reply; 4+ messages in thread
From: Brett Schwarz @ 2007-09-27 17:00 UTC (permalink / raw)
  To: Ritwik Ghosh, git

> 
> When I am running the gitk command I get the following error message:-
> 
> ========================================================================
> ===========================
> Error in startup script: syntax error in expression "$i >= [llength
> $argv] && $revtreeargs ne {}"
>     while executing
> "if {$i >= [llength $argv] && $revtreeargs ne {}} {
>     # no -- on command line, but some arguments (other than -d)
>     if {[catch {
>         set f [eval exec..."
>     (file "./gitk" line 7912)
> ========================================================================
> ============================
> 
> Is there a problem with the script or is it a problem with my
> environment...?
> 

What version of Tcl are you running? You should be using >= 8.4





       
____________________________________________________________________________________
Yahoo! oneSearch: Finally, mobile search 
that gives answers, not web links. 
http://mobile.yahoo.com/mobileweb/onesearch?refer=1ONXIC

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH] gitk: add check for required tcl version >= 8.4
@ 2007-09-29  9:32 Junio C Hamano
  2007-10-20 11:19 ` Paul Mackerras
  0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2007-09-29  9:32 UTC (permalink / raw)
  To: paulus; +Cc: git, Steffen Prohaska

From: Steffen Prohaska <prohaska@zib.de>
Date: Fri, 28 Sep 2007 22:57:22 +0200

gitk requires tcl version >= 8.4 to work flawlessly. So let's
check the tcl version and quit if it's too low.

Signed-off-by: Steffen Prohaska <prohaska@zib.de>
---

 * I do not have a ready access to older tcl/tk myself, so I
   cannot judge if this is sensible or not.  Just forwarding in
   case you missed it.

 gitk |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/gitk b/gitk
index 300fdce..6ea6489 100755
--- a/gitk
+++ b/gitk
@@ -7,6 +7,11 @@ exec wish "$0" -- "$@"
 # and distributed under the terms of the GNU General Public Licence,
 # either version 2, or (at your option) any later version.
 
+if {[info tclversion] < 8.4} {
+   puts stderr "Sorry, gitk requires tcl version >= 8.4."
+   exit 1 
+}
+
 proc gitdir {} {
     global env
     if {[info exists env(GIT_DIR)]} {
-- 
1.5.3.2.111.g5166

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-10-20 11:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-09-27 17:00 Relating to some syntax error in the gitk script Brett Schwarz
2007-09-28 20:57 ` [PATCH] gitk: add check for required tcl version >= 8.4 Steffen Prohaska
  -- strict thread matches above, loose matches on Subject: below --
2007-09-29  9:32 Junio C Hamano
2007-10-20 11:19 ` Paul Mackerras

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).