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

Drop cockpit module and stop ignoring standalone interface files #856

Merged
merged 2 commits into from Sep 30, 2022

Conversation

vmojzis
Copy link
Member

@vmojzis vmojzis commented Sep 1, 2021

No description provided.

@vmojzis
Copy link
Member Author

vmojzis commented Sep 1, 2021

This is an alternative to #853

@vmojzis
Copy link
Member Author

vmojzis commented Sep 1, 2021

Again, the build-rpm fail is expected since cockpit needs to be removed from modules-targeted-contrib.conf (but setting it to "OFF" is not necessary with this approach).

@vmojzis
Copy link
Member Author

vmojzis commented Sep 22, 2021

Rebased

@marusak
Copy link

marusak commented Jan 19, 2022

Any update on this?

@zpytela
Copy link
Contributor

zpytela commented Jan 19, 2022

Any update on this?

Now it's time to move it forward.

@vmojzis
Copy link
Member Author

vmojzis commented Mar 7, 2022

Rebased.

The module is now maintained and shipped as part of cockpit.
https://github.com/cockpit-project/cockpit/tree/main/selinux

Signed-off-by: Vit Mojzis <vmojzis@redhat.com>
Interface files without corresponding .te where ignored, unless the
module name was specified in modules.conf.

Standalone interface files are useful for backwards compatibility in
case a policy module removed from this repository.

Signed-off-by: Vit Mojzis <vmojzis@redhat.com>
@vmojzis
Copy link
Member Author

vmojzis commented Mar 21, 2022

rebased

@zpytela zpytela merged commit 3a55e3a into fedora-selinux:rawhide Sep 30, 2022
@zpytela
Copy link
Contributor

zpytela commented Sep 30, 2022

Merged, thank you for the effort.

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

Successfully merging this pull request may close these issues.

None yet

3 participants