Skip to content
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

haskellPackages.{ffmpeg-light,opencv{,-extra}}: remove overrides #336110

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

emilazy
Copy link
Member

@emilazy emilazy commented Aug 20, 2024

Description of changes

These are dead code due to the packages being marked broken, and they depend on libraries (FFmpeg 4 and OpenCV 3) we’re trying to remove from Nixpkgs. Sending this PR to haskell-updates accordingly, as requested in the Haskell room on Matrix.

cc @nh2 – the second commit here will need reverting and adjusting if you land LumiGuide/haskell-opencv#157 to get this fixed (as well cabal2nix, of course).

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

This package has been marked as broken for a year. If anyone wants to
fix it, there is a pull request updating it to the current FFmpeg API:
<acowley/ffmpeg-light#72>.
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Aug 20, 2024
This was referenced Aug 20, 2024
@maralorn
Copy link
Member

maralorn commented Aug 22, 2024

Sorry, I am a bit uncertain about this. Because I don’t understand the whole situation.

Are these Haskell packages basically obsolete because they rely on a systems library which is deprecated and gone for good?

Or is it to be expected that these libraries will become usable again at which point we need to re-add these overrides?

Well, looking at the overrides probably none of them will stay as is so I assume we can drop them.
Anyway can you please also remove the referenced patch so that we don’t have dangling files?

@emilazy
Copy link
Member Author

emilazy commented Aug 22, 2024

pkgs.opencv{2,3} are gone for good already. pkgs.ffmpeg_4 still exists for now, but we’re trying to get rid of it entirely; see #336401 (we’re mostly just blocked on Spotify, REAPER, and ROCm).

Both Haskell packages are currently broken in Nixpkgs anyway; opencv-extra because of opencv. Both opencv and ffmpeg-light have open pull requests migrating them to newer versions of OpenCV and FFmpeg. Those PRs have positive upstream feedback and also fix the build with newer GHCs, so I presume that if these packages ever begin to work again, they would need different dependency overrides, anyway. That would require patching cabal2nix in the case of opencv.

These packages have been broken since before the 24.05 release. There
is a pull request open to update them to a newer OpenCV and get them
working again: <LumiGuide/haskell-opencv#157>.
@emilazy emilazy force-pushed the push-vlmoxqvopuvv branch from c427a65 to 79872e0 Compare August 22, 2024 14:06
@emilazy
Copy link
Member Author

emilazy commented Aug 22, 2024

I’ve removed the patch. FWIW I’d also be happy to just comment out the parts that might want restoring at a later date with different versions. (But cabal2nix would need patching anyway, and I’ve pinged the Nixpkgs‐committer author of the opencv PR here.)

@maralorn
Copy link
Member

I guess the fix will be straightforward enough and there will be enough expertise around, that we will be able to add the correct overrides once they are useful.

Commenting out sounds good, but honestly, I’d expect that the next person dealing will this will actually miss the comments and then we have an override + an outdated comment. 😆

Thanks for bearing with me.

@maralorn maralorn merged commit 7331129 into NixOS:haskell-updates Aug 22, 2024
21 checks passed
@emilazy emilazy deleted the push-vlmoxqvopuvv branch August 26, 2024 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: haskell 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants