[Buildroot] [PATCH 2/2] Compile Poppler over Qt
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Mar 4 14:44:46 UTC 2014
Dear jeremie.scheer at armadeus.com,
On Tue, 4 Mar 2014 14:39:03 +0100, jeremie.scheer at armadeus.com wrote:
> From: Jeremie Scheer <jeremie.scheer at armadeus.com>
>
>
> Signed-off-by: Jeremie Scheer <jeremie.scheer at armadeus.com>
> ---
> package/poppler/poppler.mk | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/package/poppler/poppler.mk b/package/poppler/poppler.mk
> index 040327f..7dce7a0 100644
> --- a/package/poppler/poppler.mk
> +++ b/package/poppler/poppler.mk
> @@ -7,10 +7,17 @@
> POPPLER_VERSION = 0.24.4
> POPPLER_SOURCE = poppler-$(POPPLER_VERSION).tar.xz
> POPPLER_SITE = http://poppler.freedesktop.org
> -POPPLER_DEPENDENCIES = fontconfig
> POPPLER_LICENSE = GPLv2+
> POPPLER_LICENSE_FILES = COPYING
> POPPLER_CONF_OPT = --with-font-configuration=fontconfig
> +POPPLER_INSTALL_STAGING = YES
> +
> +ifeq ($(BR2_PACKAGE_QT),y)
> + POPPLER_DEPENDENCIES = qt fontconfig
> +else
> + POPPLER_DEPENDENCIES = fontconfig
> + POPPLER_CONF_OPT += --disable-poppler-qt4
Why isn't the fontconfig dependency part of the if, and not kept
outside, since it is not related to Qt ?
Also, if poppler has a --disable-poppler-qt4 in one case, it probably
has a --enable-poppler-qt4 option in the other case, no?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list