-
Notifications
You must be signed in to change notification settings - Fork 212
libdnf: Upgrade to latest rhel-9.6 #5140
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
Conversation
8bff603 to
54fc510
Compare
|
|
Ah yes we should probably use https://github.com/rpm-software-management/libdnf/commits/rhel-9.6 or so |
|
Or of course switch to just cherry picking changes for rhel-9, or create a rhel-9 branch here upstream. |
54fc510 to
084f007
Compare
There's quite a lot of changes here...I'm hopeful this fixes compatibility with clang, although I can't reproduce the build failure locally. Update submodule: libdnf Signed-off-by: Colin Walters <walters@verbum.org>
084f007 to
e331dfd
Compare
|
@cgwalters: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
I can get the original error from: Downgrading clag does nothing for me. Now that I can reproduce it in a container will try to find if there is any specific libdnf branch/commit that works. |
|
So what bit me is the fact that we don't rebuild libdnf unless you |
|
Ummm...I think what happened here is rpm dropped support for pgp in rpm-software-management/rpm#2984 So there's no maintained libdnf that builds on both fedora and rhel9 which definitely puts us in a pickle. I dunno. I guess we could also just fork at this point. |
|
So in theory in would be good to sync to this branch for rhel but for now let's put off that tech debt unless there's a reason to because it would force us to fork here which has nontrivial implications. |
|
If we do do this PR we'll also want |
coreos#5137 Avoids forking for rhel9 for now as we don't need a clang build for Fedora or RHEL. Will fix in the future when forking for rhel9 is unavoidable rhel9 fork would look as described in: coreos#5140
This workaround coreos#5137 by avoiding forking for rhel9 for now, we don't need a clang build for Fedora or RHEL. Will fix in the future when forking for rhel9 is unavoidable rhel9 fork would look as described in: coreos#5140
This works around coreos#5137 by avoiding forking for rhel9 for now, we don't need a clang build for Fedora or RHEL. Will fix in the future when forking for rhel9 is unavoidable rhel9 fork would look as described in: coreos#5140
There's quite a lot of changes here...I'm hopeful this fixes compatibility with clang, although I can't reproduce the build failure locally.