New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--magick-timeout broken (wrong path?) #323

Closed
fedov opened this Issue Sep 4, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@fedov

fedov commented Sep 4, 2017

Trying to view an svg image with "feh --magick-timeout 2 test.svg" results in the following error:

convert: no decode delegate for this image format SVG_UZZ2GD' @ error/constitute.c/ReadImage/504. convert: no images defined png:fd:5' @ error/convert.c/ConvertImageCommand/3258.
feh WARNING: /tmp/feh_029046_000001_test.svg_UZz2gD - No Imlib2 loader for that file format
feh WARNING: test.svg - No Imlib2 loader for that file format
feh: No loadable images specified.
See 'feh --help' or 'man feh' for detailed usage information

Convert seems to do conversion, because there is a file called "fd:4" in the cwd. Its viewable, seems to be a png version of the original. There seems to be some confusion about the path for the tmp file.
This happens with feh 2.19.3 and ImageMagick 6.9.9-11

@derf

This comment has been minimized.

Owner

derf commented Sep 5, 2017

This seems to be an ImageMagick bug: the png:fd:X syntax (to output a PNG image to a file descriptor) is working fine in v6.8.9-9, but not in 6.9.7-4. The next feh release will include a workaround for this.

@derf derf closed this Sep 5, 2017

@derf derf added the bug label Sep 5, 2017

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Oct 19, 2017

szptvlfn
Update to 2.21
https://git.finalrewind.org/feh/plain/ChangeLog
Sat, 07 Oct 2017 12:14:17 +0200  Daniel Friesel <derf+feh@finalrewind.org>

* Release v2.21
    * Add toggle_fixed_geometry ('g') key binding to toggle window auto-resize
    * Improve control via terminal input
    * Fix crash (segmentation fault) when using feh -O in non-index mode
    * Fix --force-aliasing (and possibly other options) missing from ~/.fehbg
      when using them for background setting

Thu, 07 Sep 2017 20:20:11 +0200  Daniel Friesel <derf+feh@finalrewind.org>

* Release v2.20
    * Fix clang/gcc warnings (Patches by orbea)
    * Add support for control via terminal input. feh will read key presses
      from the controlling terminal and handle them like X11 key presses
      inside the feh window. Note that at the moment, only lower / upper case
      ASCII letters and a very small set of additional keys are supported.
    * Fix broken ImageMagick support (see --magick-timeout) when using some
      ImageMagick versions <derf/feh#323>
    * Remove images from the filelist if they were removed by executing a
      user-defined action <derf/feh#322>

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Oct 22, 2017

szptvlfn
Update to 2.21
https://git.finalrewind.org/feh/plain/ChangeLog
Sat, 07 Oct 2017 12:14:17 +0200  Daniel Friesel <derf+feh@finalrewind.org>

* Release v2.21
    * Add toggle_fixed_geometry ('g') key binding to toggle window auto-resize
    * Improve control via terminal input
    * Fix crash (segmentation fault) when using feh -O in non-index mode
    * Fix --force-aliasing (and possibly other options) missing from ~/.fehbg
      when using them for background setting

Thu, 07 Sep 2017 20:20:11 +0200  Daniel Friesel <derf+feh@finalrewind.org>

* Release v2.20
    * Fix clang/gcc warnings (Patches by orbea)
    * Add support for control via terminal input. feh will read key presses
      from the controlling terminal and handle them like X11 key presses
      inside the feh window. Note that at the moment, only lower / upper case
      ASCII letters and a very small set of additional keys are supported.
    * Fix broken ImageMagick support (see --magick-timeout) when using some
      ImageMagick versions <derf/feh#323>
    * Remove images from the filelist if they were removed by executing a
      user-defined action <derf/feh#322>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment