From: Johannes Schindelin <johannes.schindelin@gmx.de>
To: Philip Oakley <philipoakley@iee.org>
Cc: Roberto Tyley <roberto.tyley@gmail.com>,
Stefan Beller <sbeller@google.com>,
Robert Dailey <rcdailey.lists@gmail.com>,
Heiko Voigt <hvoigt@hvoigt.net>,
Jens Lehmann <Jens.Lehmann@web.de>, Git <git@vger.kernel.org>,
Thomas Ferris Nicolaisen <tfnico@gmail.com>,
emma@gitforteams.com
Subject: Re: [Announce] submitGit for patch submission (was "Diffing submodule does not yield complete logs")
Date: Fri, 22 May 2015 21:58:19 +0200 [thread overview]
Message-ID: <93994cca5e27388aee7e7c5fd4508816@www.dscho.org> (raw)
In-Reply-To: <0B3203305DF54249930E5903864B4988@PhilipOakley>
Hi,
On 2015-05-22 19:14, Philip Oakley wrote:
> From: "Johannes Schindelin" <johannes.schindelin@gmx.de>
>> 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
next prev parent reply other threads:[~2015-05-22 19:58 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-22 8:33 [Announce] submitGit for patch submission (was "Diffing submodule does not yield complete logs") Roberto Tyley
2015-05-22 9:42 ` Johannes Schindelin
2015-05-22 14:41 ` Johannes Schindelin
2015-05-22 17:14 ` Philip Oakley
2015-05-22 19:58 ` Johannes Schindelin [this message]
2015-05-22 21:35 ` Philip Oakley
2015-05-23 9:06 ` Johannes Schindelin
2015-05-22 13:06 ` Sebastian Schuberth
2015-05-22 13:25 ` Stefan Näwe
2015-05-22 14:17 ` Junio C Hamano
2015-05-22 17:40 ` Philip Oakley
2015-05-22 19:23 ` Stefan Beller
2015-05-22 19:59 ` Johannes Schindelin
2015-05-22 20:04 ` Junio C Hamano
2015-05-22 20:44 ` Stefan Beller
2015-05-23 9:11 ` Johannes Schindelin
2015-05-23 14:29 ` Matthieu Moy
[not found] ` <CAFY1edZ+5fW0FxQPO=qN6gqv8UfQUvjsutKvBOBj0CGbS8ms6A@mail.gmail.com>
2015-05-23 9:09 ` Johannes Schindelin
2015-05-22 16:14 ` Matthieu Moy
2015-05-22 16:27 ` Junio C Hamano
2015-05-22 16:53 ` Stefan Beller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=93994cca5e27388aee7e7c5fd4508816@www.dscho.org \
--to=johannes.schindelin@gmx.de \
--cc=Jens.Lehmann@web.de \
--cc=emma@gitforteams.com \
--cc=git@vger.kernel.org \
--cc=hvoigt@hvoigt.net \
--cc=philipoakley@iee.org \
--cc=rcdailey.lists@gmail.com \
--cc=roberto.tyley@gmail.com \
--cc=sbeller@google.com \
--cc=tfnico@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).