man broken by default with recent version

sergei-mironov / xkb-switch

Switch your X keyboard layouts from the command line

MIT License

353 stars 40 forks source link

man broken by default with recent version #48

Closed orestisfl closed 4 years ago

orestisfl commented 4 years ago

Hey, this was initially reported here but I see that you recently changed something in your CMakeLists.txt regarding man. Specifically, after installing I get this error:

$ man 1 xkb-switch
man: warning: /usr/share/man/man1/xkb-switch.1.xz.gz: ignoring bogus filename
No manual entry for xkb-switch in section 1

I follow a very simple installation process: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=xkb-switch

sergei-mironov commented 4 years ago

Hi. Yep I also see this error, thanks. This could be due to .xz. compression. As a workaround, you could disable it by commenting-out corresponding line in CMakeLists.txt. Maybe we shouldn't use it by default at all.

SET(MAN_COMPRESSION "xz" CACHE STRING "Manpages compression tool")

orestisfl commented 4 years ago

cmake -DCOMPRESS_EXECUTABLE=no can work as a workaround without source code modifications but I am not sure how it works

sergei-mironov commented 4 years ago

My guess is that some distros compress manpages if they think they were not compressed by the installation procedure. We used xz compressor which is not very popular, so some distros 'fixed' the 'missing' compression by running additional gzip round. I changed default compressor to a more recognizable gz.