Hi Maxime, On Thu, 3 Mar 2022 at 07:25, Maxime Devos 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: 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