Laszlo Gaal has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/15831 )
Change subject: IMPALA-9686: Add readline to Python ...................................................................... IMPALA-9686: Add readline to Python Add the readline developer packages to the toolchain build containers. These are needed by the Python build, as it figures out which packages to build during compilation time by going through the environment and finding these packages dynamically. Any packages that miss their runtime dependencies are dropped from the build; the only sign is some warning messages in the build log. SLES12 puts ncurses include file in a separate subdirectory under /usr/include. This breaks compiling readline on SLES12 if the extra directoyr is not added to the include path, so fix this in Python's build.sh Runtime asserts were also added to the Python build driver for the issues uncovered so far. Tests: - built the container images - built the toolchain in the new container configs - tested Python from the new build for the presence of readline support Change-Id: I5bbe6fa16305a4462278306947535fad468cb3d8 --- M docker/debian8.df M docker/redhat7.df M docker/redhat8.df M docker/sles12.df M docker/ubuntu1604.df M docker/ubuntu1804.df M source/python/build.sh 7 files changed, 9 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/31/15831/2 -- To view, visit http://gerrit.cloudera.org:8080/15831 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5bbe6fa16305a4462278306947535fad468cb3d8 Gerrit-Change-Number: 15831 Gerrit-PatchSet: 2 Gerrit-Owner: Laszlo Gaal <laszlo.g...@cloudera.com>
