-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
opencv: rename and default to opencv4 #86808
Conversation
@GrahamcOfBorg build frei0r |
@GrahamcOfBorg build handbrake |
@GrahamcOfBorg build tambura shotcut soundkonverter ffmpeg-full |
@GrahamcOfBorg build ffmpeg-full |
@GrahamcOfBorg build retroshare |
@GrahamcOfBorg build shotcut |
@GrahamcOfBorg build soundkonverter |
@GrahamcOfBorg build synfigstudio |
@GrahamcOfBorg build saga |
@GrahamcOfBorg build kdenlive |
@GrahamcOfBorg build manim |
@GrahamcOfBorg build mlt |
@GrahamcOfBorg build hydron |
@GrahamcOfBorg build meguca |
@GrahamcOfBorg build libsForQt5.openbr |
@GrahamcOfBorg build liquidsoap |
@GrahamcOfBorg build printrun |
@GrahamcOfBorg build pfstools |
@GrahamcOfBorg build openmvs |
@GrahamcOfBorg build p2pvc |
@GrahamcOfBorg build olive-editor |
@GrahamcOfBorg build python2Packages.gym python37Packages.gym python38Packages.gym @GrahamcOfBorg build python2Packages.pydub python37Packages.pydub python38Packages.pydub @GrahamcOfBorg build python2Packages.pyglet python37Packages.pyglet python38Packages.pyglet |
@GrahamcOfBorg build python2Packages.pytmx python37Packages.pytmx python38Packages.pytmx @GrahamcOfBorg build python2Packages.remotecv python37Packages.remotecv python38Packages.remotecv @GrahamcOfBorg build python37Packages.baselines @GrahamcOfBorg build python37Packages.rl-coach python38Packages.rl-coach @GrahamcOfBorg build python37Packages.roboschool python38Packages.roboschool |
@GrahamcOfBorg build python37Packages.moderngl-window |
To make the checks visible after all the fixes have been commited, I'm executing all of the builds here once more: @GrahamcOfBorg build frei0r |
@Mic92 I made All of the checks are available here: https://github.com/NixOS/nixpkgs/pull/86808/checks |
Thanks a lot for your work! |
Thanks for this! |
Motivation for this change
Make opencv4 the default and make sure all packages that depend on it built with it. The general goal is to reduce closure size for users of packages which can use the latest opencv. See https://discourse.nixos.org/t/nixpkgs-policy-regarding-libraries-available-in-multiple-versions/7026/2
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
(used ofborg)opencv2
explicitly now)opencv2
explicitly now)opencv2
explicitly now (I can't test it since it's unfree))opencv2
explicitly now)opencv2
explicitly now)opencv2
explicitly now)./result/bin/
)nix path-info -S
before and after)