Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 675744 - sys-cluster/ceph-13.2.4 should not depend on net-libs/nodejs with -DWITH_MGR_DASHBOARD_FRONTEND=NO
Summary: sys-cluster/ceph-13.2.4 should not depend on net-libs/nodejs with -DWITH_MGR_...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Patrick McLean
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-17 22:17 UTC by Brad Laue
Modified: 2019-03-15 21:18 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 Brad Laue 2019-01-17 22:17:19 UTC
The current ebuild for sys-cluster/ceph-13.2.4 has a dependency on specific nodejs versions which I propose is not necessary in its current form. NodeJS would be a necessary dependency if the manager web UI were being built, but the ebuild specifies -DWITH_MGR_DASHBOARD_FRONTEND=NO in the "mycmakeargs" variable passed to cmake.

The build succeeds and ceph-mgr functions (without a web GUI) as expected with the attached patch. If -DWITH_MGR_DASHBOARD_FRONTEND=YES is set, the build does fail. The thrust of this bug report is to suggest the nodejs min/max version requirement be removed, or placed into the "mgr-frontend" USE flag if that's ever restored.

The chief difficulty is the fact that NodeJS isn't slotted and other things (like Chromium) pull in a version that is newer than that which Ceph requires (installing both on a devops workstation gets hairy as a result).
Comment 1 Larry the Git Cow gentoo-dev 2019-03-15 21:18:48 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=13a2f5a373fd291cda6031b3052b583f77ac741e

commit 13a2f5a373fd291cda6031b3052b583f77ac741e
Author:     Patrick McLean <patrick.mclean@sony.com>
AuthorDate: 2019-03-15 21:18:25 +0000
Commit:     Patrick McLean <chutzpah@gentoo.org>
CommitDate: 2019-03-15 21:18:38 +0000

    sys-cluster/ceph: Drop nodejs dep from 13.2.4  (bug #675744)
    
    Closes: https://bugs.gentoo.org/675744
    Copyright: Sony Interactive Entertainment Inc.
    Package-Manager: Portage-2.3.62, Repoman-2.3.12
    Signed-off-by: Patrick McLean <chutzpah@gentoo.org>

 sys-cluster/ceph/ceph-13.2.4.ebuild | 4 ----
 1 file changed, 4 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e9f26758cdc285372757707d8e0db28475f91ac

commit 0e9f26758cdc285372757707d8e0db28475f91ac
Author:     Patrick McLean <patrick.mclean@sony.com>
AuthorDate: 2019-03-15 21:15:22 +0000
Commit:     Patrick McLean <chutzpah@gentoo.org>
CommitDate: 2019-03-15 21:18:36 +0000

    sys-cluster/ceph: Version bump to 13.2.5
    
    Remove dependency on nodejs (bug #675744)
    
    Bug: https://bugs.gentoo.org/675744
    Copyright: Sony Interactive Entertainment Inc.
    Package-Manager: Portage-2.3.62, Repoman-2.3.12
    Signed-off-by: Patrick McLean <chutzpah@gentoo.org>

 sys-cluster/ceph/Manifest           |   1 +
 sys-cluster/ceph/ceph-13.2.5.ebuild | 322 ++++++++++++++++++++++++++++++++++++
 2 files changed, 323 insertions(+)