Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20069 )
Change subject: IMPALA-12179 (part 1): Remove dependency on lsb_release for docker CMake ...................................................................... IMPALA-12179 (part 1): Remove dependency on lsb_release for docker CMake Newer operating systems like Redhat 9 do not supply lsb_release as an official package. The /etc/os-release file provides the same information in a more convenient form. CMake 3.22 added support for reading those /etc/os-release values directly via cmake_host_system_information(). This changes docker/CMakeLists.txt to use the new CMake cmake_host_system_information() APIs to get values from /etc/os-release. This removes the lsb_release code. Testing: - Ran a docker build locally and verified it detected the distribution / version correctly Change-Id: I04afd2b1c923f1331f7234d53a105a17956e3e18 Reviewed-on: http://gerrit.cloudera.org:8080/20069 Reviewed-by: Michael Smith <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- M CMakeLists.txt M docker/CMakeLists.txt 2 files changed, 23 insertions(+), 26 deletions(-) Approvals: Michael Smith: Looks good to me, approved Joe McDonnell: Verified -- To view, visit http://gerrit.cloudera.org:8080/20069 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I04afd2b1c923f1331f7234d53a105a17956e3e18 Gerrit-Change-Number: 20069 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
