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
geant4 overhaul #46019
geant4 overhaul #46019
Conversation
Nobody maintains this and this does not contribute anything towards making reproducible research. Also this was missing recurseIntoAttrs, so Hydra used to not see it.
Success on aarch64-linux (full log) Attempted: g4py, geant4 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: g4py, geant4 Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: g4py, geant4 Partial log (click to expand)
|
@xeji I think this is a good PR and at very least it doesn't break evaluation. Could you please merge this? |
Changes LGTM. But I would limit to What's the difference between this and your previous #40618 ? |
@xeji No problem, fixed. There is no difference with previous PR. |
Just to get it back on page 1, I see 😉 |
4 months... |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: g4py, geant4 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: g4py, geant4 Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: g4py, geant4 Partial log (click to expand)
|
Let's ignore the timeout, the build succeeded the first time. |
@xeji Thank you so much! |
Motivation for this change
Update geant4 expression to conform to usual practices used in nixpkgs.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)