unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Alan Mackenzie <acm@muc.de>
Cc: 53164@debbugs.gnu.org
Subject: bug#53164: (After an ELC+ELN build, don't load the source file into a buffer.) fixed.
Date: Tue, 11 Jan 2022 14:27:14 +0200	[thread overview]
Message-ID: <83bl0i8nkt.fsf@gnu.org> (raw)
In-Reply-To: <YdyVVLMqjWX4gItw@ACM> (message from Alan Mackenzie on Mon, 10 Jan 2022 20:21:40 +0000)

> Date: Mon, 10 Jan 2022 20:21:40 +0000
> Cc: 53164@debbugs.gnu.org, acm@muc.de
> From: Alan Mackenzie <acm@muc.de>
> 
> > After reading this, I still don't understand how come you bump into
> > this problem, whereas I don't, and neither does anyone else who builds
> > the release branch with native-compilation.  Is this something
> > specific to that branch you are working on?
> 
> Indeed, yes.  It is the symbols with position which escaped from a
> compiler macro.
> 
> > If so, why is it urgent to have the fix on the release branch?  The
> > branch on which you ware working will land on master.
> 
> There was no urgency.  I though the convention was for documentation
> fixes and simple safe code fixes to go onto the release branch.  The fix
> to this bug, a single line, is well understood and certainly comes into
> the category of simple and safe.

The fix is well understood, but its possible effects aren't.  We have
been using the current code for more than a year with no problems
whatsoever.

> > There's always one more bug.  When we are so far into the pretest
> > process, problems that take unusual steps to reproduce are not
> > important enough to delay the release.
> 
> OK.  But can I ask you to consider announcing on emacs-devel when the
> criteria for making bug fixes on the release branch change?  Apologies if
> you have done, and I missed it.

This is in CONTRIBUTE:

  If you are fixing a bug that exists in the current release, you should
  generally commit it to the release branch; it will be merged to the
  master branch later by the gitmerge function.  However, when the
  release branch is for Emacs version NN.2 and later, or when it is for
  Emacs version NN.1 that is in the very last stages of its pretest,
  that branch is considered to be in a feature freeze: only bug fixes
  that are "safe" or are fixing major problems should go to the release
  branch, the rest should be committed to the master branch.  This is so
  to avoid destabilizing the next Emacs release.  If you are unsure
  whether your bug fix is "safe" enough for the release branch, ask on
  the emacs-devel mailing list.

> There're three ways we could go.  Commit it in emacs-28, master, or
> scratch/correct-warning-pos.  I'm willing to commit it again in any of
> these places.

Please install this on master (or leave it on your branch), and we
will revisit this when time comes for Emacs 28.2.

Thanks.





  reply	other threads:[~2022-01-11 12:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-10 16:50 bug#53164: After an ELC+ELN build, don't load the source file into a buffer Alan Mackenzie
2022-01-10 17:15 ` bug#53164: (After an ELC+ELN build, don't load the source file into a buffer.) fixed Alan Mackenzie
2022-01-10 18:05   ` Eli Zaretskii
2022-01-10 19:34     ` Alan Mackenzie
2022-01-10 19:53       ` Eli Zaretskii
2022-01-10 20:21         ` Alan Mackenzie
2022-01-11 12:27           ` Eli Zaretskii [this message]
2022-01-11 18:03             ` Alan Mackenzie
2022-01-11 18:45               ` Eli Zaretskii
2022-01-10 18:01 ` bug#53164: After an ELC+ELN build, don't load the source file into a buffer Eli Zaretskii

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=83bl0i8nkt.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=53164@debbugs.gnu.org \
    --cc=acm@muc.de \
    /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).