On Sat, May 08, 2010 at 09:35:19 (CEST), ceros-gu...@users.alioth.debian.org 
wrote:

> The following commit has been merged in the master branch:
> commit 81c299a45ae49fe1acd96907c81b2b66af2e922f
> Author: Andres Mejia <mcita...@gmail.com>
> Date:   Sat May 8 03:34:09 2010 -0400
>
>     Fix dependency problem for ffmpeg binary package when using extra
>     ffmpeg libs.

I don't understand this patch. What is the issue here? Currently the
ffmpeg package has dependency like these:

libavcodec52 (>= 4:0.5.1-3) | libavcodec-extra-52 (>= 4:0.5.1-3), libavcodec52 
(<< 4:0.5.1-99) | libavcodec-extra-52 (<< 4:0.5.1-99)

If I understand this patch correctly, the ffmpeg package cannot be
installed with the binary package from ffmpeg-extra.

Can you please elaborate on this change?

>     This will still leave problem with ffmpeg-dbg, but the debug package will 
> be
>     useless anyway when using extra ffmpeg libs.
>
> diff --git a/debian/changelog b/debian/changelog
> index 8f6eee5..60e1d73 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,3 +1,9 @@
> +ffmpeg (4:0.5.1-4) UNRELEASED; urgency=low
> +
> +  * Fix dependency problem for ffmpeg so it can use extra ffmpeg libs.
> +
> + -- Andres Mejia <mcita...@gmail.com>  Sat, 08 May 2010 03:19:37 -0400
> +
>  ffmpeg (4:0.5.1-3) unstable; urgency=low
>  
>    * fix ftbfs on powerpc

seperate commits please

> diff --git a/debian/rules b/debian/rules
> index b6d2051..ca852f8 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -126,12 +126,13 @@ binary-arch: build install formats.txt
>           dh_makeshlibs -p"$$pkg" -V"$$pkg (>= $(DEB_VERSION)) | $$upkg (>= 
> $(DEB_VERSION)), $$pkg (<< $(EPOCH)$(UPSTREAM_VERSION)-99) | $$upkg (<< 
> $(EPOCH)$(UPSTREAM_VERSION)-99)"; \
>       done
>       env LD_LIBRARY_PATH="$(LD_LIBRARY_PATH):$(CURDIR)/debian/tmp/usr/lib" \
> -     dh_shlibdeps
> +     dh_shlibdeps -Nffmpeg
>  # target dependencies for external packages
>       for pkg in $(LIB_PKGS); do \
>           upkg=$$(echo "$$pkg" | sed -r 's/([0-9]+)$$/-extra-\1/'); \
>           dh_makeshlibs -p"$$pkg" -V"$$pkg (>= $(SHLIBS_VERSION)) | $$upkg 
> (>= $(SHLIBS_VERSION))"; \
>       done
> +     dh_shlibdeps -pffmpeg

I guess the same could have been accomplished by putting 'ffmpeg' in the
for loop like this:

for pkg in ffmpeg $(LIB_PKGS); do \
...

But as stated above, I think this is wrong. Or I misunderstood something.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to