[Buildroot] [PATCH v2 1/1] package/mplayer: Fix linking with libvorbis/libdvdread
Vicente Olivert Riera
Vincent.Riera at imgtec.com
Tue Nov 17 11:28:32 UTC 2015
Dear Bernd Kuhls,
On 11/08/2015 03:55 PM, Bernd Kuhls wrote:
> Fixes
> http://autobuild.buildroot.net/results/7f6/7f6e5396e95a3c3b676ca59bedc5317aa88e5f4c/
> http://autobuild.buildroot.net/results/6ac/6acd9cf25c3088d313247510db47a214fff30c67/
> http://autobuild.buildroot.net/results/a0d/a0da41bb734dbcdb17e13d852cfbdba5e069429e/
> http://autobuild.buildroot.net/results/d0c/d0caf60c1ddf4e05086c93f646e32b5ddc75b6ce/
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>
Reviewed-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
Tested-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
Although this indeed fixes the problem, there is one missing thing here
(which can be done in a separate patch). We will have the same problem
with dvdnav in the future. The reason why --enable-dvdread causes a
failure and --enable-dvdnav don't is because there is an error in the
configure script which doesn't disable the autodetection when
--enable-dvdnav is passed. That error could be fixed in the future and
we will face the same problem that we are fixing here, but for dvdnav.
So, --enable-dvdnav has to be removed as well, although it can be done
in a different patch.
Regards,
Vincent.
> ---
> v2: adjust mplayer.mk to match mplayer/configure behaviour
> by fixing libdvdread and libvorbis support (Thomas)
>
> package/mplayer/mplayer.mk | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/package/mplayer/mplayer.mk b/package/mplayer/mplayer.mk
> index 2f58864..8eb59c9 100644
> --- a/package/mplayer/mplayer.mk
> +++ b/package/mplayer/mplayer.mk
> @@ -117,9 +117,10 @@ endif
> # https://github.com/pld-linux/mplayer/blob/master/mplayer-libcdio.patch
> MPLAYER_CONF_OPTS += --disable-libcdio
>
> +# We intentionally don't pass --enable-dvdread, to let the
> +# autodetection find which library to link with.
> ifeq ($(BR2_PACKAGE_LIBDVDREAD),y)
> MPLAYER_CONF_OPTS += \
> - --enable-dvdread \
> --with-dvdread-config=$(STAGING_DIR)/usr/bin/dvdread-config
> MPLAYER_DEPENDENCIES += libdvdread
> endif
> @@ -175,9 +176,10 @@ MPLAYER_DEPENDENCIES += tremor
> MPLAYER_CONF_OPTS += --enable-tremor
> endif
>
> +# We intentionally don't pass --enable-libvorbis, to let the
> +# autodetection find which library to link with.
> ifeq ($(BR2_PACKAGE_LIBVORBIS),y)
> MPLAYER_DEPENDENCIES += libvorbis
> -MPLAYER_CONF_OPTS += --enable-libvorbis
> endif
>
> ifeq ($(BR2_PACKAGE_LIBMAD),y)
>
More information about the buildroot
mailing list