unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: emacs-devel@gnu.org
Subject: Re: CONTRIBUTE and where to commit bug fixes
Date: Mon, 24 Sep 2018 09:14:03 -0400	[thread overview]
Message-ID: <jwv8t3r6o6z.fsf-monnier+gmane.emacs.devel@gnu.org> (raw)
In-Reply-To: 87pnxqyvk8.fsf@web.de

> but that's obviously not always true: depending on the release cycle,
> only fixes for bugs that where introduced in the last release are
> accepted, or even less.

Right.  Looks like you've followed this list long enough to know how it
works ;-)

> And if a fix is not ok for the release branch in a moment, should it
> go to master, or should it just wait until later?

Install it on master.  If the patch should apply to the next bug-fix
release, but can't be installed because we're in "deep freeze", we
should (but don't AFAIK) have some other branch for that patch, indeed.

> How can I find out which fixes are acceptable for the release branch at
> any moment without asking every time?

Follow the list with enough dedication to know when we're in deep freeze
and when we're not.

Maybe rather than document the way we work in more detail (which won't
really help solve the problem, obviously), we could change the way we
work to make it more clear:

- when we start the "new major release" process, we create a new branch
  (e.g. emacs-27).  Nothing new here.
- when we enter the "deep freeze" process, rather than announce it and
  hope people will know about it, we create yet another branch
  (e.g. emacs-27.1).

Then the rule can be documented more simply in the doc:
- master is for new features.
- emacs-NN is for any bug and doc fix.
- emacs-NN.MM only takes bug fixes for regressions and requires explicit
  agreement from the maintainers.

This also solves the problem of where to install non-regression bug
fixes when we're in deep freeze.


        Stefan




  parent reply	other threads:[~2018-09-24 13:14 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-07  0:56 CONTRIBUTE and where to commit bug fixes Michael Heerdegen
2018-09-24 12:53 ` Michael Heerdegen
2018-09-24 13:14 ` Stefan Monnier [this message]
2018-09-24 23:06   ` Karl Fogel
2018-09-25  2:49   ` Van L
2018-10-04 15:57   ` Michael Heerdegen
2018-10-04 16:35     ` John Wiegley
2018-10-04 16:38     ` Noam Postavsky
2018-10-04 20:43       ` Michael Heerdegen

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: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwv8t3r6o6z.fsf-monnier+gmane.emacs.devel@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    /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://git.savannah.gnu.org/cgit/emacs.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).