Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 729358 - net-fs/cvmfs-2.7.2 should create cvmfs user and should depend on dev-cpp/gtest
Summary: net-fs/cvmfs-2.7.2 should create cvmfs user and should depend on dev-cpp/gtest
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Guilherme Amadio
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-23 21:58 UTC by Paul Preney
Modified: 2020-06-26 13:20 UTC (History)
2 users (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 Paul Preney 2020-06-23 21:58:35 UTC
For CVMFS to actually work a cvmfs user needs to be created, e.g., something like this:

    useradd -r -s /bin/false -d /var/lib/cvmfs cvmfs

This is not in the ebuild and should be. 

With the other recent ebuild updates that have been added almost everything is okay (per my tests/use of cvmfs) except this dependency is missing:

    dev-cpp/gtest

i.e., compilation fails when it is not installed (due to trying to #include a file from gtest).
Comment 1 Guilherme Amadio gentoo-dev 2020-06-26 11:02:07 UTC
I missed the dependency on GTest, but Jakob (author of cvmfs) did point it out to me later that it's needed. I will try to patch it out rather than add the dependency to the ebuild. As for the user/group, I will create the appropriate ebuilds, acct-{user,group}/cvmfs as well.
Comment 2 Larry the Git Cow gentoo-dev 2020-06-26 13:16:43 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8cce53fd4cc1a2d0808f1cbcf5142980cb3107c7

commit 8cce53fd4cc1a2d0808f1cbcf5142980cb3107c7
Author:     Guilherme Amadio <amadio@gentoo.org>
AuthorDate: 2020-06-26 13:11:38 +0000
Commit:     Guilherme Amadio <amadio@gentoo.org>
CommitDate: 2020-06-26 13:15:38 +0000

    net-fs/cvmfs: add user/group and gtest as deps, fix bug #729358
    
    Closes: https://bugs.gentoo.org/729358
    
    Package-Manager: Portage-2.3.103, Repoman-2.3.23
    Signed-off-by: Guilherme Amadio <amadio@gentoo.org>

 net-fs/cvmfs/{cvmfs-2.7.2.ebuild => cvmfs-2.7.2-r1.ebuild} | 3 +++
 1 file changed, 3 insertions(+)
Comment 3 Guilherme Amadio gentoo-dev 2020-06-26 13:20:57 UTC
After looking into the code, removing the GTest includes would be a major hassle, so I just added the dependency for now.