Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 894600 - net-libs/xrootd-ceph-5.5.2 installs less files with FEATURES=test
Summary: net-libs/xrootd-ceph-5.5.2 installs less files with FEATURES=test
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Guilherme Amadio
URL:
Whiteboard:
Keywords: PMASKED
Depends on:
Blocks:
 
Reported: 2023-02-15 15:40 UTC by Agostino Sarubbo
Modified: 2023-12-15 21:12 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2023-02-15 15:40:06 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: net-libs/xrootd-ceph-5.5.2 installs less files with FEATURES=test.
Discovered on: amd64 (internal ref: ci)

NOTE:
To reproduce this issue:
1) Install the package with FEATURES=test
2) Put the list of the installed files into a file ( qlist -e PACKAGE:SLOT | sort | uniq > /tmp/qlist_test )
3) Install the package without FEATURES=test
4) Put the list of the installed files into a file ( qlist -e PACKAGE:SLOT | sort | uniq > /tmp/qlist_notest )
5) See the diff between the two files ( diff -ru /tmp/qlist_test /tmp/qlist_notest )

On this machine:
- the installed files with FEATURES=test are 7
- the installed files without FEATURES=test are 6

This type of report comes from a check that is a bit outside of the tinderbox, that's the reason because build.log was not provided (however in cases like these the build.log doesn't help much) but if you need further help to diagnose this issue feel free to ask.

In any case I'm providing a diff.
Comment 1 Agostino Sarubbo gentoo-dev 2023-02-15 15:40:24 UTC
DIFF:
--- /dev/fd/63  2023-02-15 16:32:14.333395781 +0100
+++ /dev/fd/62  2023-02-15 16:32:14.333395781 +0100
@@ -2,5 +2,6 @@
 /usr/lib64/libXrdCephPosix.so
 /usr/lib64/libXrdCephPosix.so.0
 /usr/lib64/libXrdCephPosix.so.0.0.1
+/usr/lib64/libXrdCephTests.so
 /usr/lib64/libXrdCephXattr-5.so
 /usr/share/doc/xrootd-ceph-5.5.2/README.bz2
Comment 2 Guilherme Amadio gentoo-dev 2023-02-15 16:07:07 UTC
Hi ago, the file /usr/lib64/libXrdCephTests.so is only installed with FEATURES=test, which is the expected behavior. I think the values for number of files for with/without FEATURES=test may be inverted.
Comment 3 Agostino Sarubbo gentoo-dev 2023-02-15 16:16:22 UTC
(In reply to Guilherme Amadio from comment #2)
> Hi ago, the file /usr/lib64/libXrdCephTests.so is only installed with
> FEATURES=test, 

Hi, that's exactly the point. FEATURES should not decide about the installed files
Comment 4 Guilherme Amadio gentoo-dev 2023-02-15 16:28:48 UTC
I will modify the ebuild not to install the test library, but the library is just a plugin for the test runner, which is part of net-libs/xrootd (xrootd-ceph was part of it until recently too). This is the intended behavior upstream, but I just joined the XRootD project as a developer and my plan is to migrate it from CppUnit to GoogleTest and get rid of these libraries in a future release.
Comment 5 Larry the Git Cow gentoo-dev 2023-12-15 21:12:33 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4750239d11d7ad91081557acec6747aa196e1d62

commit 4750239d11d7ad91081557acec6747aa196e1d62
Author:     Arthur Zamarin <arthurzam@gentoo.org>
AuthorDate: 2023-12-15 21:11:50 +0000
Commit:     Arthur Zamarin <arthurzam@gentoo.org>
CommitDate: 2023-12-15 21:11:50 +0000

    net-libs/xrootd-ceph: treeclean
    
    Closes: https://bugs.gentoo.org/894600
    Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>

 net-libs/xrootd-ceph/Manifest                      |  2 --
 .../files/xrootd-ceph-5.4.0_no-werror.patch        | 11 -------
 net-libs/xrootd-ceph/metadata.xml                  | 16 ---------
 net-libs/xrootd-ceph/xrootd-ceph-5.5.4.ebuild      | 38 ----------------------
 net-libs/xrootd-ceph/xrootd-ceph-5.5.5.ebuild      | 38 ----------------------
 profiles/arch/amd64/x32/package.mask               |  5 ---
 profiles/package.mask                              |  7 ----
 7 files changed, 117 deletions(-)