nodejs-packaging

#18 Do not require deps bundled in a different directory
Merged by sgallagh. Opened by tdawson.
tdawson/nodejs-packaging master  into  master

Download 18.patch

This wasn't working if there were bundled deps in a different directory, but linked into node_modules.
Removing the "not" works the way I believe this was intended.
I have tested this patch with:
no bundled dependencies: Requires proper dependencies
bundled deps in node_modules: Requires no nodejs dependencies
bundled deps in node_modules_prod and linked them into node_modules: Requires no nodejs dependencies

Signed-off-by: Troy Dawson tdawson@redhat.com

It took me a few minutes to wrap my head around the logic here, but this looks correct to me.

I'll get another update out immediately.

Pull-Request has been merged by sgallagh
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.