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

Support Fedora 41 in Dockerfile #3418

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

dmnks
Copy link
Contributor

@dmnks dmnks commented Oct 30, 2024

This is needed for "make check" to work on a Fedora 41 host where the FROM directive is overridden to fedora:41.

  • Drop the workaround for gdb-headless on F41, it's no longer available there and the bug is fixed on F40+: https://bugzilla.redhat.com/show_bug.cgi?id=2310156

  • Don't try to remove rpm-sign-libs and python3-rpm on F41 since dnf5 no longer pulls them in.

  • Make sure to call "dnf clean all" before "rpm -e" as python3-rpm is needed by dnf4 to function (hence the duplicated call).

  • Add a comment at the top saying which Fedora versions are supported by the Dockerfile. This could be checked automatically during cmake config but let's leave that for later.

  • Detect F41 vs. older by looking for the dnf5 binary, it's the easiest way.

@dmnks dmnks requested a review from a team as a code owner October 30, 2024 12:05
@dmnks dmnks requested review from ffesti and removed request for a team October 30, 2024 12:05
@dmnks
Copy link
Contributor Author

dmnks commented Oct 30, 2024

Oops, I completely forgot why we added the gdb-headless version range via ea4a98b. Spoiler alert: to support F39. We don't want to drop support for F39 just yet so let me fix this up.

@dmnks dmnks added the DONT DO NOT merge, for whatever reason label Oct 30, 2024
@dmnks dmnks marked this pull request as draft October 30, 2024 12:14
@dmnks dmnks removed the DONT DO NOT merge, for whatever reason label Oct 30, 2024
@dmnks dmnks marked this pull request as ready for review October 30, 2024 13:58
@dmnks
Copy link
Contributor Author

dmnks commented Oct 30, 2024

Fixed.

This is needed for "make check" to work on a Fedora 41 host where the
FROM directive is overridden to fedora:41.

- Drop the workaround for gdb-headless on F41, it's no longer available
  there and the bug is fixed on F40+:
  https://bugzilla.redhat.com/show_bug.cgi?id=2310156

- Don't try to remove rpm-sign-libs and python3-rpm on F41 since dnf5 no
  longer pulls them in.

- Make sure to call "dnf clean all" before "rpm -e" as python3-rpm is
  needed by dnf4 to function (hence the duplicated call).

- Add a comment at the top saying which Fedora versions are supported by
  the Dockerfile.  This could be checked automatically during cmake
  config but let's leave that for later.

- Detect F41 vs. older by looking for the dnf5 binary, it's the easiest
  way.
@ffesti ffesti merged commit 78cecdc into rpm-software-management:master Oct 31, 2024
1 check passed
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.

2 participants