-
Notifications
You must be signed in to change notification settings - Fork 29.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: update rpmbuild .spec file #71
Conversation
lgtm |
Version: %{_version} | ||
Release: 1 | ||
Summary: Node.js is a platform for building fast, scalable network applications. | ||
Summary: io.js is a platform for building fast, scalable network applications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is "io.js" lowercased even if it starts a sentence?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case we should note it somewhere like npm guys did. Maybe in readme.
Rename the package to iojs. No Conflicts: header is necessary because the package was already marked as conflicting with the Fedora nodejs package. PR-URL: nodejs#71 Reviewed-By: Rod Vagg <[email protected]>
Rename the .spec file from node.spec to iojs.spec and update the build script. Done as a separate commit to not obscure the changes from the previous commit. PR-URL: nodejs#71 Reviewed-By: Rod Vagg <[email protected]>
b4c6c19
to
993fadb
Compare
Fixes nodejs/node-v0.x-archive#8237. PR-URL: #71 Reviewed-By: Ben Noordhuis <[email protected]>
Rename the package to iojs. No Conflicts: header is necessary because
the package was already marked as conflicting with the Fedora nodejs
package.
R=@rvagg, possibly useful for nodejs/build#18.