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

wownero: init at 0.4.0.0 #49138

Merged
merged 2 commits into from Nov 11, 2018
Merged

wownero: init at 0.4.0.0 #49138

merged 2 commits into from Nov 11, 2018

Conversation

ghost
Copy link

@ghost ghost commented Oct 26, 2018

Motivation for this change

Add the cryptocurrency Wownero to nixpkgs.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

@ghost
Copy link
Author

ghost commented Oct 26, 2018

@jwinterm for review

@ghost ghost changed the title wownero: init at 0.3.0.0 wownero: init at 0.3.1.0 Oct 29, 2018
@ghost
Copy link
Author

ghost commented Oct 29, 2018

updated the PR to point to the latest release

@ghost
Copy link
Author

ghost commented Oct 29, 2018

updated again to point to the latest release

@ghost ghost changed the title wownero: init at 0.3.1.0 wownero: init at 0.3.1.1 Oct 29, 2018
@ghost ghost changed the title wownero: init at 0.3.1.1 wownero: init at 0.3.1.3 Nov 8, 2018
name = "wownero-${version}";

version = "0.3.1.3";
src = fetchgit {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use fetchFromGitHub instead of fetchgit

Copy link
Author

@ghost ghost Nov 9, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kalbasit, i needed the fetchSubmodules = true; part from fetchgit, is it supported in fetchFromGithub now?

Copy link
Member

@kalbasit kalbasit Nov 9, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it's supported, see the source for fetchFromGitHub here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! updated.


Unlike opposing forks.
'';
homepage = https://wownero.org/;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They don't respond to https, http://wownero.org/ works though.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, reported in the IRC, but it should be fixed soon.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

url updated to use plain http.

@ghost ghost changed the title wownero: init at 0.3.1.3 wownero: init at 0.4.0.0 Nov 9, 2018

It’s not a fork of another blockchain. With its own genesis block there is no degradation of privacy caused by ring signatures using different participants for the same transaction outputs.

Unlike opposing forks.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix the formatting here? Only needs newlines to keep the length under 80 columns, no other newlines needed.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the information, fixed.

hardeningDisable = [ "fortify" ];

meta = {
description = "Many coins, such privacy. Wow!";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a very descriptive description.. Can you put something better here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jwinterm any suggestions?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the description to use the first line of the long description, which happens to be the summary of it.

email = "echowss@gmail.com";
github = "fuwa0529";
name = "Haruka Akiyama";
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put this change into a separate commit in this PR.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@infinisil
Copy link
Member

@GrahamcOfBorg build altcoins.wownero

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: altcoins.wownero

Partial log (click to expand)

shrinking /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin/wownero-blockchain-import
shrinking /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin/wownerod
shrinking /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin/wownero-gen-trusted-multisig
shrinking /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin/wownero-wallet-cli
shrinking /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin/wownero-wallet-rpc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/lib  /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0/bin
patching script interpreter paths in /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0
checking for references to /build in /nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0...
/nix/store/520p3122v2gwxcvl0vapmdqq5g0r6z6h-wownero-0.4.0.0

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: altcoins.wownero

Partial log (click to expand)

shrinking /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin/wownero-blockchain-import
shrinking /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin/wownerod
shrinking /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin/wownero-blockchain-export
shrinking /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin/wownero-wallet-cli
shrinking /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin/wownero-blockchain-blackball
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/lib  /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0/bin
patching script interpreter paths in /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0
checking for references to /build in /nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0...
/nix/store/2qw0bls21g1j6v6lsm9lwag1pv6pi4yn-wownero-0.4.0.0

@GrahamcOfBorg
Copy link

Success on x86_64-darwin (full log)

Attempted: altcoins.wownero

Partial log (click to expand)

-- Installing: /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin/wownerod
-- Installing: /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin/wownero-blockchain-import
-- Installing: /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin/wownero-blockchain-export
-- Installing: /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin/wownero-blockchain-blackball
-- Installing: /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin/wownero-blockchain-usage
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/lib  /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0/bin
patching script interpreter paths in /nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0
/nix/store/mjq3xrbxfvhjl0cn8269952n87ps683f-wownero-0.4.0.0

@infinisil infinisil merged commit e2bae13 into NixOS:master Nov 11, 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

3 participants