{"thread":{"id":"2719","subject":"[PATCH] Move \"-include config.mak\" to end of configuration section","startedAt":"2005-11-30T23:23:33Z","lastAt":"2005-12-03T02:31:52Z","messageCount":12,"participants":["Timo Hirvonen","Johannes Schindelin","Junio C Hamano","Alex Riesen"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"13019","messageId":"20051201012333.44bd81f2.tihirvon@gmail.com","threadId":"2719","inReplyTo":null,"subject":"[PATCH] Move \"-include config.mak\" to end of configuration section","fromName":"Timo Hirvonen","fromEmail":"tihirvon@gmail.com","sentAt":"2005-11-30T23:23:33Z","receivedAt":"2005-11-30T23:23:33Z","isPatch":true,"sender":{"key":"tihirvon@gmail.com","avatar":null},"body":"\nThis makes it possible to define WITH_SEND_EMAIL etc. in config.mak.\n\n---\n\n Makefile |    4 +---\n 1 files changed, 1 insertions(+), 3 deletions(-)\n\nfe5fece83b2ab0d67a4277d851faf50ad944cff4\ndiff --git a/Makefile b/Makefile\nindex 984d167..c44f5da 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -75,7 +75,7 @@ RPMBUILD = rpmbuild\n # explicitly what architecture to check for. Fix this up for yours..\n SPARSE_FLAGS = -D__BIG_ENDIAN__ -D__powerpc__\n \n-\n+-include config.mak\n \n ### --- END CONFIGURATION SECTION ---\n \n@@ -254,8 +254,6 @@ ifneq (,$(findstring arm,$(uname_M)))\n \tARM_SHA1 = YesPlease\n endif\n \n--include config.mak\n-\n ifndef NO_CURL\n \tifdef CURLDIR\n \t\t# This is still problematic -- gcc does not always want -R.\n-- \n0.99.9.GIT\n"},{"id":"13022","messageId":"Pine.LNX.4.63.0512010144050.11941@wbgn013.biozentrum.uni-wuerzburg.de","threadId":"2719","inReplyTo":"20051201012333.44bd81f2.tihirvon@gmail.com","subject":"Re: [PATCH] Move \"-include config.mak\" to end of configuration section","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2005-12-01T00:46:40Z","receivedAt":"2005-12-01T00:46:40Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Thu, 1 Dec 2005, Timo Hirvonen wrote:\n\n> This makes it possible to define WITH_SEND_EMAIL etc. in config.mak.\n\n... but it breaks setups where config.mak overrides platform \ndependent settings...\n\nHth,\nDscho\n"},{"id":"13023","messageId":"20051201033201.02b47071.tihirvon@gmail.com","threadId":"2719","inReplyTo":"Pine.LNX.4.63.0512010144050.11941@wbgn013.biozentrum.uni-wuerzburg.de","subject":"[PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Timo Hirvonen","fromEmail":"tihirvon@gmail.com","sentAt":"2005-12-01T01:32:01Z","receivedAt":"2005-12-01T01:32:01Z","isPatch":true,"sender":{"key":"tihirvon@gmail.com","avatar":null},"body":"\nThis makes it possible to define WITH_SEND_EMAIL etc. in config.mak.\n\nAlso remove GIT_LIST_TWEAK because it isn't used anymore.\n\n---\n\n Makefile |   28 ++++++++++++----------------\n 1 files changed, 12 insertions(+), 16 deletions(-)\n\nf877acdc3f8c888dd82a1e81ce20c8715e3108e4\ndiff --git a/Makefile b/Makefile\nindex 984d167..00521fe 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -138,8 +138,6 @@ ALL_PROGRAMS = $(PROGRAMS) $(SIMPLE_PROG\n # Backward compatibility -- to be removed after 1.0\n PROGRAMS += git-ssh-pull$X git-ssh-push$X\n \n-GIT_LIST_TWEAK =\n-\n # Set paths to tools early so that they can be used for version tests.\n ifndef SHELL_PATH\n \tSHELL_PATH = /bin/sh\n@@ -154,20 +152,6 @@ endif\n PYMODULES = \\\n \tgitMergeCommon.py\n \n-ifdef WITH_OWN_SUBPROCESS_PY\n-\tPYMODULES += compat/subprocess.py\n-else\n-\tifneq ($(shell $(PYTHON_PATH) -c 'import subprocess;print\"OK\"' 2>/dev/null),OK)\n-\t\tPYMODULES += compat/subprocess.py\n-\tendif\n-endif\n-\n-ifdef WITH_SEND_EMAIL\n-\tSCRIPT_PERL += git-send-email.perl\n-else\n-\tGIT_LIST_TWEAK += -e '/^send-email$$/d'\n-endif\n-\n LIB_FILE=libgit.a\n \n LIB_H = \\\n@@ -256,6 +240,18 @@ endif\n \n -include config.mak\n \n+ifdef WITH_OWN_SUBPROCESS_PY\n+\tPYMODULES += compat/subprocess.py\n+else\n+\tifneq ($(shell $(PYTHON_PATH) -c 'import subprocess;print\"OK\"' 2>/dev/null),OK)\n+\t\tPYMODULES += compat/subprocess.py\n+\tendif\n+endif\n+\n+ifdef WITH_SEND_EMAIL\n+\tSCRIPT_PERL += git-send-email.perl\n+endif\n+\n ifndef NO_CURL\n \tifdef CURLDIR\n \t\t# This is still problematic -- gcc does not always want -R.\n-- \n0.99.9.GIT\n"},{"id":"13024","messageId":"7vhd9tmw1g.fsf@assigned-by-dhcp.cox.net","threadId":"2719","inReplyTo":"20051201033201.02b47071.tihirvon@gmail.com","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-12-01T01:43:39Z","receivedAt":"2005-12-01T01:43:39Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Honestly speaking, I'd personally prefer to just get rid of\n\"-include config.mak\" from Makefile.  If somebody wants to keep\nher customized set of configuration, she can have her own ./Make\nscript that would look like:\n\n\t#!/bin/sh\n\tmake WITH_SEND_EMAIL=YesPlesae NO_EXPAT=NoThanks \"$@\"\n"},{"id":"13029","messageId":"Pine.LNX.4.63.0512010906400.22709@wbgn013.biozentrum.uni-wuerzburg.de","threadId":"2719","inReplyTo":"7vhd9tmw1g.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2005-12-01T08:08:27Z","receivedAt":"2005-12-01T08:08:27Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Wed, 30 Nov 2005, Junio C Hamano wrote:\n\n> Honestly speaking, I'd personally prefer to just get rid of \"-include \n> config.mak\" from Makefile.\n\nBut it is only one line, heck only 20 bytes!\n\nAnd you can still use your \"./Make\" method, and it is not widely \nadvertised, and you can override settings from the platform-dependent \ncode, and I feel much better at having my littly config.mak, my precious!\n\nCiao,\nDscho\n"},{"id":"13032","messageId":"7vmzjlkx2f.fsf@assigned-by-dhcp.cox.net","threadId":"2719","inReplyTo":"Pine.LNX.4.63.0512010906400.22709@wbgn013.biozentrum.uni-wuerzburg.de","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-12-01T09:04:24Z","receivedAt":"2005-12-01T09:04:24Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n\n> But it is only one line, heck only 20 bytes!\n\nHey, don't get so piped up.  Timo's patch has already been\nmerged (thanks, Timo).\n\nBut think about it a bit.\n\nIf you need an override, you have to write down and maintain\nthose YesPlease _somewhere_ yourself anyway, outside what I\nship.  Either \"config.mak\" or \"Make\" script.\n\nIf we have '-include' in the Makefile, we need to make a\ndecision if what we are adding to the Makefile should be\noverridable by that config.mak every time, exactly because\nwhatever is included becomes part of the Makefile.  IOW, that\n\"only 20 bytes\" adds work for the Makefile maintainer.\n\n\"Make\" script method is a command line override to the \"make\"\nprogram, which takes precedence and does not have that problem.\n"},{"id":"13034","messageId":"81b0412b0512010140m1b96db84x70c7d92f8f644a2a@mail.gmail.com","threadId":"2719","inReplyTo":"7vmzjlkx2f.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Alex Riesen","fromEmail":"raa.lkml@gmail.com","sentAt":"2005-12-01T09:40:30Z","receivedAt":"2005-12-01T09:40:30Z","isPatch":true,"sender":{"key":"raa.lkml@gmail.com","avatar":"https://avatars.githubusercontent.com/u/324101?v=4"},"body":"On 12/1/05, Junio C Hamano <junkio@cox.net> wrote:\n> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n>\n> > But it is only one line, heck only 20 bytes!\n>\n> Hey, don't get so piped up.  Timo's patch has already been\n> merged (thanks, Timo).\n>\n> But think about it a bit.\n>\n> If you need an override, you have to write down and maintain\n> those YesPlease _somewhere_ yourself anyway, outside what I\n> ship.  Either \"config.mak\" or \"Make\" script.\n>\n> If we have '-include' in the Makefile, we need to make a\n> decision if what we are adding to the Makefile should be\n> overridable by that config.mak every time, exactly because\n> whatever is included becomes part of the Makefile.  IOW, that\n> \"only 20 bytes\" adds work for the Makefile maintainer.\n>\n> \"Make\" script method is a command line override to the \"make\"\n> program, which takes precedence and does not have that problem.\n>\n\nI have even my specific targets in the config.mak (and would actually\nlike to have the default target put before including config.mak, so\ndefault call to make is always the same).\n"},{"id":"13035","messageId":"7viru9jgku.fsf@assigned-by-dhcp.cox.net","threadId":"2719","inReplyTo":"81b0412b0512010140m1b96db84x70c7d92f8f644a2a@mail.gmail.com","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-12-01T09:45:53Z","receivedAt":"2005-12-01T09:45:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alex Riesen <raa.lkml@gmail.com> writes:\n\n> I have even my specific targets in the config.mak (and would actually\n> like to have the default target put before including config.mak, so\n> default call to make is always the same).\n\nSurely you can do that by having your own \"makefile\" and include\n\"Makefile\" I ship, and run \"make\", which prefers makefile over\nMakefile ;-).\n"},{"id":"13036","messageId":"81b0412b0512010153t3f768a82m8f9a12effa4a715e@mail.gmail.com","threadId":"2719","inReplyTo":"7viru9jgku.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Alex Riesen","fromEmail":"raa.lkml@gmail.com","sentAt":"2005-12-01T09:53:09Z","receivedAt":"2005-12-01T09:53:09Z","isPatch":true,"sender":{"key":"raa.lkml@gmail.com","avatar":"https://avatars.githubusercontent.com/u/324101?v=4"},"body":"On 12/1/05, Junio C Hamano <junkio@cox.net> wrote:\n> > I have even my specific targets in the config.mak (and would actually\n> > like to have the default target put before including config.mak, so\n> > default call to make is always the same).\n>\n> Surely you can do that by having your own \"makefile\" and include\n> \"Makefile\" I ship, and run \"make\", which prefers makefile over\n> Makefile ;-).\n\nBut that is not the same:\n - I can put my targets and variables either before or after Makefile,\nnot somewhere inbetween\n - I have to work on an especially brain-damaged filesystems, like FAT\n"},{"id":"13038","messageId":"7v7japjf53.fsf@assigned-by-dhcp.cox.net","threadId":"2719","inReplyTo":"81b0412b0512010153t3f768a82m8f9a12effa4a715e@mail.gmail.com","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-12-01T10:16:56Z","receivedAt":"2005-12-01T10:16:56Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alex Riesen <raa.lkml@gmail.com> writes:\n\n> On 12/1/05, Junio C Hamano <junkio@cox.net> wrote:\n>> > I have even my specific targets in the config.mak (and would actually\n>> > like to have the default target put before including config.mak, so\n>> > default call to make is always the same).\n\nHmph.  Do you mean something like this?\n\n-- >8 --\ndiff --git a/Makefile b/Makefile\nindex 00521fe..45db357 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -1,3 +1,6 @@\n+# The default target of this Makefile is...\n+all:\n+\n # Define MOZILLA_SHA1 environment variable when running make to make use of\n # a bundled SHA1 routine coming from Mozilla. It is GPL'd and should be fast\n # on non-x86 architectures (e.g. PowerPC), while the OpenSSL version (default\n"},{"id":"13042","messageId":"81b0412b0512010451q14e2e833o86f59399ba83188b@mail.gmail.com","threadId":"2719","inReplyTo":"7v7japjf53.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Alex Riesen","fromEmail":"raa.lkml@gmail.com","sentAt":"2005-12-01T12:51:33Z","receivedAt":"2005-12-01T12:51:33Z","isPatch":true,"sender":{"key":"raa.lkml@gmail.com","avatar":"https://avatars.githubusercontent.com/u/324101?v=4"},"body":"On 12/1/05, Junio C Hamano <junkio@cox.net> wrote:\n> Alex Riesen <raa.lkml@gmail.com> writes:\n>\n> > On 12/1/05, Junio C Hamano <junkio@cox.net> wrote:\n> >> > I have even my specific targets in the config.mak (and would actually\n> >> > like to have the default target put before including config.mak, so\n> >> > default call to make is always the same).\n>\n> Hmph.  Do you mean something like this?\n>\n> -- >8 --\n> diff --git a/Makefile b/Makefile\n> index 00521fe..45db357 100644\n> --- a/Makefile\n> +++ b/Makefile\n> @@ -1,3 +1,6 @@\n> +# The default target of this Makefile is...\n> +all:\n> +\n\nYes. In fact, that is what I have in my tree :)\n"},{"id":"13131","messageId":"Pine.LNX.4.63.0512030323290.19086@wbgn013.biozentrum.uni-wuerzburg.de","threadId":"2719","inReplyTo":"7vmzjlkx2f.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] Move couple of ifdefs after \"include config.mk\"","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2005-12-03T02:31:52Z","receivedAt":"2005-12-03T02:31:52Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Thu, 1 Dec 2005, Junio C Hamano wrote:\n\n> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n> \n> > But it is only one line, heck only 20 bytes!\n> \n> Hey, don't get so piped up.  Timo's patch has already been\n> merged (thanks, Timo).\n\nIn case I did not make it clear: I was not mad or something like that. I \nonly see 3 ways to go about that particular problem:\n\n- maintain your private config.mak\n- maintain your private Make (shell script)\n- maintain your private Makefile\n\nI actually did the third way for a while, always merging the newest \nversion. But all of a sudden -- when I was working on another project, \nwhich *has* a config.mak -- I had the idea to do the first way.\n\nActually, the second way falls a little bit short for some of my \napplications: I sometimes override settings in the first part of the \nMakefile. This is not possible with Make.\n\n> If you need an override, you have to write down and maintain\n> those YesPlease _somewhere_ yourself anyway, outside what I\n> ship.  Either \"config.mak\" or \"Make\" script.\n\nAs mentioned, with the difference that you just can't override some things \nin Make.\n\n> If we have '-include' in the Makefile, we need to make a\n> decision if what we are adding to the Makefile should be\n> overridable by that config.mak every time, exactly because\n> whatever is included becomes part of the Makefile.  IOW, that\n> \"only 20 bytes\" adds work for the Makefile maintainer.\n\n... which might have another (desired) effect: the structure of the \nMakefile is better. More organized.\n\nBesides, if you change things in an organized Makefile, you tend to be a \nlittle bit more careful where you put things, so the cost of maintaining \nis not really high (happened to me...).\n\nHth,\nDscho\n"}]}