Skip Menu |
 

This queue is for tickets about the Crypt-SSLeay CPAN distribution.

Report information
The Basics
Id: 49285
Status: resolved
Priority: 0/
Queue: Crypt-SSLeay

People
Owner: nanis [...] runu.moc.invalid
Requestors: kmx [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.57_03



Subject: Improved OpenSSL detection on Win32/strawberry perl
Download (untitled) / with headers
text/plain 412b
Hi, could you please consider the enclosed patch for Makefile.PL that improves autodetection of openssl installation on Win32? This patch is focused on Win32/strawberry as we are about to add openssl support to the next (October2009) strawberry release (openssl will be installed directly with strawberry). The patch is purely about openssl detection - the module itself works nice even now. Thanks. -- kmx
Subject: cryptssleay.diff
Download cryptssleay.diff
text/x-diff 561b
--- perl-modules\Crypt-SSLeay-0.57_01/Makefile.PL 2008-02-18 14:39:06.000000000 +0100 +++ perl-modules-patched\Crypt-SSLeay-0.57_01/Makefile.PL 2009-08-31 15:48:36.379715800 +0200 @@ -29,7 +29,9 @@ $opt_default = 1; } elsif ($^O eq 'MSWin32') { - @POSSIBLE_SSL_DIRS = 'c:\\openssl'; + push @POSSIBLE_SSL_DIRS, 'c:\\openssl'; + push @POSSIBLE_SSL_DIRS, $Config{prefix}, # strawberry perl + push @POSSIBLE_SSL_DIRS, $Config{prefix}.'\..\c', # strawberry perl } elsif ($^O eq 'VMS') { @POSSIBLE_SSL_DIRS = '/ssl$root';
Download (untitled) / with headers
text/plain 668b
On Mon Aug 31 10:27:28 2009, KMX wrote: Show quoted text
> Hi, > > could you please consider the enclosed patch for Makefile.PL that > improves autodetection of openssl installation on Win32? > > This patch is focused on Win32/strawberry as we are about to add openssl > support to the next (October2009) strawberry release (openssl will be > installed directly with strawberry). > > The patch is purely about openssl detection - the module itself works > nice even now. > > Thanks.
That's nice, thanks. I'll fold that into the next release. If you want to be credited with a human's name, drop me a line here or at david@landgren.net and I'll add it to the readme. Thanks, David
Download (untitled) / with headers
text/plain 629b
Hi, I have one little enhancement to this patch - could you please change the Makefile.PL section like this (one more line with $Config{usrinc}): elsif ($^O eq 'MSWin32') { push @POSSIBLE_SSL_DIRS, 'c:\\openssl'; push @POSSIBLE_SSL_DIRS, $Config{usrinc} if (-d $Config{usrinc}); push @POSSIBLE_SSL_DIRS, $Config{prefix}, # strawberry perl push @POSSIBLE_SSL_DIRS, $Config{prefix}.'\..\c', # strawberry perl } Since Strawberry Oct2009 we are using $Config{usrinc} value as a preferred way of getting the compilers's include directory. The other two I would keep for pre-Oct2009 strawberry. Thanks. -- kmx
Subject: Re: [rt.cpan.org #49285] Improved OpenSSL detection on Win32/strawberry perl
Date: Mon, 23 Nov 2009 21:08:47 +0100
To: bug-Crypt-SSLeay [...] rt.cpan.org
From: David Landgren <david [...] landgren.net>
Download (untitled) / with headers
text/plain 813b
kmx via RT a écrit : Show quoted text
> Queue: Crypt-SSLeay > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=49285 > > > Hi, > > I have one little enhancement to this patch - could you please change > the Makefile.PL section like this (one more line with $Config{usrinc}): > > elsif ($^O eq 'MSWin32') { > push @POSSIBLE_SSL_DIRS, 'c:\\openssl'; > push @POSSIBLE_SSL_DIRS, $Config{usrinc} if (-d $Config{usrinc}); > push @POSSIBLE_SSL_DIRS, $Config{prefix}, # strawberry perl > push @POSSIBLE_SSL_DIRS, $Config{prefix}.'\..\c', # strawberry perl > } > > Since Strawberry Oct2009 we are using $Config{usrinc} value as a > preferred way of getting the compilers's include directory. The other > two I would keep for pre-Oct2009 strawberry. > > Thanks.
I'll get onto this, thanks. David
Just a note that latest dev version 0.57_02 (which I have just tested in relation to RT 52408) does not contain this fix.

--
kmx

Download (untitled) / with headers
text/plain 301b
Hello: Yup. I am going to include this with a couple of other fixes in 0.57_03. That should happen by Aug 11. Sinan On Sun Aug 08 16:42:13 2010, KMX wrote: Show quoted text
> Just a note that latest dev version 0.57_02 (which I have just tested in > relation to RT 52408) does not contain this fix. > > -- > kmx
Dne Ne 08.srp.2010 17:32:04, NANIS napsal(a):
Show quoted text
> Hello:
>
> Yup. I am going to include this with a couple of other fixes in 0.57_03.
> That should happen by Aug 11.

That's great.

Download (untitled) / with headers
text/plain 414b
I just uploaded 0.57_03. I would appreciate it if you can check if everything still works as expected. http://search.cpan.org/~nanis/Crypt-SSLeay-0.57_03/ Thank you. -- Sinan On Sun Aug 08 17:47:29 2010, KMX wrote: Show quoted text
> Dne Ne 08.srp.2010 17:32:04, NANIS napsal(a):
> > Hello: > > > > Yup. I am going to include this with a couple of other fixes in 0.57_03. > > That should happen by Aug 11.
> > That's great.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.