[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#51838] [PATCH 00/11] guix: node-build-system: Support compiling add
From: |
Timothy Sample |
Subject: |
[bug#51838] [PATCH 00/11] guix: node-build-system: Support compiling add-ons with node-gyp. |
Date: |
Sat, 20 Nov 2021 14:55:31 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hello,
Philip McGrath <philip@philipmcgrath.com> writes:
> On 11/20/21 02:46, Liliana Marie Prikler wrote:
>> Am Freitag, den 19.11.2021, 23:33 -0500 schrieb Philip McGrath:
>>> * guix/build-system/node.scm (lower): Add an optional #:libuv?
>>> argument to tell the build system to add the version of libuv
>>> used as an input to the #:node package as an additional implicit
>>> input, so that packages needing libuv always get the correct version.
>> Would it change something if we always had libuv as implicit input? I
>> don't think it'd do much for closure size, but it might impact implicit
>> package search as proposed by my solution to the #:absent-dependencies
>> thing.
>
> From a Guix perspective, I don't think it would make much difference,
> since libuv is always needed by node itself. Maybe it would require
> more grafting than would be needed otherwise?
>
> But I think the recommendation for authors of node add-ons is to avoid
> libuv unless it's really needed, because it has fewer stability
> guarantees that other Node.js APIs.
My assumption is that most packages would ignore the libuv headers, and
they wouldn’t retain a reference to it (except via Node.js itself).
Hence, I don’t think it would make any difference either to grafting or
closure size to just always add it.
-- Tim
- [bug#51838] [PATCH 04/11] gnu: Add node-inherits., (continued)
- [bug#51838] [PATCH 04/11] gnu: Add node-inherits., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 09/11] gnu: Add node-openzwave-shared., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 06/11] gnu: Add node-string-decoder., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 08/11] gnu: Add node-nan., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 10/11] gnu: Add node-addon-api., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 11/11] gnu: Add node-sqlite3., Philip McGrath, 2021/11/14
- [bug#51838] [PATCH 00/11] guix: node-build-system: Support compiling add-ons with node-gyp., Timothy Sample, 2021/11/20
- [bug#51838] [PATCH 00/11] guix: node-build-system: Support compiling add-ons with node-gyp., Timothy Sample, 2021/11/20
- [bug#51838] [PATCH 00/11] guix: node-build-system: Support compiling add-ons with node-gyp.,
Timothy Sample <=
- [bug#51838] [PATCH v2 04/26] gnu: node: Add an npmrc file to set nodedir., Timothy Sample, 2021/11/20