{"thread":{"id":"39407","subject":"[Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","startedAt":"2015-05-22T08:33:20Z","lastAt":"2015-05-23T14:29:23Z","messageCount":21,"participants":["Roberto Tyley","Johannes Schindelin","Sebastian Schuberth","Stefan Näwe","Junio C Hamano","Matthieu Moy","Stefan Beller","Philip Oakley"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"261890","messageId":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","threadId":"39407","inReplyTo":null,"subject":"[Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Roberto Tyley","fromEmail":"roberto.tyley@gmail.com","sentAt":"2015-05-22T08:33:20Z","receivedAt":"2015-05-22T08:33:20Z","isPatch":false,"sender":{"key":"roberto.tyley@gmail.com","avatar":"https://avatars.githubusercontent.com/u/52038?v=4"},"body":"On Tuesday, 19 May 2015, Stefan Beller wrote:\n> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n> wrote:\n> > How do you send your patches inline?\n[snip]\n> This workflow discussion was a topic at the GitMerge2015 conference,\n> and there are essentially 2 groups, those who know how to send email\n> and those who complain about it. A solution was agreed on by nearly all\n> of the contributors. It would be awesome to have a git-to-email proxy,\n> such that you could do a git push master:refs/for/mailinglist\n> and this proxy would convert the push into sending patch series to the\n> mailing list. It could even convert the following discussion back into\n> comments (on Github?) but as a first step we'd want to try out a one\n> way proxy.\n>\n> Unfortunately nobody stepped up to actually do the work, yet :(\n\n\nHello, I'm stepping up to do that work :) Or at least, I'm implementing a\none-way GitHub PR -> Mailing list tool, called submitGit:\n\nhttps://submitgit.herokuapp.com/\n\nHere's what a user does:\n\n* create a PR on https://github.com/git/git\n* logs into https://submitgit.herokuapp.com/ with GitHub auth\n* selects their PR on https://submitgit.herokuapp.com/git/git/pulls\n* gets submitGit to email the PR as patches to themselves, in order to\ncheck it looks ok\n* when they're ready, get submitGit to send it to the mailing list on\ntheir behalf\n\nAll discussion of the patch *stays* on the mailing list - I'm not\nattempting to change\nanything about the Git community process, other than make it easier\nfor a wider group\npeople to submit patches to the list.\n\nFor hard-core contributors to Git, I'd imagine that git format-patch &\nsend-email\nremain the fastest way to do their work. But those tools are _unfamiliar to the\nmajority of Git users_ - so submitGit aims to cater to those users, because they\ndefinitely have valuable contributions to make, which would be tragic\nto throw away.\n\nI've been working on submitGit in my spare time for the past few\nweeks, and there\nare still features I plan to add (like guiding the user to more\n'correct' word wrapping,\nsign-off, etc), but given this discussion, I wanted to chime in and\nlet people know\nwhat's here so far. It would be great if people could take the time to\nexplore the tool\n(you don't have to raise a git/git PR in order to try sending one *to\nyourself*, for\ninstance) and give feedback on list, or in GitHub issues:\n\nhttps://github.com/rtyley/submitgit/issues\n\nI've been lucky enough to discuss the ideas around submitGit with a\nfew people at\nthe Git-Merge conf, so thanks to Peff, Thomas Ferris Nicolaisen, and Emma Jane\nHogbin Westby for listening to me (not to imply their endorsement of\nwhat I've done,\njust thanks for talking about it!).\n\nRoberto\n"},{"id":"261893","messageId":"73db761f3c737965f64ac45005f7a716@www.dscho.org","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-22T09:42:58Z","receivedAt":"2015-05-22T09:42:58Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Roberto,\n\nOn 2015-05-22 10:33, Roberto Tyley wrote:\n> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>> wrote:\n>> > How do you send your patches inline?\n> [snip]\n>> This workflow discussion was a topic at the GitMerge2015 conference,\n>> and there are essentially 2 groups, those who know how to send email\n>> and those who complain about it. A solution was agreed on by nearly all\n>> of the contributors. It would be awesome to have a git-to-email proxy,\n>> such that you could do a git push master:refs/for/mailinglist\n>> and this proxy would convert the push into sending patch series to the\n>> mailing list. It could even convert the following discussion back into\n>> comments (on Github?) but as a first step we'd want to try out a one\n>> way proxy.\n>>\n>> Unfortunately nobody stepped up to actually do the work, yet :(\n> \n> \n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n> \n> https://submitgit.herokuapp.com/\n\nWow!!!\n\nI will make sure to test it with a couple of patches I want to submit anyway.\n\nThanks so much!\nDscho\n"},{"id":"261899","messageId":"555F29B9.1040805@gmail.com","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2015-05-22T13:06:01Z","receivedAt":"2015-05-22T13:06:01Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On 5/22/2015 10:33, Roberto Tyley wrote:\n\n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n>\n> https://submitgit.herokuapp.com/\n\nThat's fantastic! Me being the one who brought up that topic at the Git \nMerge contributor's summit, I can say this sounds *very* much like the \ntool I've envisioned. Many thanks for that, I'll be sure to give it a \ntry as soon as I can.\n\nBest regards,\nSebastian\n"},{"id":"261903","messageId":"555F2E3A.1040600@atlas-elektronik.com","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Stefan Näwe","fromEmail":"stefan.naewe@atlas-elektronik.com","sentAt":"2015-05-22T13:25:14Z","receivedAt":"2015-05-22T13:25:14Z","isPatch":false,"sender":{"key":"stefan.naewe@gmail.com","avatar":"https://avatars.githubusercontent.com/u/4468?v=4"},"body":"Am 22.05.2015 um 10:33 schrieb Roberto Tyley:\n> [...]\n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n> \n> https://submitgit.herokuapp.com/\n\nThat looks really promising!\nI wonder if that wouldn't make a good addition to github's repository ui\nin general ?\n\nThanks,\n Stefan\n-- \n----------------------------------------------------------------\n/dev/random says: Diplomacy: The patriotic art of lying for one's country.\npython -c \"print '73746566616e2e6e616577654061746c61732d656c656b74726f6e696b2e636f6d'.decode('hex')\" \nGPG Key fingerprint = 2DF5 E01B 09C3 7501 BCA9 9666 829B 49C5 9221 27AF\n"},{"id":"261909","messageId":"xmqq8ucghf2s.fsf@gitster.dls.corp.google.com","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-05-22T14:17:15Z","receivedAt":"2015-05-22T14:17:15Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Roberto Tyley writes:\n\n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n>\n> https://submitgit.herokuapp.com/\n\nYay ;-)\n\n> Here's what a user does:\n>\n> * create a PR on https://github.com/git/git\n> * logs into https://submitgit.herokuapp.com/ with GitHub auth\n> * selects their PR on https://submitgit.herokuapp.com/git/git/pulls\n\nReasonable.\n\n> * gets submitGit to email the PR as patches to themselves, in order to\n> check it looks ok\n\nI can see you are trying to be careful by doing this, but I am not\nsure if this step would actually help. Those who are not familiar\nwith Git development are not expected to know what is \"ok\" in their\noriginal commit, and if they find bad formatting done by submitGit\n(e.g. adds their PR message before the three-dash line instead of\nafter it), they cannot do much about it anyway.\n\n> * when they're ready, get submitGit to send it to the mailing list on\n> their behalf\n\nNice.\n\n> All discussion of the patch *stays* on the mailing list\n\nCan you identify a reroll of an earlier submission? If you can use\nthe in-reply-to and make it a follow-up to the previous round, that\nwould be great.\n"},{"id":"261914","messageId":"5b4bab2727a9fcee3a07724c64752cf9@www.dscho.org","threadId":"39407","inReplyTo":"73db761f3c737965f64ac45005f7a716@www.dscho.org","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-22T14:41:24Z","receivedAt":"2015-05-22T14:41:24Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Roberto,\n\nOn 2015-05-22 11:42, Johannes Schindelin wrote:\n\n> On 2015-05-22 10:33, Roberto Tyley wrote:\n>> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>>> wrote:\n>>> > How do you send your patches inline?\n>> [snip]\n>>> This workflow discussion was a topic at the GitMerge2015 conference,\n>>> and there are essentially 2 groups, those who know how to send email\n>>> and those who complain about it. A solution was agreed on by nearly all\n>>> of the contributors. It would be awesome to have a git-to-email proxy,\n>>> such that you could do a git push master:refs/for/mailinglist\n>>> and this proxy would convert the push into sending patch series to the\n>>> mailing list. It could even convert the following discussion back into\n>>> comments (on Github?) but as a first step we'd want to try out a one\n>>> way proxy.\n>>>\n>>> Unfortunately nobody stepped up to actually do the work, yet :(\n>>\n>>\n>> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n>> one-way GitHub PR -> Mailing list tool, called submitGit:\n>>\n>> https://submitgit.herokuapp.com/\n> \n> Wow!!!\n> \n> I will make sure to test it with a couple of patches I want to submit anyway.\n\nI just tried this with https://github.com/git/git/pull/139 and would like to tell you about two wishes I had immediately:\n\n- If the author of a patch I am submitting is not myself, could submitGit maybe add that `From: ` line at the top of the mail?\n- The patch series is sent without a cover letter, but it would be *really* great if a path series consisting of more than one patch could have the initial comment of the Pull Request as a cover letter, with the link to the original Pull Request at the bottom? This would also be the mail to use in the \"In-reply-yo\" header instead of the first patch.\n\nThanks so much!\nDscho\n"},{"id":"261922","messageId":"vpqd21slhdc.fsf@anie.imag.fr","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2015-05-22T16:14:07Z","receivedAt":"2015-05-22T16:14:07Z","isPatch":false,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Roberto Tyley writes:\n\n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n>\n> https://submitgit.herokuapp.com/\n\nThis is absolutely awsome. A few thoughts after testing the system to\nsend two patches:\n\n* This is awesome :-). Really.\n\n* I found no way to specify a version like [PATCH v2 ...]. Similarly,\n there could be a way to say [RFC/PATCH ...].\n\n* I'm used to 'git send-email' and I have an alias to add the --signoff\n for me. I just sent a series where I forgot the signed-off-by. The\n system could warn if I didn't signoff.\n\n* I had a text+subject for the pull-request, I was expecting a cover\n letter email with them.\n\n* A simple way to add \"below-triple-dash\" comments would be cool. For\n now, I guess we can do this by having the --- within the commit\n message.\n\n* The explanation on how to register one's email were unclear to me. It\n told me that I had to register my email without telling how, and that\n I had to send the series to myself before. Once I sent the series to\n myself I got better explanation, hence reversing the order of the\n advices would already be an improvement.\n\n* The link to the original PR could be more visible. For now, I can get\n to the PR by clicking the octocat icon, but perhaps a text \"click here\n to view the PR on GitHub\" or a link on the whole title would have been\n easier to find.\n\n* I missed a way to specify In-reply-to: to follow up to my previous\n version.\n\n* The submitGit link on the top left does nothing for me. I would expect\n to come back to the home page of submitGit.\n\n* For completess: Junio just noticed in another thread that the patch\n order is reversed, PATCH 2/2 predates PATCH 1/2 and the second is\n In-Reply-To: the first.\n\n* I was about to suggest that you post a comment on the GitHub pull\n request when sending the email, but it is already the case, with a\n link to Gmane. Very very cool.\n\n* Did I mention how awesome the application is already? 'Cause it really\n is!\n\nThanks a lot for doing this!\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"},{"id":"261923","messageId":"xmqq4mn4fugi.fsf@gitster.dls.corp.google.com","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-05-22T16:27:57Z","receivedAt":"2015-05-22T16:27:57Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Roberto Tyley writes:\n\n> Here's what a user does:\n>\n> * create a PR on https://github.com/git/git\n> * logs into https://submitgit.herokuapp.com/ with GitHub auth\n\nHmm, this seems to request too much authorization, though.\n\n Repositories\n Public only\n This application will be able to read and write all public repo\n data. This includes the following:\n\n Code\n Issues\n Pull requests\n Wikis\n Settings\n Webhooks and services\n Deploy keys\n\nI really wanted to try this out, but I do not think, as the owner of\na reasonably important repository, it would be irresponsible for me\nto grant write access to Code or Settings for my primary GitHub\naccount. Also I think you reject an account that is too young (I\nfound out about submitGit via your comment on a pull request to\ngit/git and read its source before reading your message I am\nresponding to, and that was the impression I got from the recent log\nmessages there), so I cannot create and try with a throw-away\naccount, either.\n\nThat would mean that I cannot join the fun X-<.\n"},{"id":"261926","messageId":"CAGZ79kbeATbzSGHd5ub-NphB8=hxKfzv40Z1HucsC_EtEotv2Q@mail.gmail.com","threadId":"39407","inReplyTo":"CAFY1edY3+Wt-p2iQ5k64Fg-nMk2PmRSvhVkQSVNw94R18uPV2Q@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2015-05-22T16:53:26Z","receivedAt":"2015-05-22T16:53:26Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Fri, May 22, 2015 at 1:33 AM, Roberto Tyley wrote:\n> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>> wrote:\n>> > How do you send your patches inline?\n> [snip]\n>> This workflow discussion was a topic at the GitMerge2015 conference,\n>> and there are essentially 2 groups, those who know how to send email\n>> and those who complain about it. A solution was agreed on by nearly all\n>> of the contributors. It would be awesome to have a git-to-email proxy,\n>> such that you could do a git push master:refs/for/mailinglist\n>> and this proxy would convert the push into sending patch series to the\n>> mailing list. It could even convert the following discussion back into\n>> comments (on Github?) but as a first step we'd want to try out a one\n>> way proxy.\n>>\n>> Unfortunately nobody stepped up to actually do the work, yet :(\n>\n>\n> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n> one-way GitHub PR -> Mailing list tool, called submitGit:\n\nCool!\nI will try that with the next patch I want to submit.\n\n>\n> https://submitgit.herokuapp.com/\n>\n> Here's what a user does:\n>\n> * create a PR on https://github.com/git/git\n\nWhen looking at https://github.com/git/git\n\n Git Source Code Mirror - This is a publish-only repository and all\n pull requests are ignored. Please follow Documentation/SubmittingPatches\n procedure for any of your improvements.\n\nOnce this tool has proven to be usable (in a few days?), we want to reword that.\n\n Guidelines for submitting patches to Git. Half this document covers how\n to send a patch via email without it getting corrupted - which submitGit\n will do for you - but the other half is very useful, giving guidance on what\n good patches for the Git project should look like.\n\nIf it turns out this tool is widely used we may want to consider splitting up\nSubmittingPatches inside git.git into two files, one dealing with the contents\ni.e. How to write good, reviewable commits, following the coding guide lines\nand having a proper sign off, and another document on how to get your\ncontributions\nupstream (email, pull request, ...)\n\n> * logs into https://submitgit.herokuapp.com/ with GitHub auth\n> * selects their PR on https://submitgit.herokuapp.com/git/git/pulls\n> * gets submitGit to email the PR as patches to themselves, in order to\n> check it looks ok\n> * when they're ready, get submitGit to send it to the mailing list on\n> their behalf\n>\n> All discussion of the patch *stays* on the mailing list - I'm not\n> attempting to change\n> anything about the Git community process, other than make it easier\n> for a wider group\n> people to submit patches to the list.\n>\n> For hard-core contributors to Git, I'd imagine that git format-patch &\n> send-email\n> remain the fastest way to do their work. But those tools are _unfamiliar to the\n> majority of Git users_ - so submitGit aims to cater to those users, because they\n> definitely have valuable contributions to make, which would be tragic\n> to throw away.\n>\n> I've been working on submitGit in my spare time for the past few\n> weeks, and there\n> are still features I plan to add (like guiding the user to more\n> 'correct' word wrapping,\n> sign-off, etc), but given this discussion, I wanted to chime in and\n> let people know\n> what's here so far. It would be great if people could take the time to\n> explore the tool\n> (you don't have to raise a git/git PR in order to try sending one *to\n> yourself*, for\n> instance) and give feedback on list, or in GitHub issues:\n>\n> https://github.com/rtyley/submitgit/issues\n>\n> I've been lucky enough to discuss the ideas around submitGit with a\n> few people at\n> the Git-Merge conf, so thanks to Peff, Thomas Ferris Nicolaisen, and Emma Jane\n> Hogbin Westby for listening to me (not to imply their endorsement of\n> what I've done,\n> just thanks for talking about it!).\n\nWow!\nThanks for doing this,\nStefan\n\n>\n> Roberto\n"},{"id":"261929","messageId":"0B3203305DF54249930E5903864B4988@PhilipOakley","threadId":"39407","inReplyTo":"5b4bab2727a9fcee3a07724c64752cf9@www.dscho.org","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":"2015-05-22T17:14:51Z","receivedAt":"2015-05-22T17:14:51Z","isPatch":false,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Johannes Schindelin\" \n> Hi Roberto,\n>\n> On 2015-05-22 11:42, Johannes Schindelin wrote:\n>\n>> On 2015-05-22 10:33, Roberto Tyley wrote:\n>>> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>>>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>>>> wrote:\n>>>> > How do you send your patches inline?\n>>> [snip]\n>>>> This workflow discussion was a topic at the GitMerge2015 \n>>>> conference,\n>>>> and there are essentially 2 groups, those who know how to send \n>>>> email\n>>>> and those who complain about it. A solution was agreed on by nearly \n>>>> all\n>>>> of the contributors. It would be awesome to have a git-to-email \n>>>> proxy,\n>>>> such that you could do a git push \n>>>> master:refs/for/mailinglist\n>>>> and this proxy would convert the push into sending patch series to \n>>>> the\n>>>> mailing list. It could even convert the following discussion back \n>>>> into\n>>>> comments (on Github?) but as a first step we'd want to try out a \n>>>> one\n>>>> way proxy.\n>>>>\n>>>> Unfortunately nobody stepped up to actually do the work, yet :(\n>>>\n>>>\n>>> Hello, I'm stepping up to do that work :) Or at least, I'm \n>>> implementing a\n>>> one-way GitHub PR -> Mailing list tool, called submitGit:\n>>>\n>>> https://submitgit.herokuapp.com/\n>>\n>> Wow!!!\n>>\n>> I will make sure to test it with a couple of patches I want to submit \n>> anyway.\n>\n> I just tried this with https://github.com/git/git/pull/139 and would \n> like to tell you about two wishes I had immediately:\n>\n> - If the author of a patch I am submitting is not myself, could \n> submitGit maybe add that `From: ` line at the top of the mail?\n> - The patch series is sent without a cover letter, but it would be \n> *really* great if a path series consisting of more than one patch \n> could have the initial comment of the Pull Request as a cover letter, \n> with the link to the original Pull Request at the bottom? This would \n> also be the mail to use in the \"In-reply-yo\" header instead of the \n> first patch.\n>\n> Thanks so much!\n> Dscho\n\n\nA separate request would be to be able to use PRs that are for forks of \ngit/git, such as msysgit etc. (i.e. have a common --root), which would \nhelp in the upstreaming of some changes.\n\n\nI ask because I just logged in and my preparatory PR318 \n(https://github.com/msysgit/git/pull/318) for rejuvenating the \nmsvc-build system wasn't listed, probably because of the forking.\n--\nPhilip \n"},{"id":"261933","messageId":"64326DCF30A041379141BB51437E87A6@PhilipOakley","threadId":"39407","inReplyTo":"xmqq8ucghf2s.fsf@gitster.dls.corp.google.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":"2015-05-22T17:40:36Z","receivedAt":"2015-05-22T17:40:36Z","isPatch":false,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Junio C Hamano\" \n> Roberto Tyley writes:\n>\n>> Hello, I'm stepping up to do that work :) Or at least, I'm \n>> implementing a\n>> one-way GitHub PR -> Mailing list tool, called submitGit:\n>>\n>> https://submitgit.herokuapp.com/\n>\n> Yay ;-)\n>\n>> Here's what a user does:\n>>\n>> * create a PR on https://github.com/git/git\n>> * logs into https://submitgit.herokuapp.com/ with GitHub auth\n>> * selects their PR on https://submitgit.herokuapp.com/git/git/pulls\n>\n> Reasonable.\n>\n>> * gets submitGit to email the PR as patches to themselves, in order \n>> to\n>> check it looks ok\n>\n> I can see you are trying to be careful by doing this, but I am not\n> sure if this step would actually help. Those who are not familiar\n> with Git development are not expected to know what is \"ok\" in their\n> original commit, and if they find bad formatting done by submitGit\n> (e.g. adds their PR message before the three-dash line instead of\n> after it), they cannot do much about it anyway.\n\nI still think this is valuable for the spotting of the dumb mistakes we \nall make, and notice after the fact when we see the email in the hard \nlight of day. There will still be poor commit messages and the like, but \nat least the raw content is more likely to be as the author desired.\n\n>\n>> * when they're ready, get submitGit to send it to the mailing list on\n>> their behalf\n>\n> Nice.\n>\n>> All discussion of the patch *stays* on the mailing list\n>\n> Can you identify a reroll of an earlier submission? If you can use\n> the in-reply-to and make it a follow-up to the previous round, that\n> would be great.\n> --\nPhilip \n"},{"id":"261939","messageId":"CAGZ79kaLqZCazM9MPR7wswMnVuvwO+Y0Pw9zPPXPEzAW6MQbEw@mail.gmail.com","threadId":"39407","inReplyTo":"64326DCF30A041379141BB51437E87A6@PhilipOakley","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2015-05-22T19:23:02Z","receivedAt":"2015-05-22T19:23:02Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"Ok, I am trying it out now, all I have left is\n\n Register your email address\n (stefanbeller@googlemail.com)\n with submitGit's Amazon SES\n account in order for it to send\n emails from you.\n\nSo first of all:\nWhere do I find the Amazon SES account for submitGit, to register\nmy email with?\n\nAlso can I change the email in the process or change it before?\n\nThanks,\nStefan\n"},{"id":"261945","messageId":"93994cca5e27388aee7e7c5fd4508816@www.dscho.org","threadId":"39407","inReplyTo":"0B3203305DF54249930E5903864B4988@PhilipOakley","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-22T19:58:19Z","receivedAt":"2015-05-22T19:58:19Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn 2015-05-22 19:14, Philip Oakley wrote:\n> From: \"Johannes Schindelin\" \n\n>> On 2015-05-22 11:42, Johannes Schindelin wrote:\n>>\n>>> On 2015-05-22 10:33, Roberto Tyley wrote:\n>>>> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>>>>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>>>>> wrote:\n>>>>> > How do you send your patches inline?\n>>>> [snip]\n>>>>> This workflow discussion was a topic at the GitMerge2015 conference,\n>>>>> and there are essentially 2 groups, those who know how to send email\n>>>>> and those who complain about it. A solution was agreed on by nearly all\n>>>>> of the contributors. It would be awesome to have a git-to-email proxy,\n>>>>> such that you could do a git push master:refs/for/mailinglist\n>>>>> and this proxy would convert the push into sending patch series to the\n>>>>> mailing list. It could even convert the following discussion back into\n>>>>> comments (on Github?) but as a first step we'd want to try out a one\n>>>>> way proxy.\n>>>>>\n>>>>> Unfortunately nobody stepped up to actually do the work, yet :(\n>>>>\n>>>>\n>>>> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a\n>>>> one-way GitHub PR -> Mailing list tool, called submitGit:\n>>>>\n>>>> https://submitgit.herokuapp.com/\n>>>\n>>> Wow!!!\n>>>\n>>> I will make sure to test it with a couple of patches I want to submit anyway.\n>>\n>> I just tried this with https://github.com/git/git/pull/139 and would like to tell you about two wishes I had immediately:\n>>\n>> - If the author of a patch I am submitting is not myself, could submitGit maybe add that `From: ` line at the top of the mail?\n>> - The patch series is sent without a cover letter, but it would be *really* great if a path series consisting of more than one patch could have the initial comment of the Pull Request as a cover letter, with the link to the original Pull Request at the bottom? This would also be the mail to use in the \"In-reply-yo\" header instead of the first patch.\n>>\n>> Thanks so much!\n>> Dscho\n> \n> \n> A separate request would be to be able to use PRs that are for forks\n> of git/git, such as msysgit etc. (i.e. have a common --root), which\n> would help in the upstreaming of some changes.\n> \n> \n> I ask because I just logged in and my preparatory PR318\n> (https://github.com/msysgit/git/pull/318) for rejuvenating the\n> msvc-build system wasn't listed, probably because of the forking.\n\nYou can easily change what upstream your PR is intended for. For example, I made my PR from my own Git fork (which is based on msysgit/git) relative to git/git by entering the URL:\n\nhttps://github.com/git/git/compare/next...dscho:non-win-fixes?expand=true\n\nSo there is no real need for anything extra: the only git.git project that requires emails (that I am aware of) for the submission process is git/git.\n\nCiao,\nDscho\n"},{"id":"261946","messageId":"12d36f8293e517ceb5eaaf67d23056a4@www.dscho.org","threadId":"39407","inReplyTo":"CAGZ79kaLqZCazM9MPR7wswMnVuvwO+Y0Pw9zPPXPEzAW6MQbEw@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-22T19:59:26Z","receivedAt":"2015-05-22T19:59:26Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Stefan,\n\nOn 2015-05-22 21:23, Stefan Beller wrote:\n> Ok, I am trying it out now, all I have left is\n> \n> Register your email address\n> (stefanbeller@googlemail.com)\n> with submitGit's Amazon SES\n> account in order for it to send\n> emails from you.\n> \n> So first of all:\n> Where do I find the Amazon SES account for submitGit, to register\n> my email with?\n> \n> Also can I change the email in the process or change it before?\n\nFWIW I did not have to register my email. All I needed to do was to give submitGit permissions to read my personal email address and my public repositories.\n\nCiao,\nDscho\n"},{"id":"261947","messageId":"CAPc5daX-Kw=tJFAZWnJTQ0t8_mY-_qnP1n7s7Yv6fbHC1rd_FQ@mail.gmail.com","threadId":"39407","inReplyTo":"12d36f8293e517ceb5eaaf67d23056a4@www.dscho.org","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-05-22T20:04:35Z","receivedAt":"2015-05-22T20:04:35Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"On Fri, May 22, 2015 at 12:59 PM, Johannes Schindelin\n wrote:\n>\n> On 2015-05-22 21:23, Stefan Beller wrote:\n>>\n>> So first of all:\n>> Where do I find the Amazon SES account for submitGit, to register\n>> my email with?\n>>\n>> Also can I change the email in the process or change it before?\n>\n> FWIW I did not have to register my email. All I needed to do was to give submitGit\n> permissions to read my personal email address and my public repositories.\n\nHmph, I was asked way more than that (especially, read and write access).\nDoes the site ask different authorizations depending on who you are?\n"},{"id":"261950","messageId":"CAGZ79kZq=O3igxrSXRE+uFgP5fdzd_31cYpV8PeVFGbD5ee7jA@mail.gmail.com","threadId":"39407","inReplyTo":"CAPc5daX-Kw=tJFAZWnJTQ0t8_mY-_qnP1n7s7Yv6fbHC1rd_FQ@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2015-05-22T20:44:37Z","receivedAt":"2015-05-22T20:44:37Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Fri, May 22, 2015 at 1:04 PM, Junio C Hamano wrote:\n> On Fri, May 22, 2015 at 12:59 PM, Johannes Schindelin\n> wrote:\n>>\n>> On 2015-05-22 21:23, Stefan Beller wrote:\n>>>\n>>> So first of all:\n>>> Where do I find the Amazon SES account for submitGit, to register\n>>> my email with?\n>>>\n>>> Also can I change the email in the process or change it before?\n>>\n>> FWIW I did not have to register my email. All I needed to do was to give submitGit\n>> permissions to read my personal email address and my public repositories.\n>\n> Hmph, I was asked way more than that (especially, read and write access).\n> Does the site ask different authorizations depending on who you are?\n\nI was also asked for read/write on my copy of git, but as I am not\nthe maintainer nor trusted in any way, I figured that's ok.\nI still have my local copy which would notice any changes on git push.\n\nThe question I was asking was the only thing I could not answer\nor decide for myself.\n"},{"id":"261953","messageId":"BACC0468FBD24DA584507676EC4E6A5A@PhilipOakley","threadId":"39407","inReplyTo":"93994cca5e27388aee7e7c5fd4508816@www.dscho.org","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":"2015-05-22T21:35:04Z","receivedAt":"2015-05-22T21:35:04Z","isPatch":false,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Johannes Schindelin\" \n> Hi,\n>\n> On 2015-05-22 19:14, Philip Oakley wrote:\n>> From: \"Johannes Schindelin\" \n>\n>>> On 2015-05-22 11:42, Johannes Schindelin wrote:\n>>>\n>>>> On 2015-05-22 10:33, Roberto Tyley wrote:\n>>>>> On Tuesday, 19 May 2015, Stefan Beller wrote:\n>>>>>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey\n>>>>>> wrote:\n>>>>>> > How do you send your patches inline?\n>>>>> [snip]\n>>>>>> This workflow discussion was a topic at the GitMerge2015 \n>>>>>> conference,\n>>>>>> and there are essentially 2 groups, those who know how to send \n>>>>>> email\n>>>>>> and those who complain about it. A solution was agreed on by \n>>>>>> nearly all\n>>>>>> of the contributors. It would be awesome to have a git-to-email \n>>>>>> proxy,\n>>>>>> such that you could do a git push \n>>>>>> master:refs/for/mailinglist\n>>>>>> and this proxy would convert the push into sending patch series \n>>>>>> to the\n>>>>>> mailing list. It could even convert the following discussion back \n>>>>>> into\n>>>>>> comments (on Github?) but as a first step we'd want to try out a \n>>>>>> one\n>>>>>> way proxy.\n>>>>>>\n>>>>>> Unfortunately nobody stepped up to actually do the work, yet :(\n>>>>>\n>>>>>\n>>>>> Hello, I'm stepping up to do that work :) Or at least, I'm \n>>>>> implementing a\n>>>>> one-way GitHub PR -> Mailing list tool, called submitGit:\n>>>>>\n>>>>> https://submitgit.herokuapp.com/\n>>>>\n>>>> Wow!!!\n>>>>\n>>>> I will make sure to test it with a couple of patches I want to \n>>>> submit anyway.\n>>>\n>>> I just tried this with https://github.com/git/git/pull/139 and would \n>>> like to tell you about two wishes I had immediately:\n>>>\n>>> - If the author of a patch I am submitting is not myself, could \n>>> submitGit maybe add that `From: ` line at the top of the mail?\n>>> - The patch series is sent without a cover letter, but it would be \n>>> *really* great if a path series consisting of more than one patch \n>>> could have the initial comment of the Pull Request as a cover \n>>> letter, with the link to the original Pull Request at the bottom? \n>>> This would also be the mail to use in the \"In-reply-yo\" header \n>>> instead of the first patch.\n>>>\n>>> Thanks so much!\n>>> Dscho\n>>\n>>\n>> A separate request would be to be able to use PRs that are for forks\n>> of git/git, such as msysgit etc. (i.e. have a common --root), which\n>> would help in the upstreaming of some changes.\n>>\n>>\n>> I ask because I just logged in and my preparatory PR318\n>> (https://github.com/msysgit/git/pull/318) for rejuvenating the\n>> msvc-build system wasn't listed, probably because of the forking.\n>\n> You can easily change what upstream your PR is intended for. For \n> example, I made my PR from my own Git fork (which is based on \n> msysgit/git) relative to git/git by entering the URL:\n>\n> https://github.com/git/git/compare/next...dscho:non-win-fixes?expand=true\n>\n> So there is no real need for anything extra: the only git.git project \n> that requires emails (that I am aware of) for the submission process \n> is git/git.\n>\n> Ciao,\n> Dscho\n>\nDo I read you right.. That it's necessary to create a PR on git/git \nbefore submitGit can be used.\n\nAnd that if I already have a PR which goes back to an alternate fork \n(e.g. my example), then I must move or duplicate that PR onto git/git \nbefore it can be used.\n\n[I find a few UI issues with the github interface in that I'd hoped to \nbe able to see which outgoing PRs I have from my own fork but I can't \nsee how to do that]\n\nregards\n\nPhilip \n"},{"id":"261984","messageId":"696c665281fa2abd35c519e0947fef98@www.dscho.org","threadId":"39407","inReplyTo":"BACC0468FBD24DA584507676EC4E6A5A@PhilipOakley","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-23T09:06:55Z","receivedAt":"2015-05-23T09:06:55Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Philip,\n\nOn 2015-05-22 23:35, Philip Oakley wrote:\n\n> Do I read you right.. That it's necessary to create a PR on git/git\n> before submitGit can be used.\n\nYep.\n\n> And that if I already have a PR which goes back to an alternate fork\n> (e.g. my example), then I must move or duplicate that PR onto git/git\n> before it can be used.\n\nYep, the rationale being that you most likely have to rebase your PR to git/git's `master` branch, anyway.\n\nCiao,\nDscho\n"},{"id":"261985","messageId":"3f6f651940e9a9b3a6c7bfa648d41e47@www.dscho.org","threadId":"39407","inReplyTo":"CAFY1edZ+5fW0FxQPO=qN6gqv8UfQUvjsutKvBOBj0CGbS8ms6A@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-23T09:09:32Z","receivedAt":"2015-05-23T09:09:32Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Roberto,\n\nOn 2015-05-22 23:28, Roberto Tyley wrote:\n\n> I'm currently on a cycling holiday on an island off the west coast of\n> Scotland, **without a laptop**, so updates to submitGit (based on the\n> excellent feedback I've been receiving) will probably start mid-next week.\n\nAh, I am jealous. Enjoy your trip!\n\nCiao,\nDscho\n\nP.S.: The only exposure I had to Scala was for desktop application programming, so I guess I won't try my hand at patching submitGit without your hand-holding. Not urgent, of course.\n"},{"id":"261986","messageId":"ad39373fe4fcff80d41bd610a82359d3@www.dscho.org","threadId":"39407","inReplyTo":"CAPc5daX-Kw=tJFAZWnJTQ0t8_mY-_qnP1n7s7Yv6fbHC1rd_FQ@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2015-05-23T09:11:07Z","receivedAt":"2015-05-23T09:11:07Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn 2015-05-22 22:04, Junio C Hamano wrote:\n> On Fri, May 22, 2015 at 12:59 PM, Johannes Schindelin\n> wrote:\n>>\n>> On 2015-05-22 21:23, Stefan Beller wrote:\n>>>\n>>> So first of all:\n>>> Where do I find the Amazon SES account for submitGit, to register\n>>> my email with?\n>>>\n>>> Also can I change the email in the process or change it before?\n>>\n>> FWIW I did not have to register my email. All I needed to do was to give submitGit\n>> permissions to read my personal email address and my public repositories.\n> \n> Hmph, I was asked way more than that (especially, read and write access).\n> Does the site ask different authorizations depending on who you are?\n\nWell, I did not try to send to the mailing list yet. Maybe that's the difference? Or maybe it is because I just registered with Heroku before...\n\nCiao,\nDscho\n"},{"id":"261991","messageId":"vpqa8wvgyf0.fsf@anie.imag.fr","threadId":"39407","inReplyTo":"CAGZ79kaLqZCazM9MPR7wswMnVuvwO+Y0Pw9zPPXPEzAW6MQbEw@mail.gmail.com","subject":"Re: [Announce] submitGit for patch submission (was \"Diffing submodule does not yield complete logs\")","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2015-05-23T14:29:23Z","receivedAt":"2015-05-23T14:29:23Z","isPatch":false,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Stefan Beller writes:\n\n> Ok, I am trying it out now, all I have left is\n>\n> Register your email address\n> (stefanbeller@googlemail.com)\n> with submitGit's Amazon SES\n> account in order for it to send\n> emails from you.\n>\n> So first of all:\n> Where do I find the Amazon SES account for submitGit, to register\n> my email with?\n>\n> Also can I change the email in the process or change it before?\n\nI was confused at the same point, but at the top of the screen showing\nthis message, there was a link to do this registration. I was just not\nlooking at the right place.\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"}]}