From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: Juanma Barranquero <lekktu@gmail.com>, 9031@debbugs.gnu.org
Subject: bug#9031: Installed rest of patch
Date: Tue, 12 Jul 2011 00:51:04 -0400 [thread overview]
Message-ID: <jwvpqlg9kjo.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <4E1B3224.5050804@cs.ucla.edu> (Paul Eggert's message of "Mon, 11 Jul 2011 10:25:56 -0700")
>> it's better to keep code cleanups for later
> This particular fix is both a code cleanup and a bug fix.
Yes, it was OK. I just felt that Juanma's answer was a bit too much
on the "we're not yet in pretest" side, so I wanted to compensate by
reminding people that we should think of fixing bugs now, not cleaning
things up.
>> you could try to install them in the `pending' branch, but such code
>> cleanups are likely to generate merge conflicts
> Does this advice mean that code cleanups should not be installed
> anywhere now, not even in the 'pending' branch?
You can install them there, but that branch is really meant to be
a repository of patches that will get merged to trunk in some distant
future. Code that touches a lot of code is likely to encounter merge
conflicts when we get to that distant future. And changes which need to
apply to "all places where foo happens" are even more likely to be
out-of-date when we get to that distant future.
> I ask because I have some integer-overflow-and-signedness-related
> cleanups and bug-fixes in my private copy that should really get
> incorporated at some point.
Put them somewhere. But I suspect that it'll be easier to re-do them
than to merge the current patch several months from now, so it's
probably important to store somewhere the "how did I come up with this
patch" along with the patch.
> (Sorry, I don't know the current practice with regards to the
> 'pending' branch; maybe 'pending' should be documented somewhere
> under admin/?)
It's "documented" in http://bzr.savannah.gnu.org/r/emacs/README.
We don't really want to encourage people to use it too much, since the
focus should instead move to fixing bugs. But sometimes it's easier to
install the patch somewhere than to try to mark some bug report as "it's
got a patch and the patch is good to go, we just have to wait for the
trunk to open up again before we can install it".
Stefan
next prev parent reply other threads:[~2011-07-12 4:51 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-09 6:32 bug#9031: Unused vars etc. in chartab.c, composite.c, gtkutil.c Paul Eggert
2011-07-10 5:21 ` bug#9031: Installed rest of patch Paul Eggert
2011-07-10 11:37 ` Juanma Barranquero
2011-07-11 12:49 ` Stefan Monnier
2011-07-11 17:25 ` Paul Eggert
2011-07-12 4:51 ` Stefan Monnier [this message]
2011-07-12 5:36 ` Paul Eggert
2011-07-12 11:07 ` Juanma Barranquero
2011-07-12 15:44 ` Chong Yidong
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=jwvpqlg9kjo.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=9031@debbugs.gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=lekktu@gmail.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.
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).