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
vassal: 3.3.2 -> 3.14.12 #107897
vassal: 3.3.2 -> 3.14.12 #107897
Conversation
pkgs/games/vassal/default.nix
Outdated
@@ -1,19 +1,19 @@ | |||
{ stdenv, fetchurl, jre, makeWrapper }: | |||
|
|||
stdenv.mkDerivation rec { | |||
name = "VASSAL-3.3.2"; | |||
name = "VASSAL-3.4.12"; |
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.
name = "VASSAL-3.4.12"; | |
pname = "VASSAL"; | |
version = "3.4.12"; |
pkgs/games/vassal/default.nix
Outdated
|
||
src = fetchurl { | ||
url = "mirror://sourceforge/vassalengine/${name}-linux.tar.bz2"; | ||
sha256 = "1abhlkl27gyfa1lghvv76xa6ks5hiwv2s9wb9ddadm0m07f87n1w"; | ||
url = "https://github.com/vassalengine/vassal/releases/download/3.4.12/VASSAL-3.4.12-linux.tar.bz2"; |
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 = "https://github.com/vassalengine/vassal/releases/download/3.4.12/VASSAL-3.4.12-linux.tar.bz2"; | |
url = "https://github.com/vassalengine/vassal/releases/download/${version}/VASSAL-${version}-linux.tar.bz2"; |
Result of 1 package built:
|
Result of 1 package built:
|
34d4042
to
5171ba7
Compare
pkgs/games/vassal/default.nix
Outdated
url = "mirror://sourceforge/vassalengine/${name}-linux.tar.bz2"; | ||
sha256 = "1abhlkl27gyfa1lghvv76xa6ks5hiwv2s9wb9ddadm0m07f87n1w"; | ||
src = fetchzip { | ||
url = "https://github.com/vassalengine/vassal/releases/download/${version}/${pname}-linux.tar.bz2"; |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
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 didn't see that it's using the prebuilt linux, so the above comment is not relevant.
However the ${pname}-linux.tar.bz2
should be ${name}-linux.tar.bz2
Also the whole thing doesn't builds for me.
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.
Uh, sorry. This worked for me because I had the source already in the store and the sha didn't change when I rewrote the URL to use variables.
I have deleted the source and the package from the store and rebuilt, now seems to work.
pkgs/games/vassal/default.nix
Outdated
name = "VASSAL-3.3.2"; | ||
pname = "VASSAL"; | ||
version = "3.4.12"; | ||
name = "${pname}-${version}"; |
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.
name = "${pname}-${version}"; |
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.
Doesn't the derivation need a name
argument?
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.
According to this: http://blog.ielliott.io/nix-docs/stdenv-mkDerivation.html
name is set to "${pname}-${version}"
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 didn't know that. Updated
5171ba7
to
3bf874f
Compare
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.
LGTM, builds and runs on my end.
Extra changes: - use variables to avoid repetition - update licence - use fetczip
3bf874f
to
63faa12
Compare
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)