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

Piotr Popieluch <piotr1...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-review-



--- Comment #4 from Piotr Popieluch <piotr1...@gmail.com> ---
Seems that the source has multiple licenses, they all need to be specified.

dist dir needs to be installed. Binary as well. Not sure about externs dir.


%build
%nodejs_symlink_deps --check
                     ^^ shouldn't this be --build


%{?nodejs_find_provides_and_requires}
%{!?_licensedir:%global license %doc}
These macros are not needed anymore

tests seem to be included in npm package now. Is the separate download still
needed?

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to