GNU bug report logs -
#51838
[PATCH 00/11] guix: node-build-system: Support compiling add-ons with node-gyp.
Previous Next
Reported by: Philip McGrath <philip <at> philipmcgrath.com>
Date: Sun, 14 Nov 2021 12:43:01 UTC
Severity: normal
Tags: patch
Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
Hi Liliana,
Thanks for putting this together! I’m starting to think that we might
actually land this series pretty soon. :)
Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
> 4. Regexps :)
I doubt regex support will be broadly useful here. Putting the anchors
in every package name (e.g., "^tap$") makes for a lot of noise. My
(wild) guess would be that regexes will save us listing two dependencies
for one out of every ten Node packages. Given that, my preference would
be to not bother with regex support.
You wrote this in another message:
> I think it'd be beneficial if delete-dependencies could delete
> dependencies based on their name matching a regexp rather than a
> string exactly. This would make some of your lists shorter
> (e.g. "karma.*"), but there might be a debate on whether to use
> "^karma.*$" or whether to only consider regexps that match the
> dependency fully.
If nothing else, I’m certainly on the other side of this debate! :)
If every string is going to be treated as a pattern, we should have it
match fully by default. That is, the anchors should be implicit. For
the very rare (never?) case where you want to avoid anything that so
much as has “foo” in the name, it’s pretty easy to write ".*foo.*".
-- Tim
This bug report was last modified 3 years and 195 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.