[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/
signature.asc
Description: PGP signature
- Re: Command line option in addition to GUIX_PACKAGE_PATH - Reason?,
Pierre Neidhardt <=