Skip Menu |

This queue is for tickets about the Net-SSH2 CPAN distribution.

Report information
The Basics
Id: 88265
Status: resolved
Priority: 0/
Queue: Net-SSH2

People
Owner: Nobody in particular
Requestors: jpeacock [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.48
Fixed in: (no value)



Subject: Easier overrides for LIBSSH INC and LIB
--- Makefile.PL.orig 2013-02-20 15:54:35.000000000 -0500 When compiling using non-standard libssh2 paths, especially on Solaris, it is helpful to stack the deck by allowing the caller of Makefile.PL to completely specify all of the linker options. Without this patch, it is difficult/impossible to add rpath to LIBS. +++ Makefile.PL 2013-03-25 14:09:31.000000000 -0400 @@ -4,9 +4,9 @@ use Module::Install::CheckLib (); # override library and include directories here if desired -my $lib = ''; -my $inc = ''; -my $extra_libs = '-lz'; +my $lib = $ENV{LIBSSH_LIBDIR} || ''; +my $inc = $ENV{LIBSSH_INCDIR} || ''; +my $extra_libs = '-lz -lssl -lcrypto '.$ENV{LIBSSH_EXTRA}; my $use_gcrypt = 0; my @args = @ARGV;
Subject: Re: [rt.cpan.org #88265] Easier overrides for LIBSSH INC and LIB
Date: Fri, 30 Aug 2013 12:07:41 -0400
To: bug-Net-SSH2 [...] rt.cpan.org
From: Rafael Kitover <rkitover [...] gmail.com>
Good idea, I'll add that. But I think it should be called LIBSSH2_LIBDIR etc. because 'libssh' is a different library entirely.
On Fri Aug 30 12:07:57 2013, rkitover@gmail.com wrote: Show quoted text
> But I think it should be called LIBSSH2_LIBDIR etc. because 'libssh' > is a different library entirely.
That is totally a reasonable change. I only needed it to work in our environment, so I did as little as possible. ;-) John
0.53 2013-09-01 06:05:17 - add support for ldargs, and LIBSSH2_LIB, LIBSSH2_INCLUDE, LIBSSH2_LDARGS env vars while building (RT#88265)