{"thread":{"id":"25087","subject":"[PATCH] Makefile: add CC to TRACK_CFLAGS","startedAt":"2010-09-12T22:37:45Z","lastAt":"2010-09-20T13:38:05Z","messageCount":4,"participants":["Ævar Arnfjörð Bjarmason","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"150563","messageId":"1284331065-3394-1-git-send-email-avarab@gmail.com","threadId":"25087","inReplyTo":null,"subject":"[PATCH] Makefile: add CC to TRACK_CFLAGS","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2010-09-12T22:37:45Z","receivedAt":"2010-09-12T22:37:45Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"Change the git make process so that everything's rebuilt if the CC is\nchanged. Before we wouldn't rebuilt if e.g. the CC variable was\nchanged from gcc to clang.\n\nSigned-off-by: Ævar Arnfjörð Bjarmason \n---\n Makefile | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Makefile b/Makefile\nindex c27e8bc..c46893a 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -1969,7 +1969,7 @@ cscope:\n \t$(FIND) . -name '*.[hcS]' -print | xargs cscope -b\n \n ### Detect prefix changes\n-TRACK_CFLAGS = $(subst ','\\'',$(ALL_CFLAGS)):\\\n+TRACK_CFLAGS = $(CC):$(subst ','\\'',$(ALL_CFLAGS)):\\\n $(bindir_SQ):$(gitexecdir_SQ):$(template_dir_SQ):$(prefix_SQ)\n \n GIT-CFLAGS: FORCE\n-- \n1.7.2.3.313.gcd15\n"},{"id":"151006","messageId":"AANLkTikBP3pJ7ZN9jC9yQeVKssgr2KMpaY-nQtpJZiP-@mail.gmail.com","threadId":"25087","inReplyTo":"1284331065-3394-1-git-send-email-avarab@gmail.com","subject":"Re: [PATCH] Makefile: add CC to TRACK_CFLAGS","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2010-09-19T19:27:03Z","receivedAt":"2010-09-19T19:27:03Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"On Sun, Sep 12, 2010 at 22:37, Ævar Arnfjörð Bjarmason wrote:\n> Change the git make process so that everything's rebuilt if the CC is\n> changed. Before we wouldn't rebuilt if e.g. the CC variable was\n> changed from gcc to clang.\n\nIs there a reason for why this didn't get picked up other than falling\nthrough the cracks?\n"},{"id":"151039","messageId":"7viq21uqzb.fsf@alter.siamese.dyndns.org","threadId":"25087","inReplyTo":"AANLkTikBP3pJ7ZN9jC9yQeVKssgr2KMpaY-nQtpJZiP-@mail.gmail.com","subject":"Re: [PATCH] Makefile: add CC to TRACK_CFLAGS","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-09-20T01:46:16Z","receivedAt":"2010-09-20T01:46:16Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason writes:\n\n> Is there a reason for why this didn't get picked up other than falling\n> through the cracks?\n\nSimply because I wasn't actively collecting new topics during feature\nfreeze, especially for small stuff that I knew that resending after\nrelease would be trivial and more efficient use of my time than queuing it\nin 'pu' and having to look at it every time I do another push-out in order\nto decide when to merge it to 'next'.\n\nAfter a feature release like 1.7.3, my preference is to do these in this\norder:\n\n (0) take a deep breath and have bit of break ;-);\n\n (1) rewind 'next' to match 1.7.3 with selected topics that has been\n cooking there, rename 'maint' to 'maint-1.7.2', and start 'maint' at\n 1.7.3;\n\n (2) have a quiet period for at least a few days, not touching 'master' at\n all, to make sure things are stable, giving users time to give us\n feedback, and giving us time to assess potential damages caused by\n unanticipated regressions;\n\n (3) collect brown-paper-bag regression fixes, if necessary, during the\n period overlapping with (2), queuing them to 'next';\n\n (4) have (3) graduate and issue 1.7.3.1 out of 'maint';\n\nAnd after all that, open the 1.7.4 cycle, queuing new topics. That would\nalso be a good time to backmerge the safe fixes (but not new features)\nthat went to 1.7.3 to maint-1.7.2, and cut 1.7.2.4, so that it will be a\nreasonably solid base to apply later security fixes if necessary.\n"},{"id":"151116","messageId":"AANLkTikLCFb3GrMxP7dLPB=eggc2w6FOq=7ZpdSYpnki@mail.gmail.com","threadId":"25087","inReplyTo":"7viq21uqzb.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Makefile: add CC to TRACK_CFLAGS","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2010-09-20T13:38:05Z","receivedAt":"2010-09-20T13:38:05Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"On Mon, Sep 20, 2010 at 01:46, Junio C Hamano wrote:\n> Ævar Arnfjörð Bjarmason writes:\n>\n>> Is there a reason for why this didn't get picked up other than falling\n>> through the cracks?\n>\n> Simply because I wasn't actively collecting new topics during feature\n> freeze, especially for small stuff that I knew that resending after\n> release would be trivial and more efficient use of my time than queuing it\n> in 'pu' and having to look at it every time I do another push-out in order\n> to decide when to merge it to 'next'.\n\nRigth, I have no problem doing $whatever to get patches in, but the\nproblem I often have is that I don't know what state things are in,\nand what I'm supposed to do at any given time.\n\nE.g. in this case I submitted the \"send-email: use catfile() to\nconcatenate files\" patch 2 days after this one, that one got into the\nnext \"What's cooking in git.git\" post.\n\nSince both were trivial fixes and there was no comment on this one I\nwas inclined to think that it just fell through.\n\nShould I generally re-send patches that I've sent, haven't had\ncomments, and haven't appeared in subsequent \"What's cooking in\ngit.git\" posts (given that some reasonable amount of time has passed\nsince the original send) ?\n\nThen there's stuff like my \"git-am: Ignore whitespace before patches\"\nwhich had some comments, but which *I* still think is OK as-is. Should\nI just keep pushing stuff like that until someone tells me to stop?\n\nThanks, from a list member somewhat confused about the patch queue :)\n"}]}