-
Notifications
You must be signed in to change notification settings - Fork 549
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
RPM package builds #6
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Prevent check-buildroot from balking at the nginx string for the -V info
…her. It is more accurate that way, anyway
…tutions in the reverse order to prevent premature changes
In preparation for making it buildable alone
Rather than jump through hoops to fix them before building, just move them afterward.
… installed files. Upstream edits have rendered them unnecessary.
(or other distributions/architectures)
Blessed patch from Hong Li Removed (most of) the kludges to remove %{builddir} from installed files.
* add -z to tar to compress the file (to match the name) * Use rsync instead of cp -r to exclude directories/files from the tarball. * Fix noisy_system
Also move the list into an external file built by the %install phase.
This is VERY troubling, but for now just make sure they run first.
This was referenced May 29, 2014
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These still require a patch for the "natively-packaged" runtime directory to work (I mentioned that in the mail is just sent), but everything builds from rake tasks, and it should fetch all the dependencies automatically.
The one idiosyncrasy is that the user needs to be in the "mock" group to run mock, which may require them to invoke another login shell (most easily by logging out/in).
NOTE: Github's auto-pull-request tool would only let me request a pull into an existing branch. Currently this request, if accepted, will merge my changes into "master." It is probably better if it went to a new branch (mine is currently called "rpm") and then we can decide if/when to merge down to master.