git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [Announce] submitGit for patch submission (was "Diffing submodule does not yield complete logs")

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
May 22, 2015, 19:58 UTC
Message-ID
<93994cca5e27388aee7e7c5fd4508816@www.dscho.org>
In-Reply-To
<0B3203305DF54249930E5903864B4988@PhilipOakley>
Hi,
On 2015-05-22 19:14, Philip Oakley wrote:
> From: "Johannes Schindelin" <johannes.schindelin@gmx.de>
Show 47 quoted lines
>> On 2015-05-22 11:42, Johannes Schindelin wrote:
>>
>>> On 2015-05-22 10:33, Roberto Tyley wrote:
>>>> On Tuesday, 19 May 2015, Stefan Beller <sbeller@google.com> wrote:
>>>>> On Tue, May 19, 2015 at 12:29 PM, Robert Dailey
>>>>> <rcdailey.lists@gmail.com> wrote:
>>>>> > How do you send your patches inline?
>>>> [snip]
>>>>> This workflow discussion was a topic at the GitMerge2015 conference,
>>>>> and there are essentially 2 groups, those who know how to send email
>>>>> and those who complain about it. A solution was agreed on by nearly all
>>>>> of the contributors. It would be awesome to have a git-to-email proxy,
>>>>> such that you could do a git push <proxy> master:refs/for/mailinglist
>>>>> and this proxy would convert the push into sending patch series to the
>>>>> mailing list. It could even convert the following discussion back into
>>>>> comments (on Github?) but as a first step we'd want to try out a one
>>>>> way proxy.
>>>>>
>>>>> Unfortunately nobody stepped up to actually do the work, yet :(
>>>>
>>>>
>>>> Hello, I'm stepping up to do that work :) Or at least, I'm implementing a
>>>> one-way GitHub PR -> Mailing list tool, called submitGit:
>>>>
>>>> https://submitgit.herokuapp.com/
>>>
>>> Wow!!!
>>>
>>> I will make sure to test it with a couple of patches I want to submit anyway.
>>
>> I just tried this with https://github.com/git/git/pull/139 and would like to tell you about two wishes I had immediately:
>>
>> - 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?
>> - 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.
>>
>> Thanks so much!
>> Dscho
> 
> 
> A separate request would be to be able to use PRs that are for forks
> of git/git, such as msysgit etc. (i.e. have a common --root), which
> would help in the upstreaming of some changes.
> 
> 
> I ask because I just logged in and my preparatory PR318
> (https://github.com/msysgit/git/pull/318) for rejuvenating the
> msvc-build system wasn't listed, probably because of the forking.
You 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:
https://github.com/git/git/compare/next...dscho:non-win-fixes?expand=true
So 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.

Ciao, Dscho

Previous: Philip OakleyNext: Philip Oakley
Message 5 of 21 in “[Announce] submitGit for patch submission (was "Diffing submodule does not yield complete logs")”
  1. Roberto TyleyMay 22, 2015
  2. Johannes SchindelinMay 22, 2015
  3. Johannes SchindelinMay 22, 2015
  4. Philip OakleyMay 22, 2015
  5. Johannes SchindelinMay 22, 2015
  6. Philip OakleyMay 22, 2015
  7. Johannes SchindelinMay 23, 2015
  8. Sebastian SchuberthMay 22, 2015
  9. Stefan NäweMay 22, 2015
  10. Junio C HamanoMay 22, 2015
  11. Philip OakleyMay 22, 2015
  12. Stefan BellerMay 22, 2015
  13. Johannes SchindelinMay 22, 2015
  14. Junio C HamanoMay 22, 2015
  15. Stefan BellerMay 22, 2015
  16. Johannes SchindelinMay 23, 2015
  17. Matthieu MoyMay 23, 2015
  18. Matthieu MoyMay 22, 2015
  19. Junio C HamanoMay 22, 2015
  20. Stefan BellerMay 22, 2015
  21. Johannes SchindelinMay 23, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.