Skip to content
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

Won't convert images due to the latest update of imagemagick #420

Closed
1 task done
xelser opened this issue Jun 7, 2024 · 1 comment
Closed
1 task done

Won't convert images due to the latest update of imagemagick #420

xelser opened this issue Jun 7, 2024 · 1 comment
Labels

Comments

@xelser
Copy link

xelser commented Jun 7, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Operating system

Arch Linux

Installation method

Package Manager (from OS)

Betterlockscreen & Dependency-Versions

- Betterlockscreen: version: v4.2.0 (dunst: true, feh: true)
- i3lock-color: version 2.13.c.5
- ImageMagick: 7.1.1-33
- dunst (if used): 1.11.0
- feh (if used): 3.10.2

Bug description

/usr/bin/betterlockscreen: eval: line 467: unexpected EOF while looking for matching `''
WARNING: The convert command is deprecated in IMv7, use "magick"

Steps to reproduce

  1. Install betterlockscreen from the AUR.
  2. Use the betterlockscreen -u command to any directory or picture using any effects.

Relevant log output

betterlockscreen -u "/home/xelser/Pictures/xelser's Pictures/Catppuccin/" --fx dim 50
[B] Betterlockscreen
[*] Updating image cache...
[=] Detected 1 display(s) @ 1366x768 total resolution
[=] Original image(s): evening-sky.png
[=] Processing display: VGA-1 (1)
[=] Resolution: 1366x768
[*] Resizing base image...
/usr/bin/betterlockscreen: eval: line 467: unexpected EOF while looking for matching `''
[*] Rendering 'dim' effect...
WARNING: The convert command is deprecated in IMv7, use "magick"

convert: unable to open image '/home/xelser/.cache/betterlockscreen/1-VGA-1/resize.png': No such file or directory @ error/blob.c/OpenBlob/3596.
convert: no images defined `/home/xelser/.cache/betterlockscreen/1-VGA-1/dim.png' @ error/deprecate.c/ConvertImageCommand/3368.
[*] Rendering final wallpaper images...
[*] Rendering final lockscreen images...
WARNING: The convert command is deprecated in IMv7, use "magick"

WARNING: The convert command is deprecated in IMv7, use "magick"

WARNING: The convert command is deprecated in IMv7, use "magick"
@xelser xelser added the bug label Jun 7, 2024
@q12p
Copy link
Contributor

q12p commented Jun 18, 2024

This doesn't seem to be an issue with the new command "magick". Changing "convert" for "magick" makes this issue continue to occur.

This issue seems to be rather related to #417 and #386, where input is not correctly taken by the script if there are spaces or ' and ", making an invalid path detection when trying to get the image path.

@xelser xelser closed this as completed Jun 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants