[Pkg-javascript-devel] Bug#872433: Bug#872433: [with solution] Doesn't find modules installed in Debian directories

[Bastien ROUCARIES]

Let ping me at rouca at debian.org if needed

Could you also modernize this package ? policy bump, autopkg-test (see
node-tape)

On Thu, Aug 17, 2017 at 1:40 PM, Julien Puydt <julien.puydt at laposte.net> wrote:
> Package: node-resolve
> Version: 1.1.7-2
> Severity: grave
>
> Hi,
>
> I'm surprised nobody reported it yet since it basically makes the
> package useless as far as I see, but the current node resolve doesn't
> find modules installed in Debian directories : while working on another
> package, I was surprised that nothing was found. When investigating
> using strace, I saw that resolve.sync was always adding "node_modules"
> to the paths it tried, so of course it didn't find anything.
>
> I got things to work by editing node-modules-paths.js from:
>
> module.exports = function nodeModulesPaths(start, opts) {
>     var modules = opts && opts.moduleDirectory
>         ? [].concat(opts.moduleDirectory)
>         : ['node_modules'];
>
> to:
>
> module.exports = function nodeModulesPaths(start, opts) {
>     var modules = opts && opts.moduleDirectory
>         ? [].concat(opts.moduleDirectory)
>         : ['node_modules', ''];
>
> ie: I added '' to the list of things to add when generating paths to check.
>
> The following two lines fail with the unpatched node-resolve, and work
> with the patched one:
>     resolve=require('resolve')
>     resolve.sync('resolve/lib/core.js', {basedir: '/usr/lib/nodejs'})
> (it should be added in debian/tests/)
>
> I'm part of the Debian Javascript maintainers team so I can add the
> necessary patch (with the right header) and test [and probably push
> higher upstream version, std-ver and dh] to the git repository to help
> if you want -- but I'm no DD so can't upload myself.
>
> Cheers,
>
> Snark on #debian-js
>
> --
> Pkg-javascript-devel mailing list
> Pkg-javascript-devel at lists.alioth.debian.org
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel