Bug 333117 – dev-libs/openssl: RDEPEND issues with dev-lang/perl (c_rehash)

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description SpanKY 2010-08-17 04:13:28 UTC]

openssl has a DEPEND on perl because much of the openssl build scripts are written in perl.  this sucks, and i have looked at fixing this, but it's a pretty mountainous task that i havent had time/dedication for completing.

the focus though has been that it was a build time only requirement.  turns out, there is a semi-important c_rehash script installed that is written in perl.  it shouldnt be hard to re-implement this with standard POSIX shell though.

(yes, there is also /etc/ssl/misc/CA.pl, but there is a parallel /etc/ssl/misc/CA.sh already installed, so i think we can ignore it)

[Comment 1 Eray Aslan 2010-08-17 14:40:12 UTC]

Is the following enough for you to reconsider?  DEPEND is bad enough.

https://bugs.gentoo.org/show_bug.cgi?id=333069#c6

[Comment 2 Eray Aslan 2010-08-17 20:03:11 UTC]

On second thought, not enough justification to refuse RDEPEND.  It will just complicate user experience.  Sorry for the bug spam.

[Comment 3 SpanKY 2010-08-17 20:11:45 UTC]

your point doesnt really make a difference to this bug report.  openssl cannot omit perl from RDEPEND while installing c_rehash.  whether the ebuild sometimes runs c_rehash makes no difference.

two mutually exclusive options:
 - always add perl to RDEPEND
 - rewrite c_rehash in posix shell

obviously i lean towards doing the latter

[Comment 4 Eray Aslan 2010-08-18 05:04:21 UTC]

Agreed.

There was some work to make a shell alternative to c_rehash (named c_rehash.sh I believe).  I'll see if I can find it.  But even if I did, it is probably better to use the latest perl version from the openssl tarball.

[Comment 5 SpanKY 2010-08-19 21:21:02 UTC]

if people have USE=perl, then that is one thing.  but if they dont, then we'll need a pure POSIX shell variant.  and perhaps we just push this back upstream.

so once you find that rewritten code, please post it here so i dont have to rewrite it all from scratch.

[Comment 6 Eray Aslan 2010-08-19 21:34:58 UTC]

http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/openssl/openssl-c_rehash.sh

[Comment 7 SpanKY 2010-08-19 22:33:19 UTC]

thanks ... ive imported that into openssl-1.0.0a-r2

[Comment 8 Saleh Alsanad 2010-09-29 16:31:14 UTC]

btw, emerging the latest version of openssl (openssl-1.0.0a-r3) will fail since there's a slight bug in the ebuild script which I've fixed by this:

--- openssl-1.0.0a-r3.ebuild.old        2010-09-29 19:20:02.000000000 +0307
+++ openssl-1.0.0a-r3.ebuild    2010-09-29 19:20:19.000000000 +0307
@@ -7,7 +7,7 @@
 DESCRIPTION="full-strength general purpose cryptography library (including SSL v2/v3 and TLS v1)"
 HOMEPAGE="http://www.openssl.org/"
 SRC_URI="mirror://openssl/source/${P}.tar.gz
-       http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/~checkout~/packages/${PN}/${PN}-c_rehash.sh%3frev=1.7"
+       http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/~checkout~/packages/${PN}/${PN}-c_rehash.sh?rev=1.7"
 
 LICENSE="openssl"
 SLOT="0"

then, regenerate the Manifest file by this:

ebuild openssl-1.0.0a-r3.ebuild manifest

Thanks,
Saleh

[Comment 9 SpanKY 2010-09-29 16:54:47 UTC]

file new bugs for new issues