Skip to content

Draft: Add npm subpackage

Jan Staněkrequested to merge
jstanek-rh/nodejs-meta:main into main

I did not have that many comments as I thought I would have, which is nice. 🙂

This is mainly the addition of the nodejs-npm subpackage, that mirrors the relationship of the stream packages. It is also this subpackage that will try to pull the npm-bin. I did not yet run the test on it; as that would require trampling over the COPR, I wanted to first present the changes for discussion.

I'm still not sure about the Provides: npm dependency, as anything else than just this non-versioned VP will get complicated fast (mainly because the -npm subpackages have their own versions). But if we only want to support the dnf install npm case and not worry about e.g. dnf install nodejs22 && dnf install npm (= install npm for nodejs22), this simple addition might be enough. I guess only time will tell which invocations will be attempted.

Apart from the above, the minor changes include:

  • Small rewording and reformatting
  • Renaming of the Epoch-Version-Release macro to %{nodejs_evr} and moving it to place where it actually should be properly set.
  • Relying on %{version} in favor of introducing another macro for the same stuff.

Merge request reports