guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Command line option in addition to GUIX_PACKAGE_PATH - Reason?


From: Pierre Neidhardt
Subject: Re: Command line option in addition to GUIX_PACKAGE_PATH - Reason?
Date: Thu, 16 Jan 2020 15:30:23 +0100

I've merged your patch 38678.
Only one nit with guix refresh:

--8<---------------cut here---------------start------------->8---
+        ;; The short option -L is already used by --list-updaters, therefore
+        ;; it needs to be removed from %standard-build-options.
+        (let ((%load-path-option (find (lambda (option)
+                                         (member "load-path"
+                                                 (option-names option)))
+                                       %standard-build-options)))
+          (option
+           (filter (lambda (name) (not (equal? #\L name)))
+                   (option-names %load-path-option))
+           (option-required-arg? %load-path-option)
+           (option-optional-arg? %load-path-option)
+           (option-processor     %load-path-option)))
+
--8<---------------cut here---------------end--------------->8---

I would name the let-bound variable without a percent sign '%' by
convention.
Should I fix this for you?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]