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

fix eth (c++) node #834

Closed
luclu opened this issue Jun 16, 2016 · 18 comments
Closed

fix eth (c++) node #834

luclu opened this issue Jun 16, 2016 · 18 comments
Milestone

Comments

@luclu
Copy link
Contributor

luclu commented Jun 16, 2016

Current issues: #607
Ongoing PR: #841

@luclu
Copy link
Contributor Author

luclu commented Jul 9, 2016

Update script still outstanding.
Pinging @hiddentao

@hiddentao
Copy link
Contributor

hiddentao commented Jul 11, 2016

Yeah, we're waiting on the Eth team to do something about people who have previously set master passwords.

@luclu luclu modified the milestones: 0.8, 0.8.2 Jul 11, 2016
@bobsummerwill
Copy link
Contributor

So my repo-reorg work is coming within touching distance of going live.

You can preview it at https://github.com/bobsummerwill/cpp-ethereum/tree/merge_repos.

And example ZIPs, if you want to play around ... https://github.com/bobsummerwill/cpp-ethereum/releases/tag/develop-v1.2.9.282

The new Appveyor/TravisCI automation includes the fix for the dylib paths, so you won't need to do any post-processing anymore - just drop the content into your folder.

We have merged a fix related to master password which I think is the same as you are referring to above (@chriseth?) - ethereum/webthree#187. Or is the issue referred to above something different?

And the toggle will need re-enabling too, eh?

There is a chance we will do a cpp-ethereum-1.3.0 release tomorrow, or if that doesn't happen then certainly on Tuesday. We'll be back in https://github.com/ethereum/cpp-ethereum, and the webthree-umbrella era will be over.

@hiddentao
Copy link
Contributor

@bobsummerwill ethereum/webthree#187 looks like what we need - I'll comment in there for clarification.

@luclu
Copy link
Contributor Author

luclu commented Jul 23, 2016

Version 1.3.0 bob's repo should work as the following issues have been resolved:

  • HF compatibility
  • old master-passwords (created before version 1.2.9) will stall the communication to the node
  • eth_sendTransaction was depreciated and removed due to an vulnerability and replaced by eth_unlockAccountAndSendTransaction that is not implemented in 1.2.9 - which prevents using the send function

Version 1.3.1 should be released in the following days.
We might want to update gulp and externalize the eth nodes from the repo before that to prevent further growing:

download eth-nodes with gulp / remove from repo #894

@bobsummerwill
Copy link
Contributor

Yeah - on-demand download would make sense to avoid repo-bloat.

Aside - you can ditch https://github.com/ethereum/mist/blob/develop/nodes/eth/fix_darwin_paths.py too with v1.3.0.

I fixed that missing automation step during the Jenkins -> TravisCI migration.

It will literally be ZIPs from Github to grab-and-unzip.

@bobsummerwill
Copy link
Contributor

Here for now:
https://github.com/bobsummerwill/cpp-ethereum/releases/tag/v1.3.0

Moving very soon to:
https://github.com/ethereum/cpp-ethereum/releases/tag/v1.3.1

I've also added OS X Mavericks support back into our matrix (because only Mavericks binaries run successfully on the 2009 era Mac Pro hardware which TravisCI is running on). No doubt if I monkeyed around with -march settings I could build a single set of binaries which worked for all OS versions ON El Capitan, but I've done bothered with that yet.

So we have:

  • Windows x64 (min-spec we're saying Windows 7, but I think that might actually work back to Vista)
  • Ubuntu Trusty x64 (but the ZIP isn't fully standalone, though I hope to get there with static linkage binaries on Alpine using musl)
  • OS X Mavericks, Yosemite and El Capitan (all x64)

We're not building any 32-bit binaries right now, either for Windows or Linux, but the build works from source, and I would like to re-add those soon enough.

@luclu
Copy link
Contributor Author

luclu commented Jul 24, 2016

Thank you Bob for your efforts!

No doubt if I monkeyed around with -march settings I could build a single set of binaries which worked for all OS versions ON El Capitan, but I've done bothered with that yet.

Does this mean that currently there is no single OS X binary that could work on 10.9 to 10.11?
Currently our release model as well as the build scripts are one binary per platform. Do you know which binary was used in earlier versions?

@luclu luclu mentioned this issue Jul 25, 2016
@hiddentao
Copy link
Contributor

Yep, would be nice if we could have one binary for all OS X versions, allowing us to continue releasing a single Mist/Wallet build for OS X.

@frozeman
Copy link
Contributor

@bobsummerwill are they fixed on mac now automatically, or do we have to run the python script still?

The real link would be https://github.com/ethereum/webthree-umbrella/releases, but there is no 1.3.1 yet @bobsummerwill

@luclu
Copy link
Contributor Author

luclu commented Jul 25, 2016

@frozeman

Aside - you can ditch https://github.com/ethereum/mist/blob/develop/nodes/eth/fix_darwin_paths.py too with v1.3.0.

They want to release in in the following days - but essentially it is the same as bob's 1.3.0.
BUT we should seriously consider to adjust gulpfile.js to download those nodes as we do with geth.

@frozeman
Copy link
Contributor

frozeman commented Jul 25, 2016

Sure, but i also only see 1.2.9 https://github.com/ethereum/webthree-umbrella/releases

@luclu please test if you can run the binary on a mac out of the box
@bobsummerwill does this release contain the removed master password?

@luclu
Copy link
Contributor Author

luclu commented Jul 25, 2016

@luclu please test if you can run the binary on a mac out of the box

I get some libcryptopp.cpp missing error..

@luclu luclu added the ready label Jul 25, 2016
@bobsummerwill
Copy link
Contributor

You could probably just use https://github.com/bobsummerwill/cpp-ethereum/releases/download/v1.3.0/cpp-ethereum-osx-mavericks-v1.3.0.zip as a universal binary.

Yeah, @frozeman, we're mid-transition.

Please could you give me more infor on libcryptocpp.cpp issue, @luclu?

@luclu
Copy link
Contributor Author

luclu commented Jul 26, 2016

You could probably just use https://github.com/bobsummerwill/cpp-ethereum/releases/download/v1.3.0/cpp-ethereum-osx-mavericks-v1.3.0.zip as a universal binary.

This works just fine on El Capitan!

Please could you give me more infor on libcryptocpp.cpp issue, @luclu?

Ah this was stupid of me, I found a weird way to copy those files but mist some.. :P


I will stress test it now.

@bobsummerwill @alexvandesande
Could it be that those RPC requests are blocking?

bildschirmfoto 2016-07-26 um 12 00 30

Even the external ethconsole will complain:

Error: You tried to send "admin_peers" synchronously. Synchronous requests are not supported by the IPC provider.

@frozeman
Copy link
Contributor

No, the synchronous has nothing to do with blocking.
You need to do web3.admin.getPeers(console.log) to make the call, a sync call is not possible, as it uses pic sockets under the hood. Geth is simulating the synchronous way in their console, as they wrote their own pic provider

@alexvandesande
Copy link

CPP node updated

@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.
Projects
None yet
Development

No branches or pull requests

5 participants