GNU bug report logs - #62564
patches to fix age after update to v1.1.1

Previous Next

Package: guix-patches;

Reported by: Nicolas Graves <ngraves <at> ngraves.fr>

Date: Fri, 31 Mar 2023 07:34:01 UTC

Severity: normal

Done: Leo Famulari <leo <at> famulari.name>

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: 62564 <at> debbugs.gnu.org
Subject: [bug#62564] patches to fix age after update to v1.1.1
Date: Sat, 1 Apr 2023 12:40:19 -0400
On Sat, Apr 01, 2023 at 10:00:05AM +0200, Nicolas Graves via Guix-patches via wrote:
> So, in order for other concerned packages to build properly, I had to
> update a few other packages. The v3 patches seem to build fine on
> https://qa.guix.gnu.org/issue/62564, I guess it should be alright to get
> them merged.

Thanks for this patch series!

Overall, it looks good, but there are still some cosmetic issues to fix.

For each commit, please make the commit message fully describe the
changes. For example, in patch 4/14, the commit message does not mention
the changed dependencies in native-inputs. Take a look at the commit log
for examples of what to write, e.g.:

https://git.savannah.gnu.org/cgit/guix.git/commit/?id=113146d31c91e1951f469aa974fef9c0413080a8

In several of the patches that add a new package, the new package
variable is added without any space before the subsequent package
variable. Please ensure that all new packages have a newline before and
after the new variable.

Beyond these cosmetic issues, the changes look good to me.




This bug report was last modified 2 years and 87 days ago.

Previous Next


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