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

Use SauceCodePro Nerd Font as clock font failed #273

Open
Pagliacii opened this issue Jan 8, 2020 · 0 comments · May be fixed by #274
Open

Use SauceCodePro Nerd Font as clock font failed #273

Pagliacii opened this issue Jan 8, 2020 · 0 comments · May be fixed by #274

Comments

@Pagliacii
Copy link

Non-native speaker

Version of Variety

0.7.2

Describe the bug

I try to select the "SauceCodePro Nerd Font" as a clock font, but the clock could not show in wallpaper.

Context

There are some errors log like this:

WARNING: apply_clock() 'Could not execute clock convert command. Missing ImageMagick or bad filter defined? Resultcode: 256'

But I have the ImageMagick installed, and Variety works correctly before I change the clock font. If I run the clock convert command manually, it produces errors output like this:

convert: unable to open image 'Code': No such file or directory @ error/blob.c/OpenBlob/3496.
convert: no decode delegate for this image format `' @ error/constitute.c/ReadImage/562.

Finally, I run the command fc-match -f '%{file[0]} 'SauceCodePro Nerd Font:70', the output is /home/username/.local/share/fonts/Sauce Code Pro Regular Nerd Font Complete.ttf. So the reason caused this bug is that font file name contains spaces.

So it can be fixed by adding double quotes to the "fc-match -f '%{file[0]} 'SauceCodePro Nerd Font:70'" in the clock converts command.

To Reproduce

  1. download "SauceCodePro Nerd Font" to ~/.local/share/fonts, and run fc-cache
  2. select "SauceCodePro Nerd Font" as the clock font
  3. the clock does not show in the wallpaper
@Pagliacii Pagliacii linked a pull request Jan 8, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant