Merge ~sergiodj/ubuntu/+source/node-uid-number:bug1877704-install-get-uid-gid-groovy into ubuntu/+source/node-uid-number:ubuntu/groovy-devel
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Sergio Durigan Junior | ||||
Approved revision: | 277fe82690c5146bc2be10705b015f1caf10c5ca | ||||
Merged at revision: | 277fe82690c5146bc2be10705b015f1caf10c5ca | ||||
Proposed branch: | ~sergiodj/ubuntu/+source/node-uid-number:bug1877704-install-get-uid-gid-groovy | ||||
Merge into: | ubuntu/+source/node-uid-number:ubuntu/groovy-devel | ||||
Diff against target: |
22 lines (+7/-0) 2 files modified
debian/changelog (+6/-0) debian/install (+1/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Bryce Harrington (community) | Approve | ||
Canonical Server Core Reviewers | Pending | ||
Review via email: mp+398539@code.launchpad.net |
Description of the change
This is a fix for bug 1877704.
It's very simple: the problem is that the package on Focal and Groovy does not install the "get-uid-gid.js" file, which a requirement for it to function properly. node-uid-number is a dependency of npm, which is an important package, and this means that npm will have trouble working.
There's a test plan written in the bug, which I'm pasting here:
Inside a Groovy container:
# apt install npm -y
# npm install yarn -g
# rm .npm/_logs/*.log
# grep get-uid-gid .npm/_logs/
57 verbose stack Error: Cannot find module './get-uid-gid.js'
64 error Cannot find module './get-uid-gid.js'
When testing the fixed package, make sure to delete the logs under .npm/_logs/.
There's a PPA with the proposed change here:
https:/
The dep8 tests are still happy:
autopkgtest [12:02:51]: @@@@@@@
require PASS
LGTM, +1
Since this looks pretty trivial, I'm skipping all the usual review validation.
The SRU process will already add more than ample review/testing.