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
refactor(nix): synchronize changes from nixpkgs #492
Conversation
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
CC @Minion3665 |
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.
Looks good to me and builds & runs fine
why was ninja removed? |
It doesn't matter much anyway. It wasn't used in nixpkgs, so I removed it here as well |
I mean it makes building considerably faster though |
is it? IIRC nix already sets |
ninja still makes it faster, it's not just make with -jX by default |
We should likely have parity between here and nixpkgs, we could consider adding ninja to nixpkgs instead of removing it from here if that is preferred |
Looking at this benchmark it doesn't seem to matter much for a project the size of Prism Launcher (~1600 .cpp/.h/.ui files) |
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 guess for now it's fine
Updates the nix derivation to mostly match the one in nixpkgs.
See https://github.com/NixOS/nixpkgs/blob/master/pkgs/games/prismlauncher/default.nix