Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 929195 - dev-tex/rubber-1.6.4-r1 installs files into unexpected paths: /usr/share/doc/rubber-1.6.4
Summary: dev-tex/rubber-1.6.4-r1 installs files into unexpected paths: /usr/share/doc/...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Florian Schmaus
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-04-11 22:18 UTC by Agostino Sarubbo
Modified: 2024-04-12 18:47 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,53.21 KB, text/plain)
2024-04-11 22:18 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2024-04-11 22:18:21 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: dev-tex/rubber-1.6.4-r1 installs files into unexpected paths.
Discovered on: amd64 (internal ref: ci)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#QA0025
Comment 1 Agostino Sarubbo gentoo-dev 2024-04-11 22:18:22 UTC
Created attachment 890472 [details]
build.log

build log and emerge --info
Comment 2 Sebastian Pipping gentoo-dev 2024-04-11 22:41:12 UTC
I saw the message but it's (1) surpising and (2) not accurate.

For (1), /usr/share/doc/${PF} seems to be just right.

For (2) It's not ${P} but ${PF} with -r1 in practice:

  # qlist rubber | grep doc
  /usr/share/doc/rubber-1.6.4-r1/rubber.txt.bz2
  /usr/share/doc/rubber-1.6.4-r1/rubber.texi.bz2
  /usr/share/doc/rubber-1.6.4-r1/rubber.info.bz2
  /usr/share/doc/rubber-1.6.4-r1/NEWS.bz2
  /usr/share/doc/rubber-1.6.4-r1/README.md.bz2
  /usr/share/doc/rubber-1.6.4-r1/rubber.pdf
  /usr/share/doc/rubber-1.6.4-r1/rubber.html
  
Please help me understand what to do and why change is needed.  Is this a broken QA check?
Comment 3 Agostino Sarubbo gentoo-dev 2024-04-12 06:38:18 UTC
(In reply to Sebastian Pipping from comment #2)
> Please help me understand what to do and why change is needed.  Is this a
> broken QA check?

CC'in dev-portage@ to have a feedback
Comment 4 Florian Schmaus gentoo-dev 2024-04-12 08:27:18 UTC
Adding set -x to 08gentoo-pahts (from ::gentoo's metadata/install-qa-check.d):


++ doc_dirs=('/mnt/data-scratch/var-tmp/portage/dev-tex/rubber-1.6.4-r1/image/usr/share/doc/rubber-1.6.4' '/mnt/data-scratch/var-tmp/portage/dev-tex/rubber-1.6.4-r1/image/usr/share/doc/rubber-1.6.4-r1')
++ local doc_dirs
++ for x in "${doc_dirs[@]##*/}"
++ [[ rubber-1.6.4 != rubber-1.6.4-r1 ]]
++ bad_paths+=("/usr/share/doc/${x}")
++ for x in "${doc_dirs[@]##*/}"
++ [[ rubber-1.6.4-r1 != rubber-1.6.4-r1 ]]

So there appears to be /usr/share/doc/rubber-1.6.4 *and* /usr/share/doc/rubber-1.6.4-r1 in ED.

However,

$ ls -l /mnt/data-scratch/var-tmp/portage/dev-tex/rubber-1.6.4-r1/image/usr/share/doc/
total 0
drwxr-xr-x 2 flo users 0 Apr 12 10:07 rubber-1.6.4-r1
Comment 5 Larry the Git Cow gentoo-dev 2024-04-12 08:35:22 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=45869eeea51a81c30016917e073cbaef4d507336

commit 45869eeea51a81c30016917e073cbaef4d507336
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2024-04-12 08:34:44 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2024-04-12 08:35:13 +0000

    dev-tex/rubber: remove unnecessary dodir
    
    Closes: https://bugs.gentoo.org/929195
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 dev-tex/rubber/rubber-1.6.4-r1.ebuild | 1 -
 1 file changed, 1 deletion(-)
Comment 6 Sebastian Pipping gentoo-dev 2024-04-12 18:47:02 UTC
Oh damn, thanks for the help finding and fixing this!