search

redhat-cip / edeploy

Linux systems provisioning and updating made easy
Apache License 2.0
92 stars 38 forks source link

base: Add selinux targeted policies packages #142

Closed Spredzy closed 10 years ago

Spredzy commented 10 years ago

Add the respective packages containing the targeted selinux policies

ErwanAliasr1 commented 10 years ago

I'm sorry @Spredzy, I don't get the need of creating rpm_packages here. It is only used once while the deb_packages was used twice thanks to the deb/ubutnu differences.

Except that, ok for the merge.

Spredzy commented 10 years ago

Thanks @ErwanAliasr1 you're right, sorry I didn't catch that earlier, PR has been updated accordingly.

ErwanAliasr1 commented 10 years ago

Thx !

ghantoos commented 10 years ago

(i know I have nothing to do with this PR, and that it has already been merged)

I would have kept this variable in case we decided to add support for other Red Hat based distributions (e.g. centos).

ErwanAliasr1 commented 10 years ago

We already do thanks to this line: https://github.com/enovance/edeploy/blob/master/build/base.install#L200

So that's why I asked yanis not to split the package definition until we do split the case for centos|redhat.

2014-10-15 15:05 GMT+02:00 Ignace Mouzannar notifications@github.com:

(i know I have nothing to do with this PR, and that it has already been merged)

I would have kept this variable in case we decided to add support for other Red Hat based distributions (e.g. centos).

— Reply to this email directly or view it on GitHub https://github.com/enovance/edeploy/pull/142#issuecomment-59201585.