[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Gu
From: |
zimoun |
Subject: |
[bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix. |
Date: |
Thu, 3 Mar 2022 09:48:21 +0100 |
Hi Maxime,
On Thu, 3 Mar 2022 at 07:25, Maxime Devos <maximedevos@telenet.be> wrote:
> Leo Famulari schreef op wo 02-03-2022 om 18:50 [-0500]:
> > Origin snippets should only be used to remove nonfree things
> > from the upstream source code. All other changes should use
> > patch files or a build phase.
>
> Why? If it's a source code change and it fits an origin snippet,
> why not an origin snippet? Why would the source in Guix need to match
> the source upstream?
Because as I tried to explain here [1] ;-)
1: <https://issues.guix.gnu.org/54216#5>
a) the location of patches depending of their nature is **not well-documented**,
b) because it is a tension between two intentions,
c) and thus we never take the time to clearly document for consistency.
FWIW, it would be unfair for the patch to have the discussion here.
As Leo, I also think this source modification should go to a phase.
> Also, in this case, it was just copied from the 'shepherd' packages
> with some slight adjustments.
And I also think this recent commit
79be6a985799adc6d663890250f4fb7c12f015b4 introducing the 'snippet' is
unfortunate. And it should be fixed, i.e., move the substitution to a
phase.
Cheers,
simon
- [bug#54216] [PATCH] gnu: shepherd-for-guix: New package for use in Guix., Attila Lendvai, 2022/03/01
- [bug#54216] [PATCH] gnu: shepherd-for-guix: New package for use in Guix., Maxime Devos, 2022/03/01
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Attila Lendvai, 2022/03/01
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Maxime Devos, 2022/03/01
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Attila Lendvai, 2022/03/01
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., zimoun, 2022/03/02
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Leo Famulari, 2022/03/02
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Maxime Devos, 2022/03/03
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix.,
zimoun <=
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Leo Famulari, 2022/03/05
- [bug#54216] [PATCH v2] gnu: shepherd-for-guix: New package for use in Guix., Maxime Devos, 2022/03/05
- [bug#54216] [PATCH staging] gnu: shepherd: Build it from git, and clean up shepherd-for-guix., Maxime Devos, 2022/03/03
- [bug#54216] [PATCH staging] gnu: shepherd: Build it from git, and clean up shepherd-for-guix., zimoun, 2022/03/03
[bug#54216] [PATCH v3] gnu: shepherd-for-guix: New package for use in Guix., Attila Lendvai, 2022/03/03
[bug#54216] [PATCH staging] gnu: shepherd: Build it from git, and clean up shepherd-for-guix., Attila Lendvai, 2022/03/03
[bug#54216] [PATCH shepherd staging v2] gnu: shepherd: Build it from git, and clean up shepherd-for-guix., Attila Lendvai, 2022/03/04
[bug#54216] [PATCH v4] gnu: shepherd-for-guix: New package for use in Guix., Attila Lendvai, 2022/03/21