search

ros-infrastructure / ros_buildfarm

ROS buildfarm based on Docker
Apache License 2.0
81 stars 96 forks source link

Revert "Work around a packaging bug in RHEL 8.4" #899

Closed cottsay closed 3 years ago

cottsay commented 3 years ago

This reverts #889.

nuclearsandwich commented 3 years ago

What makes this change no longer necessary? From the bug linked in comments https://bugzilla.redhat.com/show_bug.cgi?id=1960791 I see it has a status of VERIFIED which suggests that it still requires a release and closure according to this doc.

cottsay commented 3 years ago

What makes this change no longer necessary?

The CMake package has been updated and provides the correct virtual package now. We're looking for the cmake-3.18 package in this list:

$ dnf provides cmake3 -q --showduplicates
cmake-3.11.4-3.el8.x86_64 : Cross-platform make system
Repo        : rhel-8-for-x86_64-appstream-rpms
Matched from:
Provide    : cmake3 = 3.11.4-3.el8

cmake-3.11.4-7.el8.x86_64 : Cross-platform make system
Repo        : rhel-8-for-x86_64-appstream-rpms
Matched from:
Provide    : cmake3 = 3.11.4-7.el8

cmake-3.18.2-11.el8_4.x86_64 : Cross-platform make system
Repo        : rhel-8-for-x86_64-appstream-rpms
Matched from:
Provide    : cmake3 = 3.18.2-11.el8_4

I see it has a status of VERIFIED which suggests that it still requires a release and closure

I'm not sure what they're up to in that bug. It seems like it's tracking some sort of documentation change or something, and it says that it's targeting RHEL 8.5. One of the folks in that thread linked me to a bug that's supposedly for RHEL 8.4, but it's not public, so I have no idea where to go to get official status on the issue.

nuclearsandwich commented 3 years ago

One of the folks in that thread linked me to a bug that's supposedly for RHEL 8.4, but it's not public, so I have no idea where to go to get official status on the issue.

Yeah, I read that. :shrug: if it works it works but it's a shame that they're publicly quiet about it from a documentation / regression tracking perspective.