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

No Imlib2 loader for that file format persists on Debian Stable (10.6) #6

Closed
J-Reis opened this issue Nov 29, 2020 · 9 comments
Closed

Comments

@J-Reis
Copy link

J-Reis commented Nov 29, 2020

This project doesn't have many recent updates and may well be dead. If so, please forgive my post … but for the time being it is still relevant for Debian Stable for which the included imlib2 lacks webp support and no backport is currently available (as far as I can tell). Also still a top search result for 'sxiv webp support'. So hopefully I'm not committing any rage inducing internet faux pas (other than with this unnecessary paragraph of justification, anyway)!

Installed this as instructed in the readme and webp.so shows up in /usr/lib/x86_64-linux-gnu/imlib2/loaders/ as expected. Feh (and other imlib2 based viewers like SXIV) still fails with 'No Imlib2 loader for that file format'. Perhaps relevant, using essentially the same process to add an HEIC loader (https://github.com/vi/imlib2-heic) worked fine. I'm out of my depth troubleshooting this, so any advice (even angrily schooling me on what doc I should have read more carefully) much appreciated.

2020-11-29_10-41-35am

@mikoim
Copy link

mikoim commented Nov 29, 2020

I got the same error on Fedora 32.

imlib2.x86_64 1.5.1-4.fc32

dlopen(file, RTLD_NOW | RTLD_LOCAL); seems to return NULL.
image

https://fossies.org/d=564/imlib2-1.7.0/src/lib/loaders.c#31

@gawen947
Copy link
Owner

The project is not dead. But since it worked fine on my side, I did not see any reason to update it.

I managed to reproduce the problem on Debian. Indeed dlopen() returns NULL because of a missing symbol, that's also what I found with gdb.

A patch is under way, it should be ready in an hour or so.

@gawen947
Copy link
Owner

Just pushed a fix to master. Could you please both check that it works on Debian and Fedora?

@mikoim
Copy link

mikoim commented Nov 29, 2020

It works! thank you.

@gawen947
Copy link
Owner

@J-Reis does it work for you now?

@J-Reis
Copy link
Author

J-Reis commented Nov 30, 2020

Yep, works perfectly. Couldn't have hoped for a better (or faster) fix, many thanks.

Also, no need to update a feature complete project that works! I'm just gun shy having been told off for posting issues on older projects. Hard for a regular joe to navigate all the prickly etiquette I find … or maybe I'm just dense. Either way, none of that is on you! Thanks again.

@gawen947
Copy link
Owner

Thanks for reporting, actually it helps a lot.

Note that compared to the version shipped with the latest version of Imlib2, this one support loading the first frame of animated WebP and is compatible with a broader range of libwebp versions.

@gadicc
Copy link

gadicc commented Nov 30, 2020

@gawen947 said:

A patch is under way, it should be ready in an hour or so.

😮 👏 🙏

@reinierpost
Copy link

reinierpost commented Dec 21, 2020

I just installed this on Ubuntu 18.04 (which is Debian-based). This check in the Makefile failed:

ifneq ($(.SHELLSTATUS),0)
$(error cannot guess imlib2 version)
endif

(it depends on make being at least 4.2 and Ubuntu 18.04 still has make 4.1)
so I took it out, and after make and sudo make install, my feh now loads .webp images. Thanks a bunch!

It would be nice if someone did a PPA for Debian/Ubuntu.

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

No branches or pull requests

5 participants