[Net-ssleay-devel] Testers for svn trunk, please.

[Mike McCauley]

Hello Florian,

Sorry for the delay in replying... I was travelling.

On Thursday 20 July 2006 22:05, Florian Ragwitz wrote:
> Hello there,
>
> I've done some work on Net::SSLeay in the last days and would like to
> hear some feedback. The most important changes include:
>
> * Use Module::Install instead of ExtUtils::MakeMaker.

Not sure I like this one: Module::Install does not seem to be a standard 
module, so it becomes another barrier to installing. It has some extensive 
prerequisites, too.

What value does this add?

Anyway, After installing Module::Install so I can make progress in the 
meantime, when I make, I see:

mikem at zulu:/usr/local/projects/net-ssleay/trunk> make
/usr/bin/perl "-Iinc" /usr/lib/perl5/5.8.7/ExtUtils/xsubpp  
-typemap /usr/lib/perl5/5.8.7/ExtUtils/typemap -typemap typemap  SSLeay.xs > 
SSLeay.xsc && mv SSLeay.xsc SSLeay.c
Error: Cannot parse ALIAS definitions from 'SSL *              s' in 
SSLeay.xs, line 835
make: *** [SSLeay.c] Error 1

mikem at zulu:/usr/local/projects/net-ssleay/trunk> perl -v

This is perl, v5.8.7 built for i586-linux-thread-multi
(with 1 registered patch, see perl -V for more detail)

Not sure what is supposed to happen with this, so I will leave this one to 
you.


> * Remove one callback per ctx/ssl limitation of SSL_set_verify and
>   SSL_CTX_set_verify.
> * Remove one callback per context limitation of
>   SSL_CTX_set_default_passwd_cb.
> * Fully port the old test suite to t/ using Test::More and remove
>   test.pl.
> * Modify set_{,r,w}fd to take either a fileno or a perl io handle as its
>   second argument. Win32 has some special handling of those functions
>   which I didn't touch, so review of the code on win32 would be very
>   welcome.
> * Bindings for MD2 and MD4.
> * Don't use IO::Select in t/.
> * Split up local and external tests.
> * Some minor fixes/improvements.

Shall test these when I get further with the issue above.

>
> Also the new test suite uses warnings.pm and probably other stuff which
> isn't available in perl pre-5.6. Would you consider that an issue?

Yes.

Cheers.

>
>
> Thanks in advance,
> Flo

-- 
Mike McCauley                               mikem at open.com.au
Open System Consultants Pty. Ltd            Unix, Perl, Motif, C++, WWW
9 Bulbul Place Currumbin Waters QLD 4223 Australia   http://www.open.com.au
Phone +61 7 5598-7474                       Fax   +61 7 5598-7070

Radiator: the most portable, flexible and configurable RADIUS server 
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, 
Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, 
TTLS, PEAP etc on Unix, Windows, MacOS, NetWare etc.