[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#62375] [PATCH 0/1] npm binary importer
From: |
Ludovic Courtès |
Subject: |
[bug#62375] [PATCH 0/1] npm binary importer |
Date: |
Mon, 17 Apr 2023 23:14:52 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Jelle Licht <jlicht@fsfe.org> skribis:
> Ludovic Courtès <ludo@gnu.org> writes:
[...]
>>> + (match npm-package
>>> + (($ <package-revision> name version home-page dependencies
>>> dev-dependencies peer-dependencies license description dist)
>>
>> Please use ‘match-record’ instead and keep lines below 80 chars. :-)
>
> The records generated by `define-json-mapping' through
> `define-record-type' seem to not work with `match-record'.
Oh right, my bad.
The other option is to call record accessors; it would avoid the risk
associated with index-based matches but may be more verbose. Your call!
Ludo’.