Your message dated Sat, 23 Aug 2014 12:29:40 +0200
with message-id <53f86d14.60...@googlemail.com>
and subject line pkg-config patch no longer necessary, better solution found
has caused the Debian Bug report #758347,
regarding kid3: Use pkg-config to determine FFmpeg linker flags
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
758347: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=758347
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Source: kid3
Version: 3.1-1
Severity: wishlist
Tags: patch
Usertags: reintroducing-ffmpeg

Dear maintainer,

I am working on reintroducing FFmpeg to Debian [1-2].

In order to make FFmpeg co-installable with Libav, the libraries were
renamed to lib*-ffmpeg. Thus using linker flags like '-lavcodec' doesn't
work with the FFmpeg packages.

To get the correct linking flags for both FFmpeg and Libav, one can
(and should) use pkg-config.

Attached patch achieves that for this package. Please apply it to
facilitate building your package with FFmpeg in Debian.

If you want to facilitate this even further, you can also add
lib*-ffmpeg-dev alternatives to the Libav build-dependencies.

While the FFmpeg package is still waiting in the NEW queue [3], it can
already be built from the git repository [4].

Best regards,
Andreas


1: https://lists.debian.org/debian-devel/2014/07/msg01010.html
2: https://bugs.debian.org/729203
3: https://ftp-master.debian.org/new/ffmpeg_7:2.3.1-1.html
4: https://anonscm.debian.org/cgit/collab-maint/ffmpeg.git

diff --git a/debian/patches/pkg-config.patch b/debian/patches/pkg-config.patch
new file mode 100644
index 0000000..9e8315c
--- /dev/null
+++ b/debian/patches/pkg-config.patch
@@ -0,0 +1,38 @@
+Description: Use pkg-config to determine FFmpeg linker flags
+
+Author: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
+Last-Update: <2014-05-18>
+
+--- kid3-3.1.orig/src/plugins/acoustidimport/cmake/modules/FindFFmpeg.cmake
++++ kid3-3.1/src/plugins/acoustidimport/cmake/modules/FindFFmpeg.cmake
+@@ -47,29 +47,7 @@ MACRO(FFMPEG_FIND varname shortname head
+ 		DOC "Location of FFMPEG Headers"
+ 	)
+ 
+-    FIND_LIBRARY(FFMPEG_${varname}_LIBRARIES
+-        NAMES ${shortname}
+-        PATHS
+-        ${FFMPEG_ROOT}/lib
+-        $ENV{FFMPEG_DIR}/lib
+-        ~/Library/Frameworks
+-        /Library/Frameworks
+-        /usr/local/lib
+-        /usr/local/lib64
+-        /usr/lib
+-        /usr/lib64
+-        /sw/lib
+-        /opt/local/lib
+-        /opt/csw/lib
+-        /opt/lib
+-        /usr/freeware/lib64
+-		NO_DEFAULT_PATH
+-        DOC "Location of FFMPEG Libraries"
+-    )
+-    FIND_LIBRARY(FFMPEG_${varname}_LIBRARIES
+-        NAMES ${shortname}
+-        DOC "Location of FFMPEG Libraries"
+-    )
++	pkg_check_modules(FFMPEG_${varname} lib${shortname})
+ 
+     IF (FFMPEG_${varname}_LIBRARIES AND FFMPEG_${varname}_INCLUDE_DIRS)
+         SET(FFMPEG_${varname}_FOUND 1)
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..9937964
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1 @@
+pkg-config.patch


--- End Message ---
--- Begin Message ---
Dear maintainer,

I have found a simpler way to make it possible to link your package against FFmpeg in Debian: The lib*-ffmpeg-dev packages now install symbolic links from the standard lib*.so library names to the suffixed ones. This makes it possible to use the normal linker flags, e.g. '-lavcodec', to link against the FFmpeg libraries with '-ffmpeg' suffix.

Thus the pkg-config patch is no longer necessary.

I wish I had found this solution sooner and am sorry for the noise I created in your inbox.

Best regards,
Andreas

--- End Message ---
_______________________________________________
pkg-kde-extras mailing list
pkg-kde-extras@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-extras

Reply via email to