You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2016/04/21 17:55:59 UTC

[Toolchain-CR] Fix bug with publishing

Tim Armstrong has uploaded a new change for review.

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

Change subject: Fix bug with publishing
......................................................................

Fix bug with publishing

A previous commit "Add support for Debug and Release+Asserts builds for
LLVM 3.7+" caused the native toolchain to publish artifacts with the
wrong versions and directories: the package name + version appeared
where previously there was only the package version. The commit
passed testing because the test was run with publishing disabled.

This fixes the problem. I did a test run with publishing enabled,
and the artifacts ended up in the right place on S3.

Change-Id: I8f14b43ada82f995e2385a27cfacfc6f23fd4a7c
---
M functions.sh
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Toolchain refs/changes/32/2832/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2832
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8f14b43ada82f995e2385a27cfacfc6f23fd4a7c
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>