GNU bug report logs - #62375
[PATCH 0/1] npm binary importer

Previous Next

Package: guix-patches;

Reported by: jlicht <at> fsfe.org

Date: Wed, 22 Mar 2023 11:26:01 UTC

Severity: normal

Tags: moreinfo, patch

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

Bug is archived. No further changes may be made.

Full log


Message #63 received at 62375-done <at> debbugs.gnu.org (full text, mbox):

From: Jelle Licht <jlicht <at> fsfe.org>
To: 62375-done <at> debbugs.gnu.org
Cc: Timothy Sample <samplet <at> ngyro.com>, Lars-Dominik Braun <lars <at> 6xq.net>
Subject: Re: [PATCH v4] import: Add binary npm importer.
Date: Tue, 25 Feb 2025 11:14:02 +0100
Hey folks,

This was merged quite some time ago into master already, so closing.

jlicht <at> fsfe.org writes:

> From: Jelle Licht <jlicht <at> fsfe.org>
>
> * guix/scripts/import.scm: (importers): Add "npm-binary".
> * guix/import/npm-binary.scm: New file.
> * guix/scripts/import/npm-binary.scm: New file.
> * Makefile.am: Add them.
>
> Co-authored-by: Timothy Sample <samplet <at> ngyro.com>
> Co-authored-by: Lars-Dominik Braun <lars <at> 6xq.net>
>
> Change-Id: I98a45068cf5b9c42790664cc743feaa7ac76f807
> ---
>
> Changes in v4:
> - Add copyright line for LDB
> - Use package-name <at> version-spec notation on CLI
> - Simplify CLI argument handling
> - Turn %registry into a parameter named %npm-registry
>
> Changes in v3:
> - Ensure that package bindings generated during recursive import match the
> package bindings used in the list of inputs
>
> Changes in v2:
> - Change *SOME-VAR* to %SOME-VAR
> - Removed unused http-error-code
> - Rebase on master
> - Refactor hash-url to use port-sha256 helper
> - use explicit record accessors instead of order-sensitive destructuring
> - address line-width styling issues
> - added basic documentation
> - added some basic tests (using simple mocks instead of with-http-server)
> - simplify import script entrypoint
>
>  Makefile.am                        |   3 +
>  doc/guix.texi                      |  33 ++++
>  guix/import/npm-binary.scm         | 279 +++++++++++++++++++++++++++++
>  guix/scripts/import.scm            |   2 +-
>  guix/scripts/import/npm-binary.scm | 121 +++++++++++++
>  tests/npm-binary.scm               | 146 +++++++++++++++
>  6 files changed, 583 insertions(+), 1 deletion(-)
>  create mode 100644 guix/import/npm-binary.scm
>  create mode 100644 guix/scripts/import/npm-binary.scm
>  create mode 100755 tests/npm-binary.scm




This bug report was last modified 142 days ago.

Previous Next


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