Re: [PATCH 1/9] Makefile: Do not install a copy of 'git' in $(gitexecdir)
- From
A Large Angry SCM <gitzilla@gmail.com>
- Date
- Jul 28, 2008, 10:41 UTC
- Message-ID
- <488DA242.3000508@gmail.com>
- In-Reply-To
- <7vabg2v83o.fsf@gitster.siamese.dyndns.org>
Junio C Hamano wrote:
Show 22 quoted lines
> A Large Angry SCM <gitzilla@gmail.com> writes: > >> Johannes Sixt wrote: >> [...] >>> diff --git a/Makefile b/Makefile >>> index 551bde9..cbab4f9 100644 >>> --- a/Makefile >>> +++ b/Makefile >>> @@ -1335,6 +1335,7 @@ endif >>> '$(DESTDIR_SQ)$(gitexecdir_SQ)/git$X'; \ >>> fi >>> $(foreach p,$(BUILT_INS), $(RM) '$(DESTDIR_SQ)$(gitexecdir_SQ)/$p' && ln '$(DESTDIR_SQ)$(gitexecdir_SQ)/git$X' '$(DESTDIR_SQ)$(gitexecdir_SQ)/$p' ;) >>> + $(RM) '$(DESTDIR_SQ)$(gitexecdir_SQ)/git$X' >>> ifneq (,$X) >>> $(foreach p,$(patsubst %$X,%,$(filter %$X,$(ALL_PROGRAMS) $(BUILT_INS) git$X)), $(RM) '$(DESTDIR_SQ)$(gitexecdir_SQ)/$p';) >>> endif >> This new action needs to be in a conditional to keep it from removing >> the ONLY git executable when bindir and execdir are the same dir. > > Heh, I love bug reports that come immediately after I tag the tip of > 'master' as -rc1. >
Well, I'm currently about a week behind on all things git so this is as timely as I could make it. ;-)