guix-patches
[Top][All Lists]
Advanced

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

[bug#57704] [PATCH core-updates] guix: packages: Remove #f from inputs w


From: Maxime Devos
Subject: [bug#57704] [PATCH core-updates] guix: packages: Remove #f from inputs when sanitizing.
Date: Fri, 9 Sep 2022 20:54:09 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0

On 09-09-2022 17:56, Liliana Marie Prikler wrote:
This makes it so that new-style inputs can be optional using regular Guile
patterns, e.g. (and (target-x86-64?) rust).

Seems useful.

* guix/packages.scm (sanitize-inputs): Filter inputs by identity before adding
labels.

Documentation is missing.

---
Note that this patch was prepared using master, but since it affects the
package record, it needs to go to core-updates.  I don' think there should
be a merge conflict here.

It does affect the package record, but it doesn't cause any rebuilds, so master should be fine:

* There aren't any current uses of #false:

(use-modules (guix packages) (gnu packages))
(package
  (inherit (specification->package "hello"))
  (inputs (list #false)))
;; guix build -f [...] --> package ‘hello@2.12.1’ has an invalid input

* In the absence of #false, the behaviour remains unchanged.

* guix/packages.scm is not used by any derivation
  (except for "guix pull" and the guix package)

As a test, I applied the patch and did
‘make && ./pre-inst-env guix build -n libreoffice’,
and it turned out I already have it installed.

Greetings,
Maxime.

Attachment: OpenPGP_0x49E3EE22191725EE.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


reply via email to

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