[Bug 1446568] Review Request: nodejs-opener - Opens stuff, like webpages and files and executables, cross-platform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1446568



--- Comment #3 from Tom Hughes <tom@xxxxxxxxxx> ---
A few rpmlint warnings:

nodejs-opener.noarch: W: spelling-error Summary(en_US) webpages -> web pages,
web-pages, passages

and:

nodejs-opener.noarch: E: wrong-script-interpreter
/usr/lib/node_modules/opener/opener.js /usr/bin/env node
nodejs-opener.noarch: E: non-executable-script
/usr/lib/node_modules/opener/opener.js 644 /usr/bin/env node

Those two relate to the fact that it is designed to be usable as a command line
program, so maybe it should be linked to bin? Though it would need to be made
executable and ideally the use of /usr/bin/env patched out.

Otherwise the #! should likely be removed?

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux