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
rasm: init at 0.117 #73518
rasm: init at 0.117 #73518
Conversation
|
||
src = fetchurl { | ||
name = "rasm_v0116_src.zip"; | ||
url = "http://www.cpcwiki.eu/forum/programming/rasm-z80-assembler-in-beta/?action=dlattach;attach=30015"; |
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.
is there a more official url, other than some download links on a forum? Would be nice if he/she had a github/gitlab, especially seeing as it's mit license anyway.
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.
it's the official url according http://www.roudoudou.com/rasm/ . If i've time i would contact author but you know, most of time, such people doesn't work with git and has no opensource culture. BTW I'll try later since it's a compatriot.
I'd contacted the author, btw he changed src file without let old or change url so this could not work. |
src.url fixed with the help of the author. |
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.
diff LGTM
commit LGTM
[3 built, 53 copied (233.6 MiB), 55.9 MiB DL]
https://github.com/NixOS/nixpkgs/pull/73518
1 package were built:
rasm
@GrahamcOfBorg build rasm |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @