[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Mauro Condarelli]

Hi,
it seems imagemagick package does not force dependency on pango, but fails if it's not found.

This trivial patch solves the problem for me:


$ git diff
diff --git a/package/imagemagick/imagemagick.mk b/package/imagemagick/imagemagick.mk
index 1594f69..e8c7ab7 100644
--- a/package/imagemagick/imagemagick.mk
+++ b/package/imagemagick/imagemagick.mk
@@ -44,6 +44,13 @@ else
  IMAGEMAGICK_CONF_OPTS += --without-fontconfig
  endif

+ifeq ($(BR2_PACKAGE_PANGO),y)
+IMAGEMAGICK_CONF_OPTS += --with-pango
+IMAGEMAGICK_DEPENDENCIES += pango
+else
+IMAGEMAGICK_CONF_OPTS += --without-pango
+endif
+
  ifeq ($(BR2_PACKAGE_FREETYPE),y)
  IMAGEMAGICK_CONF_OPTS += --with-freetype
  IMAGEMAGICK_CONF_ENV += \


Regards
Mauro Condarelli

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Thomas Petazzoni]

Dear Mauro Condarelli,

Thanks for your contribution! See below for some comments/questions.

On Thu, 6 Aug 2015 20:15:34 +0200, Mauro Condarelli wrote:
> Hi,
> it seems imagemagick package does not force dependency on pango, but fails if it's not found.

Can you give more details on how it fails?

> $ git diff
> diff --git a/package/imagemagick/imagemagick.mk b/package/imagemagick/imagemagick.mk
> index 1594f69..e8c7ab7 100644
> --- a/package/imagemagick/imagemagick.mk
> +++ b/package/imagemagick/imagemagick.mk
> @@ -44,6 +44,13 @@ else
>   IMAGEMAGICK_CONF_OPTS += --without-fontconfig
>   endif
> 
> +ifeq ($(BR2_PACKAGE_PANGO),y)
> +IMAGEMAGICK_CONF_OPTS += --with-pango
> +IMAGEMAGICK_DEPENDENCIES += pango
> +else
> +IMAGEMAGICK_CONF_OPTS += --without-pango
> +endif

This patch is not making pango a mandatory dependency of imagemagick.
It only makes it an optional dependency;

Again, can you give more details about the problem you're seeing?

The patch looks good (i.e pango is an optional dependency of
imagemagick), but the description is not. Can you clarify?

Thanks!

Thoma

-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Mauro Condarelli]

Hi Thomas,
AFAIK pango is not mandatory to compile imagemagick, but without this patch imagemagick package would fail compilation if pango is not selected (either directly or because of other dependencies).
I.e.: to compile without pango it seem mandatory to explicitly pass " --without-pango" to imagemaagick's configure script; so we have two possibilities: either automatically select pango or do something along the lines of my patch (shamelessly adapted from other stances in imagemagick.mk).
Does this clarify or did I misunderstand You?

Regards
Mauro Condarelli

Il 06/08/2015 22:48, Thomas Petazzoni ha scritto:
> Dear Mauro Condarelli,
>
> Thanks for your contribution! See below for some comments/questions.
>
> On Thu, 6 Aug 2015 20:15:34 +0200, Mauro Condarelli wrote:
>> Hi,
>> it seems imagemagick package does not force dependency on pango, but fails if it's not found.
> Can you give more details on how it fails?
>
>> $ git diff
>> diff --git a/package/imagemagick/imagemagick.mk b/package/imagemagick/imagemagick.mk
>> index 1594f69..e8c7ab7 100644
>> --- a/package/imagemagick/imagemagick.mk
>> +++ b/package/imagemagick/imagemagick.mk
>> @@ -44,6 +44,13 @@ else
>>    IMAGEMAGICK_CONF_OPTS += --without-fontconfig
>>    endif
>>
>> +ifeq ($(BR2_PACKAGE_PANGO),y)
>> +IMAGEMAGICK_CONF_OPTS += --with-pango
>> +IMAGEMAGICK_DEPENDENCIES += pango
>> +else
>> +IMAGEMAGICK_CONF_OPTS += --without-pango
>> +endif
> This patch is not making pango a mandatory dependency of imagemagick.
> It only makes it an optional dependency;
>
> Again, can you give more details about the problem you're seeing?
>
> The patch looks good (i.e pango is an optional dependency of
> imagemagick), but the description is not. Can you clarify?
>
> Thanks!
>
> Thoma
>

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Thomas Petazzoni]

Dear Mauro Condarelli,

On Thu, 6 Aug 2015 23:49:34 +0200, Mauro Condarelli wrote:

> AFAIK pango is not mandatory to compile imagemagick, but without this patch imagemagick package would fail compilation if pango is not selected (either directly or because of other dependencies).
> I.e.: to compile without pango it seem mandatory to explicitly pass " --without-pango" to imagemaagick's configure script; so we have two possibilities: either automatically select pango or do something along the lines of my patch (shamelessly adapted from other stances in imagemagick.mk).
> Does this clarify or did I misunderstand You?

Could you please paste the last ~100 lines of the build output showing
the error, as well as post somewhere the config.log from imagemagick?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Thomas Petazzoni]

Dear Mauro Condarelli,

On Thu, 6 Aug 2015 23:49:34 +0200, Mauro Condarelli wrote:

> AFAIK pango is not mandatory to compile imagemagick, but without this
> patch imagemagick package would fail compilation if pango is not
> selected (either directly or because of other dependencies).

Imagemagick without pango selected builds just fine here, and it's
using pkg-config to detect pango, so I'm not quite sure how it could
misdetect a host-installed pango on your system.

So, we'll really need the build output and config.log.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] imagemagick depends on pango even if not available

[Thomas Petazzoni]

Hello all,

Since I've got no further feedback from Mauro, I've marked this patch
as Rejected for now. Mauro, once you can provide more details, we can
revisit this as needed. Thanks!

Thomas

On Fri, 7 Aug 2015 00:04:16 +0200, Thomas Petazzoni wrote:
> Dear Mauro Condarelli,
> 
> On Thu, 6 Aug 2015 23:49:34 +0200, Mauro Condarelli wrote:
> 
> > AFAIK pango is not mandatory to compile imagemagick, but without this
> > patch imagemagick package would fail compilation if pango is not
> > selected (either directly or because of other dependencies).
> 
> Imagemagick without pango selected builds just fine here, and it's
> using pkg-config to detect pango, so I'm not quite sure how it could
> misdetect a host-installed pango on your system.
> 
> So, we'll really need the build output and config.log.
> 
> Thanks,
> 
> Thomas



-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com