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

bitcoin-unlimited: init at 1.0.1.3 #24378

Conversation

DmitryTsygankov
Copy link
Contributor

Add the Bitcoin Unlimited client to the altcoins directory, so that all sides of the Bitcoin scaling debate are represented.

Motivation for this change

There is a scaling debate going on in Bitcoin. Bitcoin Core and Bitcoin Unlimited are the two most popular reference clients representing the two sides of the debate.
The currently less popular bitcoin-classic and bitcoin-xt are already in the package repository, it's time to add bitcoin-unlimited.
The bitcoin-unlimited package code is mostly a copy of bitcoin-classic, with appropriate modifications for the source repository, and version tags.
Was able to build and install the package on a NixOS machine, then run the GUI client & download the first few blocks of the blockchain.

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
    • Linux
  • 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.

@mention-bot
Copy link

@DmitryTsygankov, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zimbatm, @zraexy and @AndersonTorres to be potential reviewers.

@rycee
Copy link
Member

rycee commented Mar 28, 2017

Looks good to me. The first line of the commit should be "bitcoin-unlimited: init at 1.0.1.3", though.

@DmitryTsygankov DmitryTsygankov force-pushed the DmitryTsygankov/bitcoin-unlimited branch from fc3583b to 73956eb Compare March 29, 2017 06:08
Add the Bitcoin Unlimited client to the altcoins directory, so that all sides of the Bitcoin scaling debate are represented.
@DmitryTsygankov DmitryTsygankov force-pushed the DmitryTsygankov/bitcoin-unlimited branch from 73956eb to 02b0ea4 Compare March 29, 2017 06:12
@DmitryTsygankov DmitryTsygankov changed the title Add the bitcoin-unlimited package bitcoin-unlimited: init at 1.0.1.3 Mar 29, 2017
@DmitryTsygankov
Copy link
Contributor Author

@rycee fixed, thanks

@rycee
Copy link
Member

rycee commented Apr 1, 2017

Sorry for the delay. Compiles and starts up OK for me. Rebased into master in 57a813f. Many thanks for your contribution!

@rycee rycee closed this Apr 1, 2017
@DmitryTsygankov DmitryTsygankov deleted the DmitryTsygankov/bitcoin-unlimited branch April 26, 2017 05:01
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