Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20063 )
Change subject: IMPALA-12203: Default to Python 3 on modern distributions ...................................................................... IMPALA-12203: Default to Python 3 on modern distributions This modifies the postinstall.sh script to set python to point to python 3 for Linux distributions that do not offer python 2. This will apply to Redhat 9 and Ubuntu 22 when support is added, but it does not apply to any older distributions. The docker images can be used for building older native-toolchain versions, so none of the older distributions can be modified to use python 3. To support this, this cleans up some old python logic in a few packages. LLVM and Kudu can use system python without any issues, and they both support python 3 for this use case. This removes some obsolete code from assert-dependencies-present.py that used the Python 2 'platform' package. These checks do not apply to any supported distributions. This also changes postinstall.sh to use the last versions of pip (20.3.4) and awscli (1.19.112) that support Python 2. These are recent enough to also support newer Python 3 like Python 3.10. Testing: - Built Docker images and ran the build - Also ran the build for several older native-toolchain commits with the new Docker images to make sure that they continue to work. Change-Id: Ib10589927509eeaa44b628709dcdb5e45c4dd034 Reviewed-on: http://gerrit.cloudera.org:8080/20063 Reviewed-by: Michael Smith <[email protected]> Reviewed-by: Laszlo Gaal <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- M buildall.sh M docker/all/assert-dependencies-present.py M docker/all/postinstall.sh M source/kudu/build.sh M source/llvm/build-source-tarball.sh 5 files changed, 34 insertions(+), 40 deletions(-) Approvals: Michael Smith: Looks good to me, but someone else must approve Laszlo Gaal: Looks good to me, approved Joe McDonnell: Verified -- To view, visit http://gerrit.cloudera.org:8080/20063 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib10589927509eeaa44b628709dcdb5e45c4dd034 Gerrit-Change-Number: 20063 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
