Skip to content

Adds a spec file for building an RPM package #1052

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

Closed
wants to merge 1 commit into from
Closed

Adds a spec file for building an RPM package #1052

wants to merge 1 commit into from

Conversation

antonyh
Copy link

@antonyh antonyh commented Jan 12, 2016

I needed to create an RPM file to install ModSecurity in environments where it's not possible to build from source and where there's no available RPM that is suitable. I figured the best place to share this is here so that others can benefit, but also so it can be reviewed and corrected, and I couldn't find any guidelines on what you consider acceptable for pull requests.

It's a single RPM spec file, installs/uninstalls cleanly, tested on Centos 6.6. I hope this is a useful addition to ModSecurity, and if there's alterations needed I'm happy to help.

@zimmerle
Copy link
Contributor

Hi @antonyh,

Thank you for the patch. We don't usually host the specs file into our repository. But that is may be a good idea. Like you said it will be very helpful for the community in general.

I am going to test the package generation.

@zimmerle zimmerle added this to the v2.9.1 milestone Jan 25, 2016
@zimmerle zimmerle self-assigned this Jan 25, 2016
@zimmerle
Copy link
Contributor

Hi @antonyh,

Thank you again for the pull request. We had our community meeting this week, where we decide to have a wiki page to host everything related to the package creation, including the recipes.

The wiki page is available here:
https://github.com/SpiderLabs/ModSecurity/wiki/Package-Generation

I already placed you rpm recipe there! Thanks!

If you can, please fill the information about how to actually generate the packages, it will be very helpful to the community.

@zimmerle zimmerle closed this Jan 29, 2016
@antonyh
Copy link
Author

antonyh commented Feb 1, 2016

I've updated the wiki based on memory (and shell history) - it should be accurate. I'll double check it later to confirm and update if needed. I've also added how to install, as it wasn't obvious to me how to get rpm to pull in dependencies.

Thank you for taking the time to review this and to share it with the community.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants