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
librepcb: 0.1.4 -> 0.1.5 #99659
librepcb: 0.1.4 -> 0.1.5 #99659
Conversation
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.
The project uses tags, so we can leverage this by fetching the tag instead of the commit hash. See suggestion below.
|
||
src = fetchFromGitHub { | ||
owner = "LibrePCB"; | ||
repo = "LibrePCB"; | ||
fetchSubmodules = true; | ||
rev = "ae04eef5a71b5ba66ae2cee6b631c1c933ace535"; | ||
sha256 = "0wk5qny1jb6n4mwyyrs7syir3hmwxlwazcd80bpxharmsj7p0rzc"; | ||
rev = "a435c5095c6f94df4da130fe832ea97f42d17901"; |
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.
rev = "a435c5095c6f94df4da130fe832ea97f42d17901"; | |
rev = version; |
Please squash the second commit into the first one |
9544c60 / NixOS#96672 removed the samba option `syncPasswordsByPam`. Need to remove this option from the pam module, otherwise it will cause build errors
Otherwise, it's possible building a flake may cause the live CD to run out of memory.
This reverts commit fa6064a.
This fixes both `nix-shell` failing to eval with `nixUnstable`, as well as ofborg's failure to eval on aarch64 due to passing an "unexpected arg" (https://github.com/NixOS/ofborg/blob/1112e3a8c8621203fae71283d700e0383b5e1e80/ofborg/src/nix.rs#L334-L340).
reopened a fresh one, hope thats ok... |
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)