Bug 1275009
Summary: | Review Request: nodejs-chroma-js - JavaScript library for color conversions | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | William Moreno <williamjmorenor> |
Component: | Package Review | Assignee: | Piotr Popieluch <piotr1212> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | package-review, piotr1212 |
Target Milestone: | --- | Flags: | piotr1212:
fedora-review-
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-02-18 16:15:34 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 956806, 1274930 |
Description
William Moreno
2015-10-24 20:30:52 UTC
Thanks for packaging. This needs some work before I can approve: - License: should be BSD, not GPL, it also seems that there are more license files, please check if this is multi-licensed and set correct licenses - URL: Please set correct url which points to the actual package - BuildRequires, I expect there are some deps missing, at least grunt-cli - %prep, use "cp -p" to preserve timestamps - %build, package needs to be build with "grunt" - Please add %check section with tests - %install, nothing is being installed - %changelog, incorrect version 1.0.0 instead of 1.1.1 If you need help/clarification on some items let me know. Thanks for take the review. I am not sure about how to build the sources with grunt in %%build. Can please provide me a example. Excuse me for the late response you can build with: BuildRequires: npm(grunt-cli) BuildRequires: npm(grunt-contrib-clean) BuildRequires: npm(grunt-contrib-coffee) BuildRequires: npm(grunt-replace) BuildRequires: npm(grunt-contrib-uglify) %build %nodejs_symlink_deps --build grunt Only thing is that dependency grunt-replace is not packaged yet. You will have to package that one first (or just use sed to set the version in chroma.js, and patch out grunt-replace out of the Gruntfile.js). This is a quite old review request, do you still want to package this or can we close this request? Not rigth now, thanks for the review. |