GNU bug report logs - #27898
Clean-up of package phases

Previous Next

Package: guix-patches;

Reported by: Kei Kebreau <kei <at> openmailbox.org>

Date: Tue, 1 Aug 2017 07:20:02 UTC

Severity: normal

Done: Ricardo Wurmus <rekado <at> elephly.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Leo Famulari <leo <at> famulari.name>
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: Kei Kebreau <kei <at> openmailbox.org>, 27898 <at> debbugs.gnu.org
Subject: [bug#27898] [PATCH 1/2] gnu: Use 'modify-phases' syntax.
Date: Wed, 2 Aug 2017 18:46:14 -0400
[Message part 1 (text/plain, inline)]
On Wed, Aug 02, 2017 at 10:08:18PM +0200, Marius Bakke wrote:
> Leo Famulari <leo <at> famulari.name> writes:
> > I agree, wow! Thanks for this. It should help new Schemers to feel more
> > comfortable editing packages.
> >
> > This might be annoying but it should wait until after the core-updates
> > branch is done and merged into the master branch. I want to minimize the
> > number of merge conflicts because they are 1) annoying and 2) relatively
> > opaque when reading the Git history. Marius's suggestion is another good
> > one, and would help with issues like that.
> >
> > But if we were to wait until after core-updates and push it as one
> > commit, I wouldn't mind. It's up to the two of you and everyone else :)
> 
> Yeah I guess the squashed patch is okay. If we delay the patches until
> after 'staging' and 'python-updates' as well, no merge will be
> necessary. But it won't be a pretty cherry-pick either, by then.

True, we should not wait *too* long. I think that doing it after
core-updates and before staging and python-updates could be good
compromise, because those branches touch a relatively small number of
modules. WDYT?

> I don't really have a strong opinion either way. But I'd hate to the one
> resolving potentially 100s of merge conflicts at once ;-) Cherry-picking
> at least restricts the set of conflicts to these changes.

OTOH, if Kei doesn't have an unsquashed version of this patch, it will
be annoying to split it up :)
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 7 years and 277 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.