supernovas: init at 1.5.0#437400
Conversation
cd7ef9c to
9dd92e4
Compare
|
Thanks for the review @iamanaws. Just pushed the changes. |
|
|
Aw Darwin not building. I'll take a stab at fixing it, otherwise I'll change the platform to just Linux. |
|
I've managed to rewrite the build system for this library in cmake, which will make it quite a bit easier to consume (not just for nix, but for other packagers). That'll take a bit to land, so if you don't mind checking out #438803 @iamanaws as that'll be an optional dep for this in the meantime, I'd appreciate it. |
|
Just to have it as reference, this pkg depends on either Sigmyne/SuperNOVAS#221 or Sigmyne/SuperNOVAS#222 |
|
I've tweaked @kiranshila's CMake configuration quite a bit, partly because it did not play well with Fedora and Ubuntu packaging requirements. For example, those Linux distros don't want to have and All of the above has been merged to the SuperNOVAS is currently packaged both as an RPM (Fedora 41+, EPEL10) and for Debian. Those two Linux packages try to provide the same package structure and same install locations, except for the bits where the two distros differ on requirements (e.g. in naming conventions or install locations). I think it would be a good idea if NixOS followed the same template too, as much as possible... There are some ways though that NixOS could deviate from the Debian and RPM packaging scheme:
I'm planning to obsolete the above subpackages in Debian and Fedora/EPEL also. Let me know if you need more info on building and packaging SuperNOVAS. |
04b40c3 to
1bafed1
Compare
|
Alright @iamanaws, this should be good to go now. I've incorporated the |
Update pkgs/by-name/su/supernovas/package.nix Co-authored-by: Angel J <78835633+Iamanaws@users.noreply.github.com>
1bafed1 to
b4aac7e
Compare
|
iamanaws
left a comment
There was a problem hiding this comment.
Approved automatically following the successful run of nixpkgs-review.
|
Excellent! Thank you! Now for the hard part of finding someone with merge access haha. |
|
Perhaps @Aleksanaa could merge? |
Initial packaging for SuperNOVAS astrometry library for C/C++. Future PRs will include dependencies like CALCEPH for ephemeris integration.
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.