Created attachment 220783 [details]
See URL for changelog. Runtime tested under Sway on 11.4 i386 (jail) with Python 3.7 and 13.0 amd64 (host) with Python 3.9.
I'll try to test it soon and also improve rar support:
I've been recently asked to add a dependency to archivers/libunrar5.
RAR archives work if one of the following is available at runtime (with decreasing priority)
libunrar5 is the preferred way, so I plan to add it as dependency.
Created attachment 220848 [details]
patch by jbeich + add libunrar to RUN_DEPENDS
Thanks, patch looks fine.
I've expanded the patch to include run-dependency on libunrar (see comment above)
I guess RUN_DEPENDS is correct here (not required at build time) instead of LIB_DEPENDS. But feel free to change that.
Created attachment 220878 [details]
Update to a newer snapshot and make libunrar5 dependency optional.
I mainly use mcomix as a gallery viewer and with comics packed into *.zip. libunrar5 and unrar don't work with my *.rar comics unlike p7zip.
$ file /tmp/foo.rar
/tmp/foo.rar: RAR archive data, v4, os: Win32
$ mcomix /tmp/foo.rar
00:00:48 [MainThread] WARNING: ! Could not read /tmp/foo.rar
$ truss mcomix /tmp/foo.rar 2>&1 | fgrep libunrar
openat(AT_FDCWD,"/usr/local/lib/libunrar.so.5",O_RDONLY|O_CLOEXEC|O_VERIFY,00) = 14 (0xe)
$ tar tf /tmp/foo.rar
Must be some weird RAR variant that is supported by p7zip but not by libunrar.
Adding libunrar as option is probably the best we can do for now without touching the source.
New version only contains minor changes, so that's also fine with me.
A commit references this bug:
Date: Fri Dec 25 01:01:11 UTC 2020
New revision: 559206
graphics/py-mcomix: update to 1.2.1g20201223
- Add optional dependency on libunrar
Approved by: Stefan Ehmann (maintainer)