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
CI: Update build flags for fedora 40 #984
Conversation
aee6d6f
to
3fee5a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first two commits LGTM. For the last, maybe it would be good to document in the commit message why those need to change. I assume it's just due to changes in Fedora 40, and I trust you know what you're doing. But especially after Recent Events in xz, I guess it's better to state it explicitly.
For the first two commits (the refactors):
Reviewed-by: Alejandro Colomar <alx@kernel.org>
Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Alejandro Colomar <alx@kernel.org>
Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Alejandro Colomar <alx@kernel.org>
3fee5a0
to
1d6c70a
Compare
Updated to tackle your last comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
libpam is enabled to provide `passwd` binary from this package, as there are several password quality checks that are enabled through a PAM module. Same reason to disable account-tools-setuid. sssd is disabled because `files provider` has been removed in sssd, and the underlying functionality in shadow isn't needed anymore. libcrack dependency was disabled some time ago, but the upstream repo wasn't updated. Doing it now. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Alejandro Colomar <alx@kernel.org>
Now that fedora 40 is out let's update the flags.