Skip to content
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

selinux-module failing on RHEL 7, Makefile not there #88

Closed
ncsutmf opened this issue Mar 24, 2016 · 4 comments
Closed

selinux-module failing on RHEL 7, Makefile not there #88

ncsutmf opened this issue Mar 24, 2016 · 4 comments

Comments

@ncsutmf
Copy link
Contributor

ncsutmf commented Mar 24, 2016

Regarding a comment from Pull Request #73

It appears RedHat decided to split selinux into the selinux-policy and selinux-policy-devel packages for RHEL 7, and put /usr/share/selinux/devel/Makefile in the devel package. I assume this would also affect CentOS 7 and recent versions of Fedora as well.

The .te file gets created but no .mod or .pp file in /usr/share/selinux.

I didn't notice it until recently because the .mod and .pp files were already there, but I think the addition of the prefix caused it to attempt to install the module under a new name and then start giving puppet errors.

RHEL 6 seams to have the Makefile in selinux-policy and as best I can tell there is no selinux-policy-devel for RHEL 6.

Thoughts on the best way to fix this?

@legooolas
Copy link

I'd suggest that either:

  • The "package_name" param should be changed to allow a list of packages, which includes selinux-policy-devel on RHEL7-based distros
  • A separate "module_package_name" param is added and used to include the package only when the selinux::module definition is used.

I'm happy to provide a patch for whichever of these is preferable (or something else :) as I'm adding the package explicitly for now and it would be nice to have the selinux module handle this on my behalf.

@ncsutmf
Copy link
Contributor Author

ncsutmf commented Apr 25, 2016

Currently the package is policycoreutils-python which causes selinux-policy to install as well. I have already submitted a pull request #89 to change it to policycoreutils-devel, which requires policycoreutils-python and also causes selinux-policy-devel to install.

@legooolas
Copy link

legooolas commented Apr 27, 2016

Fab -- I'll use this handy workaround of overriding the package_name to be policycoreutils-python-devel for now :)

@ncsutmf
Copy link
Contributor Author

ncsutmf commented Apr 27, 2016

policycoreutils-devel (no 'python' on the name)

@jfryman jfryman closed this as completed May 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants