unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’
@ 2021-04-25 13:38 Ludovic Courtès
  2023-07-20 18:40 ` bug#48018: [PATCH core-updates 2/4] gnu: imagemagick: Fix converting webp images Saku Laesvuori via Bug reports for GNU Guix
  2023-11-06 11:07 ` bug#48018: reviewed, checked and re-based the previous commit Wicki Gabriel (wicg)
  0 siblings, 2 replies; 6+ messages in thread
From: Ludovic Courtès @ 2021-04-25 13:38 UTC (permalink / raw)
  To: 48018

ImageMagick’s ‘convert’ refers to the ‘dwebp’ program with a leading
space, which prevents conversion to/from webp:

--8<---------------cut here---------------start------------->8---
$ guix time-machine --commit=2a624253bc55e2a94f6581d6e790303575436c96 -- \
   environment -C --ad-hoc strace imagemagick libwebp -- \
   strace -f -e execve convert t.webp t.png
execve("/gnu/store/2rmnafvwqvl460qa370gn18yl09dx91h-profile/bin/convert", ["convert", "t.webp", "t.png"], 0x7ffe6637b988 /* 10 vars */) = 0
strace: Process 5 attached
[pid     5] execve("/gnu/store/2rmnafvwqvl460qa370gn18yl09dx91h-profile/bin/ dwebp", [" dwebp", "-pam", "/tmp/magick-FacyU5v0_s0CnpxpMzs0"..., "-o", "/tmp/magick-4R_OusxZBx5fjGrfq0KU"...], 0x7ffd7f3538e8 /* 10 vars */) = -1 ENOENT (No such file or directory)
[pid     5] +++ exited with 1 +++
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=5, si_uid=1000, si_status=1, si_utime=0, si_stime=0} ---
convert: unable to open image `t.webp': No such file or directory @ error/blob.c/OpenBlob/2924.
convert: delegate failed `' dwebp' -pam '%i' -o '%o'' @ error/delegate.c/InvokeDelegate/1970.
convert: unable to open file `/tmp/magick-e8GzrEcauG0mZICNZggy_vG77mtn29lu': No such file or directory @ error/constitute.c/ReadImage/615.
convert: no images defined `t.png' @ error/convert.c/ConvertImageCommand/3229.
+++ exited with 1 +++
--8<---------------cut here---------------end--------------->8---

Ludo’.




^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#48018: [PATCH core-updates 2/4] gnu: imagemagick: Fix converting webp images
  2021-04-25 13:38 bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Ludovic Courtès
@ 2023-07-20 18:40 ` Saku Laesvuori via Bug reports for GNU Guix
  2023-11-06 11:07 ` bug#48018: reviewed, checked and re-based the previous commit Wicki Gabriel (wicg)
  1 sibling, 0 replies; 6+ messages in thread
From: Saku Laesvuori via Bug reports for GNU Guix @ 2023-07-20 18:40 UTC (permalink / raw)
  To: 64751; +Cc: 48018, Saku Laesvuori

Fixes <https://issues.guix.gnu.org/48018>.

* gnu/packages/imagemagick.scm (imagemagick)[inputs]: Add libwebp.
---
The comments in the code imply that this could be applied to master, but
`guix refresh` claimed it to have over 3 000 dependents. Go figure.

 gnu/packages/imagemagick.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/imagemagick.scm b/gnu/packages/imagemagick.scm
index 169dc12f35..55e100a71c 100644
--- a/gnu/packages/imagemagick.scm
+++ b/gnu/packages/imagemagick.scm
@@ -158,6 +158,8 @@ (define-public imagemagick
                (base32
                 "0didbs10i9zb4dgripa851j7fivxb9jar7l3vvxz6i4kn6xvdv7r"))
               (patches (search-patches "imagemagick-Fix-tests.patch"))))
+    (inputs (cons `("libwebp" ,libwebp)
+                  (package-inputs imagemagick/stable)))
     (arguments
      (append (package-arguments imagemagick/stable)
              '(#:tests? #t)))))
-- 
2.41.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* bug#48018: reviewed, checked and re-based the previous commit
  2021-04-25 13:38 bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Ludovic Courtès
  2023-07-20 18:40 ` bug#48018: [PATCH core-updates 2/4] gnu: imagemagick: Fix converting webp images Saku Laesvuori via Bug reports for GNU Guix
@ 2023-11-06 11:07 ` Wicki Gabriel (wicg)
  2024-01-03 22:20   ` Maxim Cournoyer
  1 sibling, 1 reply; 6+ messages in thread
From: Wicki Gabriel (wicg) @ 2023-11-06 11:07 UTC (permalink / raw)
  To: 48018@debbugs.gnu.org


[-- Attachment #1.1: Type: text/plain, Size: 134 bytes --]

I've checked the patch above which works like a charm.  I've created a new patch.  Please credit the original author before merging.

[-- Attachment #1.2: Type: text/html, Size: 544 bytes --]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Fix-imagemagick-webp-related-failures.patch --]
[-- Type: text/x-patch; name="0001-gnu-Fix-imagemagick-webp-related-failures.patch", Size: 1125 bytes --]

From 37faa5cac3545406721cfaaf4b2f97841a889510 Mon Sep 17 00:00:00 2001
Message-ID: <37faa5cac3545406721cfaaf4b2f97841a889510.1699268702.git.wicg@zhaw.ch>
From: Gabriel Wicki <gabriel@erlikon.ch>
Date: Mon, 6 Nov 2023 12:01:47 +0100
Subject: [PATCH] gnu: Fix imagemagick webp related failures.

This fixes #48018, as suggested by Saku Laesvuori on the issue tracker.

* gnu/packages/imagemagick.scm (imagemagick/stable)[inputs]: Add libwebp.

Change-Id: I3c02d0fb6c421f7d865b19af359e20068d00b60a
---
 gnu/packages/imagemagick.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/packages/imagemagick.scm b/gnu/packages/imagemagick.scm
index b407af3588..ac366a378b 100644
--- a/gnu/packages/imagemagick.scm
+++ b/gnu/packages/imagemagick.scm
@@ -124,6 +124,7 @@ (define-public imagemagick/stable
                ("libtiff" ,libtiff)
                ("libpng" ,libpng)
                ("libjpeg" ,libjpeg-turbo)
+               ("libwebp" ,libwebp)
                ("pango" ,pango)
                ("freetype" ,freetype)
                ("bzip2" ,bzip2)

base-commit: d52ee267ee8245a6edfcd137c73d20b036ec08d1
-- 
2.41.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* bug#48018: reviewed, checked and re-based the previous commit
  2023-11-06 11:07 ` bug#48018: reviewed, checked and re-based the previous commit Wicki Gabriel (wicg)
@ 2024-01-03 22:20   ` Maxim Cournoyer
  2024-01-04 13:13     ` bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Simon South
  0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2024-01-03 22:20 UTC (permalink / raw)
  To: Wicki Gabriel (wicg); +Cc: 48018-done@debbugs.gnu.org

Hi,

"Wicki Gabriel (wicg)" <wicg@zhaw.ch> writes:

> I've checked the patch above which works like a charm.  I've created a
> new patch.  Please credit the original author before merging.

I've applied it on core-updates, as imagemagick causes a large number of
rebuilds.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’
  2024-01-03 22:20   ` Maxim Cournoyer
@ 2024-01-04 13:13     ` Simon South
  2024-01-04 17:51       ` Maxim Cournoyer
  0 siblings, 1 reply; 6+ messages in thread
From: Simon South @ 2024-01-04 13:13 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 48018@debbugs.gnu.org, Wicki Gabriel (wicg)

Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
> I've applied it on core-updates, as imagemagick causes a large number
> of rebuilds.

Unfortunately a small bug crept in with commit 62d1806f42: ",name" needs
to be changed to "$name" or building ImageMagick fails with

unquote: expression not valid outside of quasiquote in form (unquote name)

-- 
Simon South
simon@simonsouth.net




^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’
  2024-01-04 13:13     ` bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Simon South
@ 2024-01-04 17:51       ` Maxim Cournoyer
  0 siblings, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2024-01-04 17:51 UTC (permalink / raw)
  To: Simon South; +Cc: 48018-done@debbugs.gnu.org, Wicki Gabriel (wicg)

Hi,

Simon South <simon@simonsouth.net> writes:

> Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
>> I've applied it on core-updates, as imagemagick causes a large number
>> of rebuilds.
>
> Unfortunately a small bug crept in with commit 62d1806f42: ",name" needs
> to be changed to "$name" or building ImageMagick fails with
>
> unquote: expression not valid outside of quasiquote in form (unquote name)

Thanks for the heads-up.  That's now fixed.  Apologies for sloppy QA
:-).

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-01-04 20:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-04-25 13:38 bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Ludovic Courtès
2023-07-20 18:40 ` bug#48018: [PATCH core-updates 2/4] gnu: imagemagick: Fix converting webp images Saku Laesvuori via Bug reports for GNU Guix
2023-11-06 11:07 ` bug#48018: reviewed, checked and re-based the previous commit Wicki Gabriel (wicg)
2024-01-03 22:20   ` Maxim Cournoyer
2024-01-04 13:13     ` bug#48018: ImageMagick’s ‘convert’ fails to delegate to ‘dwebp’ Simon South
2024-01-04 17:51       ` Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).