You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Michael Smith (Code Review)" <ge...@cloudera.org> on 2022/06/01 18:00:48 UTC

[Impala-ASF-CR] [tools] Fix building shell pypi package

Hello Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18578

to look at the new patch set (#2).

Change subject: [tools] Fix building shell_pypi_package
......................................................................

[tools] Fix building shell_pypi_package

Fixes an error building shell_pypi_package with junit_xml_wrapper.sh.
Previously `make shell_pypi_package` would error with

  /home/michael/Impala/bin/junitxml_command_wrapper.sh: line 46:
  DIST_DIR=/home/michael/Impala/shell/dist CLEAN_DIST=true
  /home/michael/Impala/shell/packaging/make_python_package.sh:
  No such file or directory

Uses `env` and CMake tokenization to ensure junitxml has a valid
executable to run.

Change-Id: I60ffac3edf1a6027afa4ca46ab6dadfc6bfc660a
---
M CMakeLists.txt
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/18578/2
-- 
To view, visit http://gerrit.cloudera.org:8080/18578
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I60ffac3edf1a6027afa4ca46ab6dadfc6bfc660a
Gerrit-Change-Number: 18578
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>