-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Conversation
|
||
toDownload.push({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redundant code L153-L156
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, fixed!
hiccups during the unzip stage. |
Fixed unzipping. |
Sweet, works for me. Let's wait for travis to ACK. |
|
||
return merge.apply(null, streams); | ||
if (0 <= osArch.indexOf('linux')) { | ||
ret = shell.exec(`tar -xzf ./nodes/geth/${zipFileName} -C ./nodes/geth/${osArch}`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tar -xf
suffices as it is not gunzipped. This should fix travis. (http://stackoverflow.com/a/15744167)
Though I wonder about this behaviour
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @hiddentao
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@luclu should be fixed
Works great for me on with all --platform combinations. |
This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread. |
This cleans up and fixes the node download logic, fixing the Linux x64 geth node problem too.