Summary: | cargo.eclass has a logic error / oversight in doman | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jannik Glückert <jannik.glueckert> |
Component: | Eclasses | Assignee: | Gentoo Rust Project <rust> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | audvare, dschridde+gentoobugs, jannik.glueckert, jstein, navi, randy |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Jannik Glückert
2020-04-02 10:16:26 UTC
Indeed it's wrong. I currently use a workaround and move man directoy and call doman explicitly later in install phase, but it should be fixed. Would it be possible to entirely remove this? Man pages are generally not handled by Cargo, so cargo.eclass trying to install them is rather surprising. yes, it needs to go. it's a historical thing that assumes that such directory structure is standard, which turned out not quite the fact. I'll all in favor for removing it, but first I'll need to find all ebuilds that actually rely on it, and replace with proper doman calls. Can't just make a eclass change that affects results of installation. if you happen to know such ebuilds - let me know =) Otherwise I'll have to emerge all cargo.eclass's cargo_src_install() consumers and check file lists. (In reply to Georgy Yakovlev from comment #3) > Can't just make a eclass change that affects results of installation. Of course. > if you happen to know such ebuilds - let me know =) > > Otherwise I'll have to emerge all cargo.eclass's cargo_src_install() > consumers and check file lists. I only have a few Rust-based packages installed, but maybe this fish shell script helps: ❯ for inherit in /var/db/pkg/*/*/INHERITED grep -q cargo $inherit || continue set -l package (string replace /var/db/pkg/ '' (dirname $inherit)) set -l contents (dirname $inherit)/CONTENTS grep -q '^dir /usr/share/man$' $contents && echo $package end app-benchmarks/hyperfine-1.11.0 app-misc/skim-0.9.3 dev-vcs/mercurial-5.7 gui-libs/greetd-0.7.0 sys-apps/bat-0.17.1-r1 sys-apps/exa-0.9.0-r1 sys-apps/fd-8.2.1 sys-apps/ripgrep-12.1.1 sys-apps/zram-generator-0.3.2 x11-terms/alacritty-0.7.2 I also briefly checked whether https://www.portagefilelist.de could be of help, but they index only files, not directories, and the paths to those files are not searchable. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e24170cba4af14df622c4a2d465534583d30d625 commit e24170cba4af14df622c4a2d465534583d30d625 Author: Georgy Yakovlev <gyakovlev@gentoo.org> AuthorDate: 2021-07-02 00:34:04 +0000 Commit: Georgy Yakovlev <gyakovlev@gentoo.org> CommitDate: 2021-07-03 06:59:09 +0000 cargo.eclass: support EAPI=8, misc changes remove cargo-snapshot* unpacker, it was needed for separate dev-util/cargo we used to have. Bug: https://bugs.gentoo.org/715890 Signed-off-by: Georgy Yakovlev <gyakovlev@gentoo.org> eclass/cargo.eclass | 39 ++++++++++++++++++++++++++------------- 1 file changed, 26 insertions(+), 13 deletions(-) |