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
cfitsio: 3.430 -> 3.450 #61242
cfitsio: 3.430 -> 3.450 #61242
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.
Tested on Ubuntu.
Hello @xbreak and welcome around here :)
Yes, definitely! You can add yourself to the maintainer list if you wish to maintain this package. |
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.
Can you also clarify the license in meta? This is MIT so
license = licenses.mit;
and the comment about BSD-style licensing can be removed
The derivation is also updated to optionally build cfitsio with bzip2 support (enabled by default).
8b3e4db
to
6f07c3f
Compare
@c0bw3b: Thanks for the review. I've updated the PR with the requested changes. Apart from your comments I also changed the url to use https since HEASARC are phasing out unencrypted FTP access. |
Perfect. Thanks for your contribution. |
Forgot to mention: wrt security fixes, the grave public vulnerabilities (4 CVE published) were already fixed in v3.43 that we ship in release 19.03 Now 3.44 may have cleaned up some more codebase but it's undocumented what was fixed exactly and there is no CVE. So exploitability is very close to null. |
Motivation for this change
Changes from Review
This also includes a fixup of theNot needed if bzip2.out is used instead.cfitsio.pc
file to point to the store path of the bzip2 libraries.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)* I had to manually build some packages due to error cloning builder process (I'm running single user on CentOS without sandboxing).
** Tested to load fits files with giv
Built ok: