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
Add CF Checker and its cfunits dependency #90606
base: master
Are you sure you want to change the base?
Conversation
I just spotted that my editor automatically removed a stray trailing space in |
Rebased on current master. At some point that stray space got cleaned up so it's no longer part of the changes here. |
I marked this as stale due to inactivity. → More info |
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.
lgtm
not super familiar with python & nixpkgs so my approval might not be worth much but since you labelled yourself as a maintainer I'm sure you can handle whatever problems arise 🤷
Motivation for this change
Add a package that I use and its dependency.
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)I'm not 100% sure if I've gone about fixing the path to
libudunits.so
in cfunits the right way, or whether I have categorised properly. CF conventions are used in atmospheric modelling, but I feel like cfchecker is not "large" enough to fit intoapplications/science
and is more of a "tool".Also, this currently doesn't build on master due to coveralls being broken there (see e.g. https://hydra.nixos.org/build/121765313)