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
pythia: 8.244 -> 8.303 #99993
pythia: 8.244 -> 8.303 #99993
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.
- Diff LGTM minus comment on hepmc version locking.
- Commits LGTM
- Builds via
nix-review
onx86_64-linux
:
https://github.com/NixOS/nixpkgs/pull/99993
2 packages built:
pythia sacrifice
@@ -26755,7 +26755,9 @@ in | |||
|
|||
nlojet = callPackage ../development/libraries/physics/nlojet { }; | |||
|
|||
pythia = callPackage ../development/libraries/physics/pythia { }; | |||
pythia = callPackage ../development/libraries/physics/pythia { | |||
hepmc = hepmc2; |
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.
Why do you lock hepmc = hepmc2
if pythia
seems to support hepmc3
(i.e. has configure flags for hepmc3)?
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.
Have not had time to test hepmc3
yet. I am using the sacrifice
interface which only works with hepmc2
. There is no real rush to switch now.
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.
Maybe add a comment to that effect about why the version is restricted
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.
It's not restricted, just that the default is unchanged. pythia.override { hepmc=hepmc3; }
will compile fine.
Motivation for this change
Bump pythia version to the latest version
Closes: #90120
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)