Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Joel Brobecker <brobecker@adacore.com>
Cc: palves@redhat.com, gdb-patches@sourceware.org
Subject: Re: proposal for branch patches after first release
Date: Fri, 30 Nov 2012 09:15:00 -0000	[thread overview]
Message-ID: <83boefv4e3.fsf@gnu.org> (raw)
In-Reply-To: <20121129210440.GL3581@adacore.com>

> Date: Thu, 29 Nov 2012 22:04:40 +0100
> From: Joel Brobecker <brobecker@adacore.com>
> Cc: Eli Zaretskii <eliz@gnu.org>, gdb-patches@sourceware.org
> 
> > > An alternative would be to require some predictably formatted string
> > > in the log entry or the commit message, which you then could grep for.
> 
> > A one line at the head would be quite standard, given people have come
> > used to mail subject == git commit summary.  This might be the easiest,
> > though making this a "branch-only" requirement is very error prone.
> > 
> > I wouldn't mind at all enforcing better and more self-contained commit
> > log style, FWIW.
> 
> While I agree it's a good idea to move towards this principle,
> and for both head & branch, I think there is too much of a chance
> for oversights.

We post all patches here, most of them undergo a review before being
pushed, so we could keep an eye on that.  CVS allows one to change the
commit message, so we could do that retrospectively if someone forgets
(unless the git conversion will become confused by such changes).

If we keep these records on  ChangeLog, the fix is even simpler: just
make another commit with a proper description.

> I also think that having the information spread out, be it in
> the revision logs, or in a bugs database, makes it hard for anyone
> who wants to figure out what changes exactly a minor release
> contains and for what purpose.

This is an argument in favor of ChangeLog.

> For all these reasons, I think that having the developer write up
> the description himself, with approval from Eli, and put it somewhere
> in a file, would be best. If we want to call it known-problems or
> anything else, instead of putting that info in the NEWS files,
> that would be fine as well.

This, too, would be fine with me.


  reply	other threads:[~2012-11-30  9:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-29 14:14 Joel Brobecker
2012-11-29 16:14 ` Eli Zaretskii
2012-11-29 16:32   ` Pedro Alves
2012-11-29 21:04     ` Joel Brobecker
2012-11-30  9:15       ` Eli Zaretskii [this message]
2012-11-30 10:55       ` Pedro Alves
2012-12-07 21:16     ` Tom Tromey
2012-12-03  6:51 ` Doug Evans
2012-12-03  9:16   ` Joel Brobecker

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=83boefv4e3.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=palves@redhat.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