Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 613402 - net-analyzer/cacti-spine should alternatively support dev-libs/libressl
Summary: net-analyzer/cacti-spine should alternatively support dev-libs/libressl
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement
Assignee: Gentoo Netmon project
URL:
Whiteboard:
Keywords: NeedPatch
Depends on:
Blocks: libressl-support
  Show dependency tree
 
Reported: 2017-03-21 08:38 UTC by Reuben Farrelly
Modified: 2019-08-28 16:48 UTC (History)
0 users

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


Attachments
Updated ebuild (cacti-spine-1.2.1-r1.ebuild,925 bytes, text/plain)
2019-02-19 12:56 UTC, Reuben Farrelly
Details

Note You need to log in before you can comment on or make changes to this bug.
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(-)