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
catch: add separate derivation for the catch2 branch #41488
Conversation
Why is it necessary to have both v1 and v2 in nixpkgs? /cc @edwtjo @knedlsepp |
I don't think there's very much value in keeping both of them. As far as I can tell only |
They're still tagging 1.x releases, so presumably there's some value in the distinction. I do not know anything about the package, though, so I'm happy to change the PR to replace the old derivation if that's the consensus. I'll let this sit for a day or two in case anyone else would share an opinion. |
I've rebased and changed the patch so it replaces the old |
@GrahamcOfBorg build catch cpp-gsl libjson-rpc-cpp |
Failure on x86_64-darwin (full log) Attempted: catch, cpp-gsl The following builds were skipped because they don't evaluate on x86_64-darwin: libjson-rpc-cpp Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: catch, cpp-gsl, libjson-rpc-cpp Partial log (click to expand)
|
This is a regression, catch1 builds on all three platforms. |
Success on x86_64-linux (full log) Attempted: catch, cpp-gsl, libjson-rpc-cpp Partial log (click to expand)
|
I'm afraid I'm not going to be able to put any more time into this. Perhaps keeping the original approach of keeping |
Motivation for this change
The
catch
package issues releases for both v1 and v2 branches.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)