{"thread":{"id":"32368","subject":"[PATCH v3 1/4] Makefile: remove tracking of TCLTK_PATH","startedAt":"2012-12-17T07:17:12Z","lastAt":"2012-12-17T07:17:12Z","messageCount":1,"participants":["Christian Couder"],"isPatch":true,"patchVersion":3,"patchTotal":4},"messages":[{"id":"205017","messageId":"20121217071716.26209.18070.chriscool@tuxfamily.org","threadId":"32368","inReplyTo":null,"subject":"[PATCH v3 1/4] Makefile: remove tracking of TCLTK_PATH","fromName":"Christian Couder","fromEmail":"chriscool@tuxfamily.org","sentAt":"2012-12-17T07:17:12Z","receivedAt":"2012-12-17T07:17:12Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"It looks like we are tracking the value of TCLTK_PATH in the main\nMakefile for no good reason, as this is done in git-gui too and the\nGIT-GUI-VARS is not used in the Makefile.\n\nThis patch removes the useless code used to do this tracking.\n\nMaybe this code should have been moved to gitk-git/Makefile by\n62ba514 (Move gitk to its own subdirectory, 2007-11-17).\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\n---\n .gitignore |  1 -\n Makefile   | 14 +-------------\n 2 files changed, 1 insertion(+), 14 deletions(-)\n\ndiff --git a/.gitignore b/.gitignore\nindex f702415..6d69ae1 100644\n--- a/.gitignore\n+++ b/.gitignore\n@@ -1,7 +1,6 @@\n /GIT-BUILD-OPTIONS\n /GIT-CFLAGS\n /GIT-LDFLAGS\n-/GIT-GUI-VARS\n /GIT-PREFIX\n /GIT-SCRIPT-DEFINES\n /GIT-USER-AGENT\ndiff --git a/Makefile b/Makefile\nindex 4ad6fbd..585b2eb 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -2624,18 +2624,6 @@ ifdef GIT_PERF_MAKE_OPTS\n \t@echo GIT_PERF_MAKE_OPTS=\\''$(subst ','\\'',$(subst ','\\'',$(GIT_PERF_MAKE_OPTS)))'\\' >>$@\n endif\n \n-### Detect Tck/Tk interpreter path changes\n-ifndef NO_TCLTK\n-TRACK_VARS = $(subst ','\\'',-DTCLTK_PATH='$(TCLTK_PATH_SQ)')\n-\n-GIT-GUI-VARS: FORCE\n-\t@VARS='$(TRACK_VARS)'; \\\n-\t    if test x\"$$VARS\" != x\"`cat $@ 2>/dev/null`\" ; then \\\n-\t\techo 1>&2 \"    * new Tcl/Tk interpreter location\"; \\\n-\t\techo \"$$VARS\" >$@; \\\n-            fi\n-endif\n-\n test_bindir_programs := $(patsubst %,bin-wrappers/%,$(BINDIR_PROGRAMS_NEED_X) $(BINDIR_PROGRAMS_NO_X) $(TEST_PROGRAMS_NEED_X))\n \n all:: $(TEST_PROGRAMS) $(test_bindir_programs)\n@@ -2910,7 +2898,7 @@ ifndef NO_TCLTK\n \t$(MAKE) -C gitk-git clean\n \t$(MAKE) -C git-gui clean\n endif\n-\t$(RM) GIT-VERSION-FILE GIT-CFLAGS GIT-LDFLAGS GIT-GUI-VARS GIT-BUILD-OPTIONS\n+\t$(RM) GIT-VERSION-FILE GIT-CFLAGS GIT-LDFLAGS GIT-BUILD-OPTIONS\n \t$(RM) GIT-USER-AGENT GIT-PREFIX GIT-SCRIPT-DEFINES\n \n .PHONY: all install profile-clean clean strip\n-- \n1.8.1.rc1.2.g8740035\n"}]}