Package Details: xnviewmp 0.93.1-2

Git Clone URL: https://aur.archlinux.org/xnviewmp.git (read-only)
Package Base: xnviewmp
Description: An efficient multimedia viewer, browser and converter.
Upstream URL: http://www.xnview.com/en/xnviewmp/
Keywords: graphics
Licenses: custom
Submitter: oliwer
Maintainer: Corax
Last Packager: Corax
Votes: 264
Popularity: 0.492433
First Submitted: 2008-07-25 19:01
Last Updated: 2019-03-16 14:41

Pinned Comments

Corax commented on 2017-01-21 15:34

I have created a new package: xnviewmp-system-libs. This is exactly the same build, except that the bundled Qt/icu libs are removed. Please try it if you want to use XnView without the bundled libs, and discuss any issue related to this configuration here: https://aur.archlinux.org/packages/xnviewmp-system-libs/. Do keep in mind that this is an experimental package though, and things may break when I try to fix other things...

Corax commented on 2017-01-20 21:49

If makepkg fails because the checksum is incorrect, please flag the package out-of-date and I will update the PKGBUILD.

Upstream does not version the released archives and does not announce new releases either, so the checksum is bound to become wrong every time there is a new release, and there's nothing I can do about that.

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 7 8 9 10 ... Next › Last »

goldpaw commented on 2017-02-01 14:37

@Corax happy to help! :D

And I totally agree that the way the xnview team does their versioning is a problem. We're bound to get checksum errors and bugs from time to time since they don't really label or announce anything. But it's easy enough for us to check the md5 of the source and manually modify the PKGBUILD when needed. We'll figure it out!

Thanks for your continued work, wouldn't know what to do without this package! :)

Corax commented on 2017-01-24 22:10

@goldpaw I implemented your suggestion in the latest commit (I didn't bump the pkgrel as there is no point in making everyone rebuild exactly the same package). Thanks again!

Corax commented on 2017-01-23 21:27

@goldpaw I actually thought about doing that (very briefly though). I was concerned that it could be deceiving as the source archive would be renamed to the wrong version if the PKGBUILD it out-of-date, but that only happens if you've never run makepkg before, which probably means that you don't keep the PKGBUILD repo after building, in which case it doesn't matter how the sources are named. Also doing so would allow me (and others) to keep the various versions around. So yes definitely a good idea, but... Yeah I remember now, there's actually a problem with that. As we've seen last week, upstream doesn't necessarily bump the version number when publishing a new release. I think I'll work around that by appending a rel (release) number. Anyway thanks for mentioning this, I hadn't considered it seriously enough :-)

DungeonMaster commented on 2017-01-22 22:03

@Corax

Beautiful. Works great.

Just a hint for those who might be experiencing problems with QT_QPA_PLATFORMTHEME=qt5ct I downloaded a package in the AUR called qt5gtk2 and I use QT_QPA_PLATFORMTHEME=qt5gtk

qt5gtk is explained well in this article:

http://dglava.github.io/en/qtplatform-style.html

goldpaw commented on 2017-01-22 15:21

@Corax the problem with the wrong md5sum is that the download is stored with the same name each time. This leads to PKGBUILD assuming that the file is already downloaded, and thus it attempts to use the existing one instead. Which leads to a md5sum mismatch, of course.

The solution is to modify the PKGBUILD slightly, so that it includes the pkgver$ variable in the downloaded filename. All that needs changing are the two sources. Not that it's important to use " instead of ' here, otherwise it won't change the $pkgver variable into the actual file version.

I changed the thwo source lines to this, and was able to build it without manually having to delete the old download first:

source_x86_64=("XnViewMP_linux_x64_$pkgver.tgz::http://download.xnview.com/XnViewMP-linux-x64.tgz")
source_i686=("XnViewMP_linux_$pkgver.tgz::http://download.xnview.com/XnViewMP-linux.tgz")

Yay! :D

Corax commented on 2017-01-21 15:34

I have created a new package: xnviewmp-system-libs. This is exactly the same build, except that the bundled Qt/icu libs are removed. Please try it if you want to use XnView without the bundled libs, and discuss any issue related to this configuration here: https://aur.archlinux.org/packages/xnviewmp-system-libs/. Do keep in mind that this is an experimental package though, and things may break when I try to fix other things...

Corax commented on 2017-01-21 12:13

@blitz qt5-location is a dependency of qt5-webkit so there's no need to specify it. I've just checked that all needed qt5-* dependencies are there. And yes, it's much smaller, that's one of the reasons why it's good to get rid of the bundled libs.

@fuan_k You're absolutely right, the libicu 54 libs are gone, my src tree was not in a clean state >.< Will fix right now.

Regarding QT_QPA_PLATFORMTHEME, it's a good point as I use it too (although I don't use qt5ct anymore, I use gtk2 from qt5-styleplugins), I'll try to see how it interacts with the rest...

fuan_k commented on 2017-01-20 23:05

The only reliable way I found to fix the font issue is to use qt5ct and add this to the environment variables (or just add the following to xnview.sh maybe?)

QT_QPA_PLATFORMTHEME=qt5ct

This will force the Qt application to use whatever you set up. You can also use the "--platformtheme qt5ct" command line option.
You'll notice this in terminal output when running xnviewmp: qt5ct: using qt5ct plugin

Unfortunately, there is still a weird problem: if you use a different desktop environment (i3wm for example) and if you load another picture (from a file manager), the theme will get swapped to the Fusion Theme. More details there: https://wiki.archlinux.org/index.php/qt#Qt5

Fonts will stay fine though, but some widget buttons will get shuffled around and such. Very strange. Anyway, that should go to the XnViewMP developer. ;)

fuan_k commented on 2017-01-20 22:46

Had an error with this line, as it couldn't find the file (I never had it): rm "${pkgdir}/opt/${pkgname}/lib/"libicu*.54*

You might want to change for the following (I could be wrong, but I tested it and it worked I think):
if [[ -e "${pkgdir}/opt/${pkgname}/lib/"libicu*.54* ]]; then
rm "${pkgdir}/opt/${pkgname}/lib/"libicu*.54*
fi

Anyway, I still get issues with this package:
-remove qt.conf as usual (otherwise I get: This application failed to start because it could not find or load the Qt platform plugin "xcb"
in "".)
-the font is indeed still the wrong one. Still investigating... Have you guys tried using strace on this program? It's crazy the amount of libraries it's looking for in places that don't even exist... ugh.

blitz commented on 2017-01-20 22:28

xnviewmp 0.84-1 build works
with system Qt5 libs
without bundled /opt/xnviewmp/lib/"lib*
Total Installed Size: 36.54 MiB

The only missing dependency is extra/qt5-location

>Upstream released a new version today (?), which fixes the libicu issue
XnViewMP
Version 0.84 x64 (Jan 17 2017)
Libformat version 6.95