git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* appropriate mailing list for gitk patches
@ 2014-11-13 21:10 lennart spitzner
  2014-11-13 21:32 ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: lennart spitzner @ 2014-11-13 21:10 UTC (permalink / raw)
  To: git

Hello folks,

I have a patch for gitk. Should i send it to this list, or is there some
other?  git/Documentation/SubmittingPatches mentions
git://ozlabs.org/~paulus/gitk (the master of which i based my commit
on), but never mentions any list(s) by name (only "the Git mailing
list", which i presume is this one).

Also, a minor question regarding patches: I wondered why patches do not
mention the commit id that the patch(es) are based on. My current guess
is that the diffs containing the sha1's is considered sufficient. Is
this the case? Are there more/other aspects?

Lennart

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: appropriate mailing list for gitk patches
  2014-11-13 21:10 appropriate mailing list for gitk patches lennart spitzner
@ 2014-11-13 21:32 ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2014-11-13 21:32 UTC (permalink / raw)
  To: lennart spitzner; +Cc: git

lennart spitzner <lsp@informatik.uni-kiel.de> writes:

> I have a patch for gitk. Should i send it to this list, or is there some
> other?  git/Documentation/SubmittingPatches mentions
> git://ozlabs.org/~paulus/gitk (the master of which i based my commit
> on), but never mentions any list(s) by name (only "the Git mailing
> list", which i presume is this one).

You are in the right place ;-)  Make sure you Cc: paulus as the area
maintainer on your patches, though.

> Also, a minor question regarding patches: I wondered why patches do not
> mention the commit id that the patch(es) are based on. My current guess
> is that the diffs containing the sha1's is considered sufficient. Is
> this the case? Are there more/other aspects?

A new development is done based on the tip of 'master' by convention
around here, so people know it is meant to apply there unless the
patch submitter says otherwise explicitly.

People send patches on a random commit of their own that is not
available to the public (e.g. you may be working on some big topic,
discover a simple bug that is not related to the topic, and create a
fix for that simple bug right there on top and send that out as a
patch).  Base commit object name would be useless in such a case
anyway.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-11-13 21:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-13 21:10 appropriate mailing list for gitk patches lennart spitzner
2014-11-13 21:32 ` Junio C Hamano

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).