Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

update geth 1.5.2 download links #1436

Merged
merged 9 commits into from
Nov 21, 2016
Merged

update geth 1.5.2 download links #1436

merged 9 commits into from
Nov 21, 2016

Conversation

luclu
Copy link
Contributor

@luclu luclu commented Nov 20, 2016

Change update URL to https://raw.githubusercontent.com/ethereum/mist/luclu_update-geth-1.5.2/clientBinaries.json to test.

Tested:

  • linux64
  • mac
  • win64
  • win32

@mention-bot
Copy link

@luclu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @alexvandesande and @hiddentao to be potential reviewers.

@frozeman
Copy link
Contributor

@evertonfraga can you test the rest?

@frozeman
Copy link
Contributor

If they only provide MD5, lets add a md5 check.

@luclu
Copy link
Contributor Author

luclu commented Nov 21, 2016

Copied from the chat:

Mah win isn’t working yet. While unpacking geth-windows-amd64-1.5.2-c8695209\\geth.exe it should extract geth.exe but instead creates a new folder called dows-amd64-1.5.2-c8695209.
The first missing characters geth-win have the same length as geth.exe. ;P

Hmm, this behaviour is caused by missing error handling at the unzip statement. unzip fails with this error on both windows zips:

Error: invalid signature: 0x80014

As a migration path we can upload new non-problematic zips to bintray/github.
Here is a basic script to trigger the error:

var fs = require('fs');
var unzip = require('unzip');

fs.createReadStream('geth-windows-386-1.5.2-c8695209.zip').pipe(unzip.Extract({ path: 'output' }));

Apperently this is a 2 years old issue. There also is a fork fixing it as the packing won’t receive update anymore: https://github.com/glebdmitriew/node-unzip-2 - this works.

@frozeman
Copy link
Contributor

frozeman commented Nov 21, 2016

Why closing this? This PR is for updating the client binaries URLs.

@frozeman frozeman reopened this Nov 21, 2016
@evertonfraga
Copy link
Member

@frozeman this is automatic behavior from github. Ram merged the issue on ethereum-client-binaries, then GH thinks it has to be closed here.

@frozeman frozeman merged commit 26b3bcc into develop Nov 21, 2016
@luclu luclu deleted the luclu_update-geth-1.5.2 branch November 22, 2016 16:27
@evertonfraga evertonfraga added this to the 0.8.8 milestone Dec 5, 2016
@evertonfraga evertonfraga added this to the 0.8.8 milestone Dec 5, 2016
@lock
Copy link

lock bot commented Mar 31, 2018

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.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants