{"thread":{"id":"50022","subject":"Preparing for 2.20.1 brown-paper-bag maintenance release","startedAt":"2018-12-13T06:29:33Z","lastAt":"2018-12-14T02:40:16Z","messageCount":6,"participants":["Junio C Hamano","Johannes Schindelin","Ævar Arnfjörð Bjarmason","Duy Nguyen"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"365229","messageId":"xmqq4lbiey7s.fsf@gitster-ct.c.googlers.com","threadId":"50022","inReplyTo":null,"subject":"Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-12-13T06:29:27Z","receivedAt":"2018-12-13T06:29:33Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here is an excerpt from a draft edition of \"What's cooking\" report\nfor topics that are about an immediate 2.20.1 maintenance release,\nwith five topics that I plan to merge to 'next' and then to 'maint'\nsoonish (they're all marked as \"Will merge to 'next' and then to\n'master'\").\n\nThey'll be in 'pu' but not in 'next' in today's pushout, but unless\nI hear breakage reports in time, I am hoping to merge them to 'next'\nduring tomorrow's integration cycle, so that we can start the new\nweek with 2.20.1.\n\nThanks.\n\n--------------------------------------------------\n\n* ds/hash-independent-tests-fix (2018-12-12) 1 commit\n - .gitattributes: ensure t/oid-info/* has eol=lf\n\n Test portability fix.\n\n\n* jc/run-command-report-exec-failure-fix (2018-12-12) 1 commit\n - run-command: report exec failure\n\n A recent update accidentally squelched an error message when the\n run_command API failed to run a missing command, which has been\n corrected.\n\n\n* nd/show-gitcomp-compilation-fix (2018-12-12) 1 commit\n - parse-options: fix SunCC compiler warning\n\n Portability fix for a recent update to parse-options API.\n\n\n* js/help-commands-verbose-by-default-fix (2018-12-12) 2 commits\n - help -a: handle aliases with long names gracefully\n - help.h: fix coding style\n\n \"git help -a\" did not work well when an overly long alias is\n defined, which has been corrected.\n\n\n* js/mailinfo-format-flowed-fix (2018-12-13) 1 commit\n - t4256: mark support files as LF-only\n\n Test portability fix.\n"},{"id":"365241","messageId":"nycvar.QRO.7.76.6.1812131337270.43@tvgsbejvaqbjf.bet","threadId":"50022","inReplyTo":"xmqq4lbiey7s.fsf@gitster-ct.c.googlers.com","subject":"Re: Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-12-13T12:50:54Z","receivedAt":"2018-12-13T12:51:18Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Thu, 13 Dec 2018, Junio C Hamano wrote:\n\n> * ds/hash-independent-tests-fix (2018-12-12) 1 commit\n>  - .gitattributes: ensure t/oid-info/* has eol=lf\n> \n>  Test portability fix.\n>\n> [...]\n>\n> * js/mailinfo-format-flowed-fix (2018-12-13) 1 commit\n>  - t4256: mark support files as LF-only\n> \n>  Test portability fix.\n\nWe found one more issue that I'd like to get fixed while we're at it: \none test case in t9902 now needs the PERL prerequisite.\n\nLet me coordinate with Stolee who will send this.\n\nCiao,\nDscho\n"},{"id":"365255","messageId":"87r2el1q0g.fsf@evledraar.gmail.com","threadId":"50022","inReplyTo":"xmqq4lbiey7s.fsf@gitster-ct.c.googlers.com","subject":"Re: Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2018-12-13T14:05:03Z","receivedAt":"2018-12-13T14:05:08Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Dec 13 2018, Junio C Hamano wrote:\n\n> Here is an excerpt from a draft edition of \"What's cooking\" report\n> for topics that are about an immediate 2.20.1 maintenance release,\n> with five topics that I plan to merge to 'next' and then to 'maint'\n> soonish (they're all marked as \"Will merge to 'next' and then to\n> 'master'\").\n>\n> They'll be in 'pu' but not in 'next' in today's pushout, but unless\n> I hear breakage reports in time, I am hoping to merge them to 'next'\n> during tomorrow's integration cycle, so that we can start the new\n> week with 2.20.1.\n> [...]\n> * nd/show-gitcomp-compilation-fix (2018-12-12) 1 commit\n>  - parse-options: fix SunCC compiler warning\n>\n>  Portability fix for a recent update to parse-options API.\n\nSince I reported this, just a clarification: Unlike 46c0eb5843\n(\"files-backend.c: fix build error on Solaris\", 2018-11-25) this one's\nnot an error on suncc, just a warning (and we have 20-30 of those exact\nwarnings in our code).\n\nSo if you wanted to minimize 2.20.1 this could be held back, but also it\nlooks obviously correct so it's fine that it makes it into that point\nrelease. Just FYI.\n"},{"id":"365278","messageId":"CACsJy8AB0gQAvAWh3vtiSFnZWXtdvQdi4czBoR2B8TkECMrQtQ@mail.gmail.com","threadId":"50022","inReplyTo":"87r2el1q0g.fsf@evledraar.gmail.com","subject":"Re: Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2018-12-13T18:08:00Z","receivedAt":"2018-12-13T18:08:28Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Thu, Dec 13, 2018 at 3:05 PM Ævar Arnfjörð Bjarmason\n<avarab@gmail.com> wrote:\n>\n>\n> On Thu, Dec 13 2018, Junio C Hamano wrote:\n>\n> > Here is an excerpt from a draft edition of \"What's cooking\" report\n> > for topics that are about an immediate 2.20.1 maintenance release,\n> > with five topics that I plan to merge to 'next' and then to 'maint'\n> > soonish (they're all marked as \"Will merge to 'next' and then to\n> > 'master'\").\n> >\n> > They'll be in 'pu' but not in 'next' in today's pushout, but unless\n> > I hear breakage reports in time, I am hoping to merge them to 'next'\n> > during tomorrow's integration cycle, so that we can start the new\n> > week with 2.20.1.\n> > [...]\n> > * nd/show-gitcomp-compilation-fix (2018-12-12) 1 commit\n> >  - parse-options: fix SunCC compiler warning\n> >\n> >  Portability fix for a recent update to parse-options API.\n>\n> Since I reported this, just a clarification: Unlike 46c0eb5843\n> (\"files-backend.c: fix build error on Solaris\", 2018-11-25) this one's\n> not an error on suncc, just a warning (and we have 20-30 of those exact\n> warnings in our code).\n>\n> So if you wanted to minimize 2.20.1 this could be held back, but also it\n> looks obviously correct so it's fine that it makes it into that point\n> release. Just FYI.\n\nThere's also a bug in my patch (-2 is already being used by\nparse_opt_unknown_cb and my patch will change behavior of git-blame at\nleast in theory). I'll reroll later. I think it's better to hold it\nback.\n-- \nDuy\n"},{"id":"365279","messageId":"CACsJy8CexGXtyBa1oNP0jVx-+8JPTcfTd5wf1ZyfJWf8HDtLVQ@mail.gmail.com","threadId":"50022","inReplyTo":"CACsJy8AB0gQAvAWh3vtiSFnZWXtdvQdi4czBoR2B8TkECMrQtQ@mail.gmail.com","subject":"Re: Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2018-12-13T18:17:49Z","receivedAt":"2018-12-13T18:18:17Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Thu, Dec 13, 2018 at 7:08 PM Duy Nguyen <pclouds@gmail.com> wrote:\n> There's also a bug in my patch (-2 is already being used by\n> parse_opt_unknown_cb and my patch will change behavior of git-blame at\n> least in theory).\n\nAh no. Too many magic numbers in parse-options.c code. It's working\nfine but I'll need to give these some names to avoid confusion in the\nfuture.\n-- \nDuy\n"},{"id":"365320","messageId":"xmqq1s6kesqb.fsf@gitster-ct.c.googlers.com","threadId":"50022","inReplyTo":"CACsJy8CexGXtyBa1oNP0jVx-+8JPTcfTd5wf1ZyfJWf8HDtLVQ@mail.gmail.com","subject":"Re: Preparing for 2.20.1 brown-paper-bag maintenance release","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-12-14T02:40:12Z","receivedAt":"2018-12-14T02:40:16Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Duy Nguyen <pclouds@gmail.com> writes:\n\n> On Thu, Dec 13, 2018 at 7:08 PM Duy Nguyen <pclouds@gmail.com> wrote:\n>> There's also a bug in my patch (-2 is already being used by\n>> parse_opt_unknown_cb and my patch will change behavior of git-blame at\n>> least in theory).\n>\n> Ah no. Too many magic numbers in parse-options.c code. It's working\n> fine but I'll need to give these some names to avoid confusion in the\n> future.\n\nOK.\n\n"}]}