Message ID | 1393940343-15899-1-git-send-email-jeremie.scheer@armadeus.com |
---|---|
State | Superseded |
Headers | show |
Dear jeremie.scheer@armadeus.com, On Tue, 4 Mar 2014 14:39:03 +0100, jeremie.scheer@armadeus.com wrote: > From: Jeremie Scheer <jeremie.scheer@armadeus.com> > > > Signed-off-by: Jeremie Scheer <jeremie.scheer@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
On 03/04/2014 03:44 PM, Thomas Petazzoni wrote: > Dear jeremie.scheer@armadeus.com, > > On Tue, 4 Mar 2014 14:39:03 +0100, jeremie.scheer@armadeus.com wrote: >> From: Jeremie Scheer <jeremie.scheer@armadeus.com> >> >> >> Signed-off-by: Jeremie Scheer <jeremie.scheer@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 ? You're right, I will put it out of the Qt condition, as 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? No, there is no enable option for Qt4 compilation. By default, Poppler tries to compile over Qt, but you can disable it with the --disable-poppler-qt4 option. > > Thanks! > > Thomas
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 +endif ifeq ($(BR2_PACKAGE_LCMS2),y) POPPLER_CONF_OPT += --enable-cms=lcms2