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
wownero: init at 0.4.0.0 #49138
Conversation
@jwinterm for review |
updated the PR to point to the latest release |
updated again to point to the latest release |
name = "wownero-${version}"; | ||
|
||
version = "0.3.1.3"; | ||
src = fetchgit { |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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/; |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
|
||
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. |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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!"; |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jwinterm any suggestions?
There was a problem hiding this comment.
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"; | ||
}; |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@GrahamcOfBorg build altcoins.wownero |
Success on aarch64-linux (full log) Attempted: altcoins.wownero Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: altcoins.wownero Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: altcoins.wownero Partial log (click to expand)
|
Motivation for this change
Add the cryptocurrency Wownero to nixpkgs.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)