Bug #78848 for Crypt-SSLeay: Allow CPATH and LIBRARY_PATH to be specificed on the command line

Wed Aug 08 07:58:15 2012 nanis [...] runu.moc.invalid - Ticket created

Subject:

Allow CPATH and LIBRARY_PATH to be specificed on the command line

Add two command line options --include and --library to point to specific header file and library locations. -- Sinan

Fri Jan 18 16:44:01 2013 quanah.gibsonmount [...] gmail.com - Correspondence added

This is really bad. It used to be trivially easy to build Crypt::SSLeay against a custom OpenSSL install location. This is now completely broken.

Fri Jan 18 16:44:02 2013 The RT System itself - Status changed from 'new' to 'open'

Tue Jan 22 15:54:33 2013 quanah.gibsonmount [...] gmail.com - Correspondence added

On Fri Jan 18 16:44:01 2013, MISHIKAL wrote: Show quoted text

> This is really bad. It used to be trivially easy to build Crypt::SSLeay > against a custom OpenSSL install location. This is now completely broken.

What this needs, is to add the INC variable, just like Crypt::OpenSSL::Random does, so that the path to the OpenSSL headers can be specified. I can work around the library portion by setting LIBS to point to my libraries. However, there is no corresponding INC variable to set where to find the headers.

Tue Jan 22 15:56:57 2013 quanah.gibsonmount [...] gmail.com - Correspondence added

On Tue Jan 22 15:54:33 2013, MISHIKAL wrote: Show quoted text

> What this needs, is to add the INC variable, just like > Crypt::OpenSSL::Random does, so that the path to the OpenSSL headers can > be specified. I can work around the library portion by setting LIBS to > point to my libraries. However, there is no corresponding INC variable > to set where to find the headers.

--- Crypt-SSLeay-0.64/Makefile.PL.orig 2013-01-22 12:55:32.000000000 -0800 +++ Crypt-SSLeay-0.64/Makefile.PL 2013-01-22 12:55:54.000000000 -0800 @@ -40,6 +40,8 @@ LIBS => [q{-lz -lssl -lcrypto -lssl32 -lssleay32 -leay32}], + 'INC' => '', # e.g., '-I/usr/include/other' + ($opt_static ? (LINK_TYPE => 'static') : ()), BUILD_REQUIRES => {

Tue Jan 22 16:03:06 2013 quanah.gibsonmount [...] gmail.com - Correspondence added

On Tue Jan 22 15:56:57 2013, MISHIKAL wrote: Show quoted text

> On Tue Jan 22 15:54:33 2013, MISHIKAL wrote:

> > What this needs, is to add the INC variable, just like > > Crypt::OpenSSL::Random does, so that the path to the OpenSSL headers can > > be specified. I can work around the library portion by setting LIBS to > > point to my libraries. However, there is no corresponding INC variable > > to set where to find the headers.

> > --- Crypt-SSLeay-0.64/Makefile.PL.orig 2013-01-22 12:55:32.000000000

-0800 Show quoted text

> +++ Crypt-SSLeay-0.64/Makefile.PL 2013-01-22 12:55:54.000000000

-0800 Show quoted text

> @@ -40,6 +40,8 @@ > > LIBS => [q{-lz -lssl -lcrypto -lssl32 -lssleay32 -leay32}], > > + 'INC' => '', # e.g., '-I/usr/include/other' > + > ($opt_static ? (LINK_TYPE => 'static') : ()), > > BUILD_REQUIRES => { > >

With the above patch, I can once again link against a custom openssl installation.

Tue Apr 09 08:21:43 2013 MGI [...] cpan.org - Correspondence added

To build with my custom openssl installation, I had to set the INC environment variable, to fix the -L directives in the Makefile (from non-existing /usr/local/lib on my box), and to add -lssl -lcrypto to EXTRALIBS and LDLOADLIBS. The config spec did not find them from my $LD_LIBRARY_PATH, and reported them as missing. Thanks

Thu Apr 10 09:58:06 2014 nanis [...] runu.moc.invalid - Correspondence added

Hello: Thank you for your report. I recently uploaded 0.65_08 to CPAN and it should now handle custom locations better. Once this goes through CPAN testers, I am going to update the documentation and upload 0.66. Best, -- Sinan On Tue Apr 09 08:21:43 2013, MGI wrote: Show quoted text

> To build with my custom openssl installation, I had to set the INC > environment variable, to fix the -L directives in the Makefile (from > non-existing /usr/local/lib on my box), and to add -lssl -lcrypto to > EXTRALIBS and LDLOADLIBS. > The config spec did not find them from my $LD_LIBRARY_PATH, and > reported them as missing. > Thanks

Thu Apr 10 09:58:08 2014 nanis [...] runu.moc.invalid - Status changed from 'open' to 'patched'

Thu Apr 24 19:27:23 2014 nanis [...] runu.moc.invalid - Status changed from 'patched' to 'resolved'

Thu Apr 24 19:27:23 2014 nanis [...] runu.moc.invalid - Fixed in 0.72 added