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
colobot: init at 0.1.12-alpha #103077
colobot: init at 0.1.12-alpha #103077
Conversation
bbfe9f5
to
c9e32fa
Compare
pkgs/games/colobot/default.nix
Outdated
|
||
stdenv.mkDerivation rec { | ||
pname = "colobot"; | ||
version = "0.1.12-alpha"; |
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.
We should probably add a comment to also update the data derivation and vice versa.
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.
@freezeboy Ideally the comment saying updates to this derivation should go along with updates to ./data.nix
, should be here, right above the version, and it shouldn't have typos :)
Result of 2 packages marked as broken and skipped:
|
d62b4d5
to
14084c6
Compare
14084c6
to
5281fa5
Compare
Result of 1 package marked as broken and skipped:
|
Why is it broken ? Edit: Ok: physfs is broken |
5281fa5
to
b582bec
Compare
b582bec
to
1c7103c
Compare
I don't understand why ofborg is not happy, looks like an unrelated problem |
The ofborg error is not a problem with ofborg. There's a policy that within nixpkgs, all packages should be able to evaluate without using declarations from nixpkgs/pkgs/top-level/aliases.nix Line 683 in 7d75a77
So, use |
Oh ok, didn't know that, is there a way to check it locally as nix-build succeeds anyway ? |
1c7103c
to
fb31d13
Compare
apparently was caused by another package |
pkgs/games/colobot/default.nix
Outdated
# Mayb require an update tp package colobot-data as well | ||
# in file data.nix next to this one |
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.
So I'd remove these and rewrite it upwards.
# Mayb require an update tp package colobot-data as well | |
# in file data.nix next to this one |
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.
What do you mean "upwards"?
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.
What do you mean "upwards"?
Right above version = "...";
.
This one is easy:
|
908004f
to
98ad579
Compare
98ad579
to
05e9c4c
Compare
Initial integration without the game musics to reduce the size of the game. It works without them Data are in another derivation to let them live their live independently as they are quite big. Limited to linux platform only because physfs is currently broken on darwin.
05e9c4c
to
80bff15
Compare
Just rebased the commit, I think it should be ok |
Motivation for this change
Initial integration without the game musics to reduce the size of the
game. It works without them
Data are in another derivation to let them live their live independently
as they are quite big.
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)