[Bug 1796268] Package review request - nodejs-p-try

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

 



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



--- Comment #5 from Sergio Monteiro Basto <sergio@xxxxxxxxxx> ---
Fixing typos, the package isn't aproved yet 

- Buildarch is noarch is correct ,  we don't want try to build where nodejs
doesn't exist, all other packages have the same [1] 

- using_tarballs_from_the_npm_registry don't have test.js source, the rest of
the source is the same , I think the rule is obsolete , download from npm and
test.js from github is an optional that cause some mistakes because test.js
file have to be download from github and upload to fedora in every update ... 

[1] 
https://src.fedoraproject.org/rpms/nodejs-p-locate/blob/master/f/nodejs-p-locate.spec#_19

-- 
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
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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

  Powered by Linux