Tim Armstrong has posted comments on this change.

Change subject: Download GCC deps from S3, not gnu.org
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2553/1/source/gcc/build.sh
File source/gcc/build.sh:

Line 33:   download_dependency $LPACKAGE "mpfr-${MPFR_VERSION}.tar.bz2" 
$THIS_DIR
       :   tar xjf "../mpfr-${MPFR_VERSION}.tar.bz2"
       :   ln -s mpfr-${MPFR_VERSION} mpfr
> Are these paths correct? This doesn't read well... it looks like something 
I was emulating what the LLVM build script does but you're right, it doesn't 
make much sense. Changed to just do everything in .


-- 
To view, visit http://gerrit.cloudera.org:8080/2553
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I810b8debe91a1450c05f5cff39e885c95a6632f0
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to