Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

altcoins.mist: init at 0.10.0 #38661

Merged
merged 1 commit into from Apr 19, 2018

Conversation

jbboehr
Copy link
Contributor

@jbboehr jbboehr commented Apr 9, 2018

Motivation for this change

Having an available package for the Ethereum Mist browser.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

At first I attempted to package it using node2nix, however ran into some issues. Currently, it is using the binary releases, patching the ELF, but it still needs to run inside an FHS user env because the program attempts to download the geth binary to the home directory, which is linked to the standard /usr/lib paths. Specifying a geth binary is currently not working [1] [2].

I've tested it on my system running 18.03 with Gnome 3.

It should be possible to add a Mac build as well, but I do not have a Mac available to test and so did not.

I did not add myself as a maintainer because I'm not sure if I'll be able to maintain it going forward. Please let me know how to proceed, if I should add it anyway or if someone else is interested in adopting it.

@etu
Copy link
Contributor

etu commented Apr 10, 2018

@jbboehr Could you squash the fix of the hash together with the init commit?

@jbboehr jbboehr force-pushed the ethereum-mist-browser-master branch from 0b44892 to 7cb50de Compare April 10, 2018 19:09
@jbboehr
Copy link
Contributor Author

jbboehr commented Apr 10, 2018

@etu Done!

@matthewbauer matthewbauer merged commit 22639db into NixOS:master Apr 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants