Re: [Ssh-sftp-perl-users] Net::SSH2 on perl 5.6

[hugues <Hug...@af...>]

>>>>> "David" == David Robins <dbr...@cp...> writes:

    David> On Friday February 3, 2006 00:51, hugues wrote:
    >> I just wanted to know if there were some fundamental reason why your
    >> module actually requires perl 5.8 or if there might be some hope to
    >> make it run under perl 5.6 too ?

    David> Nothing particular, just the convenience of not having to jump through hoops 
    David> and include cruft for compatibility.

    David> However, I am aware there are a lot of perl 5.6 installations out there (the 
    David> last place I worked still used it; they brought in a contractor to help with 
    David> upgrading to 5.8 but I don't think they actually did upgrade), and I'm 
    David> willing to make Net::SSH2 work with 5.6, especially if met halfway by someone 
    David> such as yourself helping with testing/changes.

I'm glad to hear that, and will provide any help i'm able to.

    >> I've been trying just the obvious things, i.e removing all 'use 5.008'
    >> in Makefile.PL and all the .pm files, and adding a typemap entry for
    >> 'const char *' as it is defined in the default one for perl5.8 but not
    >> for perl5.6.
    >> 
    >> With that mods, i managed to compile the stuff, but got stuck there: 
    >> 
    >> root-w2# prove -v -b t/Net-SSH2.t  
    David> ...
    >> ok 8 - API date yyyymmddhhmm 
    >> event is not of type AVPtr at blib/lib/Net/SSH2.pm line 396. 

    David> I searched and found something that looks relevant:

    David> http://www.mail-archive.com/pe...@pe.../msg00982.html

    David> Looks like 5.6 doesn't like "AV *" in an XS signature and requires it be typed 
    David> as an SV * and then checked/cast (a typemap entry might work there too; it 
    David> would be nice if the entries added can be 5.6-only (to keep the 5.8 path 
    David> cleaner), which will probably require two typemap files).

Thanks, i'll follow that path and let you know where it led me.

    >> ok 9 - banner is SSH-2.0-libssh2_0.12 
    >> # Looks like you planned 72 tests but only ran 9. 
    >> # Looks like your test died just after 9. 
    >> dubious 
    >> Test returned status 255 (wstat 65280, 0xff00) 
    >> DIED. FAILED tests 10-72 

    David> Looks like it crashed there; if you check the core file you can probably 
    David> determine what happened using a debugger such as gdb or dbx by looking at the 
    David> stack trace and variables.

    >> Before diving any further, i thought i might ask you if there was hope
    >> along that path, or if i'd better put all my strength in getting perl 5.8
    >> installed on my productions servers ...

    David> Sure, there is hope.  I don't have any 5.6 installations myself, though, so 
    David> I'll need you (and any others on the list that are interested) to test and 
    David> provide patches for 5.6 compatibility.  If you're able to do that, great, if 
    David> not, go with the upgrade if you can :).

I'll sure be glad to help at least by testing and maybe even
with providing patches if i can manage that.

    >> Thanks anyway for both maintaining Net::SSH::Perl and providing us
    >> with that new shiny  Net::SSH2 module !

    David> Glad to contribute, thanks for the feedback.

Thank YOU.

-- 
Hugues Lafarge                    || Email: Hug...@af...
Agence France Presse              || Phone: +33 1 40 41 77 15
4 rue de la bourse, 75002 Paris   ||   Fax: +33 1 40 41 79 24
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

This e-mail, and any file transmitted with it, is confidential and  intended solely for the use of the individual or entity to whom it is addressed. If you have received this email in error, please  contact the sender and delete the email from your system. If you are  not the named addressee you should not disseminate, distribute or copy  this email.

For more information on Agence France-Presse, please visit our web site at http://www.afp.com

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-