Package Details: keyfinder-cli 1.1.4-2

Git Clone URL: https://aur.archlinux.org/keyfinder-cli.git (read-only, click to copy)
Package Base: keyfinder-cli
Description: Estimate the musical key of many different audio file formats
Upstream URL: https://github.com/evanpurkhiser/keyfinder-cli
Licenses: GPL-3.0-only
Submitter: BrainDamage
Maintainer: BrainDamage
Last Packager: BrainDamage
Votes: 1
Popularity: 0.34
First Submitted: 2025-04-20 08:19 (UTC)
Last Updated: 2025-11-22 11:08 (UTC)

Latest Comments

BrainDamage commented on 2025-11-22 10:44 (UTC) (edited on 2025-11-22 10:48 (UTC) by BrainDamage)

You are correct, however, the problem is not on the PKGBUILD side, it's upstream. I did not change anything regarding man path installation, and the value of the buildsystem flags I am using are correctly set even if relying on defaults.

The buildsystem parameters let you choose the man directory base path (CMAKE_INSTALL_MANDIR) "/usr/share/man", man{number} has to be appended from the install script to add to the relevant section. I could force CMAKE_INSTALL_MANDIR to be "/usr/share/man/man1", but it would be incorrect beause if the project were to ship another man section, like eg a section 2, it'd end up either in "/usr/share/man/man1" or "/usr/share/man/man1/man2" depending how they do it which would be equally incorrect. Upstream is misunderstanding what the parameter CMAKE_INSTALL_MANDIR is for and should add make the installation path ${CMAKE_INSTALL_MANDIR}/man${man_section}/${projectname}.${man_section}.${extension} by itself.

I am going to add a post-buildsystem temporary workaround that moves the file to the correct place, and open a bug report upstream.

EDIT: upstream already solved the issue, but hasn't released yet:https://github.com/evanpurkhiser/keyfinder-cli/commit/dc7033634e5e72ff20cf7a41d53216086c3af3d8

tkna commented on 2025-11-22 10:18 (UTC) (edited on 2025-11-22 10:20 (UTC) by tkna)

I apologize if I'm mistaken, but it appears that manual keyfinder-cli.1.gz should be stored in /usr/share/man/man1/ rather than /usr/share/man/.