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

refactor: use primary package names instead of aliases #192

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

richm
Copy link
Contributor

@richm richm commented Sep 16, 2023

The package names python3-libselinux and python3-policycoreutils
are the actual package names. The others are aliases - they
are Provides: in the RPM. Using the actual package names makes
it easier for ostree integration as well.

Signed-off-by: Rich Megginson [email protected]

The package names `python3-libselinux` and `python3-policycoreutils`
are the actual package names.  The others are aliases - they
are `Provides:` in the RPM.  Using the actual package names makes
it easier for ostree integration as well.

Signed-off-by: Rich Megginson <[email protected]>
@richm
Copy link
Contributor Author

richm commented Sep 16, 2023

[citest]

@richm
Copy link
Contributor Author

richm commented Sep 16, 2023

see also ansible-collections/ansible.posix#494

@bachradsusi
Copy link
Member

Ack.

FTR: libselinux-python3 and policycoreutils-python3 packages were renamed 5 years ago in order to follow standard python naming - https://src.fedoraproject.org/rpms/policycoreutils/c/c6b051c966b286ccb98d478a9584dce691ee8854

On RHEL7 libselinux still built with "libselinux-python3" with provides "python3-libselinux" - but it does not seem to be relevant for this case as there's no python3 version of policycoreutils

@richm richm merged commit fb9c7ac into linux-system-roles:main Sep 19, 2023
@richm richm deleted the correct-package-names branch September 19, 2023 13:34
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.

3 participants