{"thread":{"id":"20906","subject":"[PATCH 2/2] Re: Makefile: remove unused CURL_SYNONYMS variable","startedAt":"2009-09-12T05:52:54Z","lastAt":"2009-09-12T05:52:54Z","messageCount":1,"participants":["Nicolas Sebrecht"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"122934","messageId":"20090912055254.GA4752@vidovic","threadId":"20906","inReplyTo":"7vd45wloj4.fsf@alter.siamese.dyndns.org","subject":"[PATCH 2/2] Re: Makefile: remove unused CURL_SYNONYMS variable","fromName":"Nicolas Sebrecht","fromEmail":"nicolas.s.dev@gmx.fr","sentAt":"2009-09-12T05:52:54Z","receivedAt":"2009-09-12T05:52:54Z","isPatch":true,"sender":{"key":"nicolas.s.dev@gmx.fr","avatar":null},"body":"The 11/09/09, Junio C Hamano wrote:\n> \n> This looks wrong.  What's filtering out of what?\n> \n> I think you meant to finish the revert of ad17f01 (Makefile: install\n> hardlinks for git-remote-<scheme> supported by libcurl if possible,\n> 2009-08-07) that c9e388b (Make the \"traditionally-supported\" URLs a\n> special case, 2009-09-03) attempted to do but missed.\n> \n> Perhaps you meant to do this?\n> \n> diff --git a/Makefile b/Makefile\n> index 8c44e35..1ac02d1 100644\n> --- a/Makefile\n> +++ b/Makefile\n> @@ -1655,7 +1655,7 @@ export gitexec_instdir\n>  install: all\n>  \t$(INSTALL) -d -m 755 '$(DESTDIR_SQ)$(bindir_SQ)'\n>  \t$(INSTALL) -d -m 755 '$(DESTDIR_SQ)$(gitexec_instdir_SQ)'\n> -\t$(INSTALL) $(filter-out $(CURL_SYNONYMS), $(ALL_PROGRAMS)) '$(DESTDIR_SQ)$(gitexec_instdir_SQ)'\n> +\t$(INSTALL) $(ALL_PROGRAMS) '$(DESTDIR_SQ)$(gitexec_instdir_SQ)'\n>  \t$(INSTALL) git$X git-upload-pack$X git-receive-pack$X git-upload-archive$X git-shell$X git-cvsserver '$(DESTDIR_SQ)$(bindir_SQ)'\n>  \t$(MAKE) -C templates DESTDIR='$(DESTDIR_SQ)' install\n>  ifndef NO_PERL\n\nYes, exactly. Sorry for the mistake.\n\nOh, and I just saw that my sendemail.cc configuration has been ignored\nresulting in not cc'ing the mailing list (fixed for this mail). I guess\nit comes from some --cc options added at command line. Shouldn't --cc\nadd new cc to the cc list instead of override sendemail.cc?\n\n-- \nNicolas Sebrecht\n"}]}