GNU bug report logs - #41219
Enforce "files" directive in node build system

Previous Next

Package: guix-patches;

Reported by: goodoldpaul <at> autistici.org

Date: Tue, 12 May 2020 21:27:01 UTC

Severity: normal

Done: Jelle Licht <jlicht <at> fsfe.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Jelle Licht <jlicht <at> fsfe.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#41219: closed (Enforce "files" directive in node build system)
Date: Wed, 09 Dec 2020 21:46:01 +0000
[Message part 1 (text/plain, inline)]
Your message dated Wed, 09 Dec 2020 22:45:40 +0100
with message-id <86eejy64cb.fsf <at> fsfe.org>
and subject line Re: [bug#41219] [PATCH 2/2] guix: Enforce package.json "files" directive.
has caused the debbugs.gnu.org bug report #41219,
regarding Enforce "files" directive in node build system
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
41219: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=41219
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: goodoldpaul <at> autistici.org
To: guix-patches <at> gnu.org
Subject: Enforce "files" directive in node build system
Date: Tue, 12 May 2020 21:26:22 +0000
Hi,

I'm sending a couple of patches to fix 
https://issues.guix.gnu.org/40710, I tried to base my implementation on 
[0].

The first patch adds "globstar" support to (guix glob), namely the 
ability of recursively matching subdirectories in a glob pattern (i.e. 
"foo/**/bar.scm" matches both "foo/bar.scm" and "foo/baz/bar.scm").

The second patch adds (guix glob) to the imported modules of 
node-build-system and uses that to parse glob patterns in the "files" 
array of a package.json and then install all the matching files.

I tested the patches by verifying that

./pre-inst-env guix build -K node-semver node-util-deprecate 
node-statsd-parser node-stack-trace node-oop node-mersenne 
node-long-stack-traces node-far node-env-variable node-color-name

runs without error and   and by running make check 
TESTS="tests/glob.scm" .


[Message part 3 (message/rfc822, inline)]
From: Jelle Licht <jlicht <at> fsfe.org>
To: paul <goodoldpaul <at> autistici.org>, 41219-done <at> debbugs.gnu.org
Subject: Re: [bug#41219] [PATCH 2/2] guix: Enforce package.json "files"
 directive.
Date: Wed, 09 Dec 2020 22:45:40 +0100
Dear Giacomo,

paul <goodoldpaul <at> autistici.org> writes:
> Dear Jelle,
>
> do you believe that you can merge the globstar patch? Otherwise I would 
> just close this issue if you agree with it.

I finally pushed your globstar patch to master in 371ba7b4b.

> Thank you for your time,

Thank you for your patience!
 - Jelle


This bug report was last modified 4 years and 165 days ago.

Previous Next


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