Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 613402

Summary: net-analyzer/cacti-spine should alternatively support dev-libs/libressl
Product: Gentoo Linux Reporter: Reuben Farrelly <reuben-gentoo-bugzilla>
Component: Current packagesAssignee: Gentoo Netmon project <netmon>
Status: RESOLVED FIXED    
Severity: enhancement Keywords: NeedPatch
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 561854    
Attachments: Updated ebuild

Description Reuben Farrelly 2017-03-21 08:38:08 UTC
All of the ebuilds for this package have openssl hard set as a CDEPEND which seems to be incorrect.  spine does build with LibreSSL, so rather than forcing OpenSSL to be installed we should allow either.

It seems from the documentation that crypto support is required, and after testing it appears that LibreSSL does in fact satisfy this and work just fine.  So we should really make the CDEPEND something like ( openssl:* || libressl:* )
Comment 1 Anthony Basile gentoo-dev 2017-03-21 13:37:11 UTC
(In reply to Reuben Farrelly from comment #0)
> All of the ebuilds for this package have openssl hard set as a CDEPEND which
> seems to be incorrect.  spine does build with LibreSSL, so rather than
> forcing OpenSSL to be installed we should allow either.
> 
> It seems from the documentation that crypto support is required, and after
> testing it appears that LibreSSL does in fact satisfy this and work just
> fine.  So we should really make the CDEPEND something like ( openssl:* ||
> libressl:* )

I don't know the package.  Does it depend on the libressl/openssl libraries?  If so, then we need libressl:0= or similar.
Comment 2 Reuben Farrelly 2017-06-19 23:20:25 UTC
Yes - this package depends only the libraries and not on the openssl binary file.

There is a single binary ELF executable output from the ebuild along with a txt config file and txt Changelog.
Comment 3 Reuben Farrelly 2019-02-19 12:56:05 UTC
Created attachment 565854 [details]
Updated ebuild

ebuild cacti-spine-1.2.1-r1.  Adds libressl support, updates copyright year and notice.
Comment 4 Larry the Git Cow gentoo-dev 2019-08-28 16:48:38 UTC
The bug has been closed via the following commit(s):

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

commit b704d0f8ccfe2d4554616aae3dc9258b86051e6f
Author:     Stefan Strogin <steils@gentoo.org>
AuthorDate: 2019-08-28 16:43:07 +0000
Commit:     Stefan Strogin <steils@gentoo.org>
CommitDate: 2019-08-28 16:48:27 +0000

    net-analyzer/cacti-spine: add USE=libressl
    
    Reported-by: Reuben Farrelly <reuben-gentoo-bugzilla@reub.net>
    Closes: https://bugs.gentoo.org/613402
    Package-Manager: Portage-2.3.73, Repoman-2.3.17
    Signed-off-by: Stefan Strogin <steils@gentoo.org>

 net-analyzer/cacti-spine/cacti-spine-1.2.5-r1.ebuild | 4 +++-
 net-analyzer/cacti-spine/cacti-spine-99999.ebuild    | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)