diff mbox series

[PATCH/next,RFC,1/1] package/opencv3: disable ffmpeg support

Message ID 20250622105922.2981702-1-bernd@kuhls.net
State Superseded
Headers show
Series [PATCH/next,RFC,1/1] package/opencv3: disable ffmpeg support | expand

Commit Message

Bernd Kuhls June 22, 2025, 10:59 a.m. UTC
OpenCV3 is incompatible with the upcoming bump of ffmpeg to 7.1.x.

According to the OpenCV3 changelog no new release will occur for the 3.x
branch:
https://github.com/opencv/opencv/wiki/OpenCV-Change-Logs-v2.2%E2%80%90v4.10#version3420
"OpenCV 3.4.20 is last official release in 3.4 branch."

Therefore ffmpeg support is removed so an unmaintained package does not
block the version bump of ffmpeg.

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
I sent this patch as RFC for the 2025.11 release because I do not think
the ffmpeg 7.x bump, needed[1] for Kodi 22, will occur in this release
cycle due to Kodi 22 most likely not being released during this period.
But already now it is clear to me that OpenCV3 will lose its ffmpeg
support, see the patch comment above.

I would like to suggest however that we remove OpenCV3 completely from
buildroot and replace it, where possible, with OpenCV4.
In preparation for this switch I sent patches for packages already
having working OpenCV4 support.

Beyond that I identified packages with optional dependencies to OpenCV3
which are incompatible with OpenCV4:
- gtkiostream
- mjpg-streamer: https://github.com/jacksonliam/mjpg-streamer/commit/216c090707ffd71fdf86b0c5b182954846e01bd4
- vlc: https://code.videolan.org/videolan/vlc/-/merge_requests/927

Which way do you prefer?
Remove ffmpeg support from OpenCV3 and keep the package or remove
the OpenCV3 package?

[1] https://github.com/xbmc/xbmc/commit/72fe098c8436c96763f677b4c65d32988b931b5b

 package/opencv3/Config.in  | 9 ---------
 package/opencv3/opencv3.mk | 8 +-------
 2 files changed, 1 insertion(+), 16 deletions(-)
diff mbox series

Patch

diff --git a/package/opencv3/Config.in b/package/opencv3/Config.in
index a672497108..097201887f 100644
--- a/package/opencv3/Config.in
+++ b/package/opencv3/Config.in
@@ -241,15 +241,6 @@  config BR2_PACKAGE_OPENCV3_BUILD_PERF_TESTS
 
 comment "3rd party support"
 
-config BR2_PACKAGE_OPENCV3_WITH_FFMPEG
-	bool "ffmpeg support"
-	depends on BR2_PACKAGE_FFMPEG_ARCH_SUPPORTS
-	select BR2_PACKAGE_BZIP2
-	select BR2_PACKAGE_FFMPEG
-	select BR2_PACKAGE_FFMPEG_SWSCALE
-	help
-	  Use ffmpeg from the target system.
-
 config BR2_PACKAGE_OPENCV3_WITH_GSTREAMER1
 	bool "gstreamer-1.x"
 	depends on BR2_USE_MMU # gstreamer1 -> libglib2
diff --git a/package/opencv3/opencv3.mk b/package/opencv3/opencv3.mk
index 218174c17e..a66c7a011e 100644
--- a/package/opencv3/opencv3.mk
+++ b/package/opencv3/opencv3.mk
@@ -226,17 +226,11 @@  OPENCV3_CONF_OPTS += \
 	-DWITH_VA=OFF \
 	-DWITH_VA_INTEL=OFF \
 	-DWITH_VTK=OFF \
+	-DWITH_FFMPEG=OFF \
 	-DWITH_XINE=OFF
 
 OPENCV3_DEPENDENCIES += host-pkgconf zlib
 
-ifeq ($(BR2_PACKAGE_OPENCV3_WITH_FFMPEG),y)
-OPENCV3_CONF_OPTS += -DWITH_FFMPEG=ON
-OPENCV3_DEPENDENCIES += ffmpeg bzip2
-else
-OPENCV3_CONF_OPTS += -DWITH_FFMPEG=OFF
-endif
-
 ifeq ($(BR2_PACKAGE_OPENCV3_WITH_GSTREAMER1),y)
 OPENCV3_CONF_OPTS += -DWITH_GSTREAMER=ON
 OPENCV3_DEPENDENCIES += gstreamer1 gst1-plugins-base