You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2018/08/06 23:26:38 UTC

[arrow] branch master updated: ARROW-3001: [Packaging] Don't modify PATH during rust release verification

This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 7bead55  ARROW-3001: [Packaging] Don't modify PATH during rust release verification
7bead55 is described below

commit 7bead55da79e6e3e309e148af267c776c7eb2735
Author: Krisztián Szűcs <sz...@gmail.com>
AuthorDate: Mon Aug 6 19:26:34 2018 -0400

    ARROW-3001: [Packaging] Don't modify PATH during rust release verification
    
    Sadly rustup has a side effect, it modifies .profile or .zprofile.
    
    Author: Krisztián Szűcs <sz...@gmail.com>
    
    Closes #2384 from kszucs/ARROW-3001 and squashes the following commits:
    
    cffef9ae <Krisztián Szűcs> rustup no modify path
---
 dev/release/verify-release-candidate.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dev/release/verify-release-candidate.sh b/dev/release/verify-release-candidate.sh
index 220a79b..49cd950 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -254,7 +254,9 @@ test_rust() {
   export RUSTUP_HOME=`pwd`/test-rustup
   export CARGO_HOME=`pwd`/test-rustup
 
-  curl https://sh.rustup.rs -sSf | sh -s -- -y
+  curl https://sh.rustup.rs -sSf | sh -s -- -y --no-modify-path
+
+  export PATH=$RUSTUP_HOME/bin:$PATH
   source $RUSTUP_HOME/env
 
   # build and test rust