-----Original Message----- From: Kevin P. Fleming [mailto:kpfleming@xxxxxxxxxx] Sent: 2010年7月25日 16:50 To: Ævar Arnfjörð Bjarmason Cc: Lin, Lynn; git@xxxxxxxxxxxxxxx Subject: Re: [PATCH] Makefile: don't include git version file on 'make clean' On 07/24/2010 02:36 PM, Ævar Arnfjörð Bjarmason wrote: > On Sat, Jul 24, 2010 at 03:53, <Lynn.Lin@xxxxxxx> wrote: >> From: Lynn Lin <Lynn.Lin@xxxxxxx> >> >> --- >> Makefile | 4 +++- >> git-gui/Makefile | 4 +++- >> 2 files changed, 6 insertions(+), 2 deletions(-) >> >> diff --git a/Makefile b/Makefile >> index bc3c570..eb28b98 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -238,7 +238,9 @@ all:: >> >> GIT-VERSION-FILE: FORCE >> @$(SHELL_PATH) ./GIT-VERSION-GEN >> --include GIT-VERSION-FILE >> +ifneq "$(MAKECMDGOALS)" "clean" >> + -include GIT-VERSION-FILE >> +endif >> >> uname_S := $(shell sh -c 'uname -s 2>/dev/null || echo not') >> uname_M := $(shell sh -c 'uname -m 2>/dev/null || echo not') >> diff --git a/git-gui/Makefile b/git-gui/Makefile >> index 197b55e..91e1ea5 100644 >> --- a/git-gui/Makefile >> +++ b/git-gui/Makefile >> @@ -9,7 +9,9 @@ all:: >> >> GIT-VERSION-FILE: FORCE >> @$(SHELL_PATH) ./GIT-VERSION-GEN >> --include GIT-VERSION-FILE >> +ifneq "$(MAKECMDGOALS)" "clean" >> + -include GIT-VERSION-FILE >> +endif >> >> uname_S := $(shell sh -c 'uname -s 2>/dev/null || echo not') >> uname_O := $(shell sh -c 'uname -o 2>/dev/null || echo not') >> -- >> 1.7.1 > > This patch needs a rationale, why was it needed? The "-include" > directive will simply ignore files that don't exist (as opposed to > "include"), so including GIT-VERSION-FILE during "make clean' > shouldn't be an issue. Just guessing here, but since GIT-VERSION-FILE has a 'FORCE' prerequisite, that means that the operations to generate it will be run even for 'make clean', which is not useful for the cleaning operation. It's probably not harmful either... but maybe the OP has some more significant reason for this patch. Yes, when we run 'make clean' ,it also generate the git version file,then remove it .It's not necessary to trigger the operation when run 'make clean' command Lynn -- Kevin P. Fleming Digium, Inc. | Director of Software Technologies 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA skype: kpfleming | jabber: kfleming@xxxxxxxxxx Check us out at www.digium.com & www.asterisk.org ��.n��������+%������w��{.n��������n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�