python3Packages.boxx: fix build#382056
Conversation
- fn dependency is no longer required - run tests only on Linux because xvfb is not available on other platforms
|
I am curious, are you actually using As you've probably noticed, these packages aren't currently in a good place and AFAIK, neither upstream nor the current nixpkgs maintainer @lucasew are interested in supporting them. I was originally planning to fix the build and tests in #380403 just like you did (and I did "kind of" fix the tests for boxx), but Following the discussion with the current nixpkgs maintainer in #379872, we decided that we might as well just remove these packages (#380443). I think that unless you are actively using Thoughts? |
I am not using them. I guess we can remove them if nobody use them. |
|
what exactly is interesting? this pr fixed the build of boxx and also bpycv because boxx is its dependency |
|
AFAIK, the build for |
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.