From: Jason Holden <jason.k.holden.swdev@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 2/3] SubmittingPatches: mention subsystems with dedicated repositories
Date: Tue, 1 Jan 2013 20:52:34 -0500 [thread overview]
Message-ID: <20130102015233.GA25288@gmail.com> (raw)
In-Reply-To: <1357082695-29713-3-git-send-email-gitster@pobox.com>
On Tue, Jan 01, 2013 at 03:24:54PM -0800, Junio C Hamano wrote:
>
> ------------------------------------------------
> +Subsystems with dedicated maintainers
> +
> +Some parts of the system have dedicated maintainers with their own
> +repositories.
> +
> + - git-gui/ comes from git-gui project, maintained by Pat Thoyts:
> +
> + git://repo.or.cz/git-gui.git
> +
> + - gitk-git/ comes from Paul Mackerras's gitk project:
> +
> + git://ozlabs.org/~paulus/gitk
> +
> + - po/ comes from the localization coordinator, Jiang Xin:
> +
> + https://github.com/git-l10n/git-po/
> +
> +Patches to these parts should be based on their trees.
> +
> +------------------------------------------------
> An ideal patch flow
>
Any reason to leave out the maintainers email addresses? If we add that, all
the content of the MAINTAINERS file we had been discussing would then be in
this file.
My only other suggestion for this series might be to augment the file with
a patch submittal example(s). I found the best example to be in
git-send-email's man page, but maybe enumerate the example out for the
most common workflows. Maybe something like:
-----------------------------------------------------------
Example of sending patches for a new feature:
Create the patches:
$ git format-patch --cover-letter -M origin/master -o outgoing/
$ edit outgoing/0000-*
To send your completed patches for initial consideration:
$ git send-email outgoing/* -to git@vger.kernel.org -cc gitster@pobox.com
To send your reviewed patches for inclusion:
$ git send-email outgoing/* -to gitster@pobox.com -cc git@vger.kernel.org
-Jason
next prev parent reply other threads:[~2013-01-02 2:05 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-01 23:24 [PATCH 0/3] Update SubmittingPatches Junio C Hamano
2013-01-01 23:24 ` [PATCH 1/3] SubmittingPatches: who am I and who cares? Junio C Hamano
2013-01-01 23:24 ` [PATCH 2/3] SubmittingPatches: mention subsystems with dedicated repositories Junio C Hamano
2013-01-02 1:52 ` Jason Holden [this message]
2013-01-02 2:14 ` Junio C Hamano
2013-01-02 17:22 ` Junio C Hamano
2013-01-04 20:58 ` [PATCH] SubmittingPatches: Document how to request a patch review tag Jason Holden
2013-01-04 21:47 ` Junio C Hamano
2013-01-06 7:32 ` Michael Haggerty
2013-01-06 9:01 ` Junio C Hamano
2013-01-01 23:24 ` [PATCH 3/3] SubmittingPatches: remove overlong checklist Junio C Hamano
2013-01-02 9:08 ` [PATCH 0/3] Update SubmittingPatches Jeff King
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130102015233.GA25288@gmail.com \
--to=jason.k.holden.swdev@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.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.
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 NNTP newsgroup(s).