From 03279a08471d4aad48f9352245640465f5f20142 Mon Sep 17 00:00:00 2001 From: Troy Dawson Date: Sep 01 2020 17:31:08 +0000 Subject: Do not require deps bundled in a different directory 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 --- diff --git a/nodejs.req b/nodejs.req index 945c23a..129606b 100755 --- a/nodejs.req +++ b/nodejs.req @@ -635,7 +635,7 @@ def has_only_bundled_dependencies(module_dir_path): bundled_dependency_iter = ( os.path.realpath(path) for path in dependency_path_iter - if not os.path.islink(path) or not path.startswith(module_root_path) + if not os.path.islink(path) or path.startswith(module_root_path) ) return any(bundled_dependency_iter)