public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jakub Jelinek <jakub@redhat.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: Joseph Myers <joseph@codesourcery.com>,
	       "Richard Earnshaw (lists)" <Richard.Earnshaw@arm.com>,
	gcc@gcc.gnu.org
Subject: Re: gcc-cvs mails for personal/vendor branches for merge commits
Date: Thu, 16 Jan 2020 14:35:00 -0000	[thread overview]
Message-ID: <20200116141147.GC10088@tucnak> (raw)
In-Reply-To: <20200116114002.GB13162@adacore.com>

On Thu, Jan 16, 2020 at 12:40:02PM +0100, Joel Brobecker wrote:
> > I think there's a similar issue not just for merges but for 
> > non-fast-forward pushes as well.  As a glibc example, consider 
> > <https://sourceware.org/ml/glibc-cvs/2019-q4/msg00666.html> and the long 
> > series of subsequent mails in the glibc-cvs archives.  It's correct that 
> > the five or so rebased patches on the branch got mailed out again.  It's 
> > *not* desirable that the 50 or so patches that were already on master, 
> > that the branch got rebased on top of, got mailed out again.
> 
> At heart, it is really the same issue. New commits were brought into
> this branch, and thus if email notification is enabled for that branch,
> then those commits should trigger emails for their own as well.
> 
> Typically, branches were non-fast-forward changes are allowed are
> branches that are personal and not shared. In those instances,
> the typical setup is to disable emails on those development branches.
> It sounds to me like turning emails off for branches that can
> do non-fast-forward is the better solution here.

Couldn't it be then per-branch setting, whether to mail even commits
that aren't new to the repository or not (like I understood it is already
possible to decide per-branch whether to send mails at all)?
E.g. for GCC, I'd certainly like to see mails for all commits on the
trunk and release branches (but we don't allow merge commits on them
anyway), and for other branches (personal, vendor, devel) I think mailing
only about new commits not already in the repository would be better over
not mailing at all.

	Jakub

  reply	other threads:[~2020-01-16 14:12 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-15 15:07 Jakub Jelinek
2020-01-15 15:42 ` Joseph Myers
2020-01-15 15:51   ` Jakub Jelinek
2020-01-15 16:01     ` Joseph Myers
2020-01-15 16:42   ` Jason Merrill
2020-01-15 16:58     ` Joseph Myers
2020-01-15 17:09       ` Jason Merrill
2020-01-15 17:57       ` Iain Sandoe
2020-01-15 20:20         ` Jason Merrill
2020-01-19 14:10         ` Segher Boessenkool
2020-01-19 22:40           ` Julien "_FrnchFrgg_" Rivaud
2020-01-20 18:42             ` Segher Boessenkool
2020-01-15 19:21       ` Richard Earnshaw (lists)
2020-01-16  9:55         ` Joseph Myers
2020-01-16 14:28   ` Joel Brobecker
2020-01-16 14:35     ` Jakub Jelinek [this message]
2020-01-16 14:43       ` Richard Earnshaw (lists)
2020-01-16 18:51         ` Jakub Jelinek
2020-01-16 22:03       ` Joseph Myers
2020-01-16 18:58     ` Joseph Myers
2020-01-17 16:47       ` Joel Brobecker
2020-01-17 17:22         ` Iain Sandoe
2020-01-17 17:59           ` Joel Brobecker
2020-01-17 19:15             ` Jason Merrill
2020-01-17 22:39               ` Joel Brobecker
2020-01-18  9:50                 ` Iain Sandoe
2020-01-18 11:07                   ` Jason Merrill
2020-01-18 14:30                 ` Joseph Myers
2020-01-17 17:23         ` Joseph Myers
2020-01-17 21:14           ` 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=20200116141147.GC10088@tucnak \
    --to=jakub@redhat.com \
    --cc=Richard.Earnshaw@arm.com \
    --cc=brobecker@adacore.com \
    --cc=gcc@gcc.gnu.org \
    --cc=joseph@codesourcery.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 read-only IMAP folder(s) and NNTP newsgroup(s).