https://bugzilla.redhat.com/show_bug.cgi?id=1285412 --- Comment #2 from Tom Hughes <tom@xxxxxxxxxx> --- Needs BRs on npm(lex-parser) and npm(test) when tests are enables. Also it does have a run time dependency on lex-parser that is not declared in the package.json because ebnf-parser.js requires it. It will work when used as part of jison because node will search up the tree and find lex-parser but it wouldn't work if used on it's own. So I think we should convert the devDependency on lex-parser to a normal one with something like: %nodejs_fixdep --dev -r lex-parser %nodejs_fixdep lex-parser 0.1.0 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review