Hi Ludo, ter 26 mai 2020 às 11:56:21 (1590504981), nandre@riseup.net enviou: > dom 24 mai 2020 às 22:51:16 (1590371476), ludo@gnu.org enviou: > > It looks good to me overall! Some nitpicking: > > > > We’d rather use ‘substitute-keyword-arguments’ to augment > > #:configure-flags without touching the other keyword arguments (there > > are several examples in the source). > > .... > > > > We generally avoid concatenating text like this, for the reasons > > explained at: > > > > https://guix.gnu.org/manual/en/html_node/Synopses-and-Descriptions.html > > .... > > > > Regarding the format of patches, you can take a look at this: > > > > https://guix.gnu.org/manual/en/html_node/Submitting-Patches.html > > I'll send a new patch taking your warnings into account. I'm a little bit short on time, so I couldn't do everything I was supposed to. I'm sending the attached patch for your consideration, just in case someone wants to try it out, though I still need to clone the git repo, try to build it on a pre-inst-env, and try to compile on archs other than i686 and x86_64. I've run './etc/indent-code.el', guix lint and there where no errors. It might take me some time to properly set up everything here and complete the remaining steps, so please do tell me if you think the reasoning on the previous email does not hold up or is not worth the trouble. This diff was taken upon commit 018cffc9c9e5a5855733f5f45a1c4d396bb6a321.