Skip Menu |

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

Report information
The Basics
Id: 116570
Status: open
Priority: 0/
Queue: Crypt-OpenSSL-ECDSA

People
Owner: Nobody in particular
Requestors: Bill.Sirinek [...] theice.com
Cc:
AdminCc:

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



Subject: potentially unnecessary dependency in Crypt-OpenSSL-ECDSA
Date: Fri, 29 Jul 2016 19:07:06 +0000
To: "'bug-Crypt-OpenSSL-ECDSA [...] rt.cpan.org'" <bug-Crypt-OpenSSL-ECDSA [...] rt.cpan.org>
From: Bill Sirinek <Bill.Sirinek [...] theice.com>
It appears that the makefile for Crypt::OpenSSL::ECDSA forces the use of perl 5.18 or newer. This is not realistic for most enterprise users. Even the latest RHEL 7 uses perl 5.16. I changed the Makefile.PL file to say "use 5.016001" instead of 5.018001 and the module built, tested and installed with no warnings or errors. I'm not even sure 5.16 is required, you might consider removing the statement entirely unless there's some feature only available in a more recent version of perl that this module uses that I am not aware of. I have copied and pasted the output of perl -v as well as the build and test output. I did not do a make install because I converted this to an rpm file with the cpan2rpm software and installed the rpm. I installed the resulting RPM and it seems to work okay. Thanks Bill bsirinek@myhost $ perl -v This is perl 5, version 16, subversion 3 (v5.16.3) built for x86_64-linux-thread-multi (with 28 registered patches, see perl -V for more detail) Copyright 1987-2012, Larry Wall Perl may be copied only under the terms of either the Artistic License or the GNU General Public License, which may be found in the Perl 5 source kit. Complete documentation for Perl, including FAQ lists, should be found on this system using "man perl" or "perldoc perl". If you have access to the Internet, point your browser at http://www.perl.org/, the Perl Home Page. bsirinek@myhost $ head Makefile.PL use 5.016001; use ExtUtils::MakeMaker; # You might need to set this if your openssl headers and libs anre in an unusual place: my $openssl_prefix = $ENV{OPENSSL_PREFIX} || '/usr'; # See lib/ExtUtils/MakeMaker.pm for details of how to influence # the contents of the Makefile that is written. WriteMakefile( NAME => 'Crypt::OpenSSL::ECDSA', bsirinek@myhost $ perl Makefile.PL Checking if your kit is complete... Looks good Writing Makefile for Crypt::OpenSSL::ECDSA Writing MYMETA.yml and MYMETA.json bsirinek@myhost $ make cp lib/Crypt/OpenSSL/ECDSA.pm blib/lib/Crypt/OpenSSL/ECDSA.pm AutoSplitting blib/lib/Crypt/OpenSSL/ECDSA.pm (blib/lib/auto/Crypt/OpenSSL/ECDSA) /usr/bin/perl /usr/share/perl5/vendor_perl/ExtUtils/xsubpp -typemap /usr/share/perl5/ExtUtils/typemap -typemap typemap ECDSA.xs > ECDSA.xsc && mv ECDSA.xsc ECDSA.c gcc -c -I. -I/usr/include -D_REENTRANT -D_GNU_SOURCE -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic -DVERSION=\"0.07\" -DXS_VERSION=\"0.07\" -fPIC "-I/usr/lib64/perl5/CORE" -DPERL5 -DOPENSSL_NO_KRB5 ECDSA.c Running Mkbootstrap for Crypt::OpenSSL::ECDSA () chmod 644 ECDSA.bs rm -f blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so gcc -shared -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic -Wl,-z,relro ECDSA.o -o blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so \ -L/usr/lib -lcrypto \ chmod 755 blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so cp ECDSA.bs blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.bs chmod 644 blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.bs Manifying blib/man3/Crypt::OpenSSL::ECDSA.3pm bsirinek@myhost $ make test PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/Crypt-OpenSSL-ECDSA.t .. ok All tests successful. Files=1, Tests=21, 0 wallclock secs ( 0.03 usr 0.00 sys + 0.03 cusr 0.00 csys = 0.06 CPU) Result: PASS bsirinek@myhost $ Bill Sirinek - Senior Systems Engineer Intercontinental Exchange | ICE 5660 New Northside Drive NW | 3rd Floor | Atlanta, GA 30328 Tel: 770.857.2468 | Mobile: 770.851.9902 | Fax: 770.857.0010 bill.sirinek@theice.com<mailto:bill.sirinek@theice.com> Show quoted text
________________________________ This message may contain confidential information and is intended for specific recipients unless explicitly noted otherwise. If you have reason to believe you are not an intended recipient of this message, please delete it and notify the sender. This message may not represent the opinion of Intercontinental Exchange, Inc. (ICE), its subsidiaries or affiliates, and does not constitute a contract or guarantee. Unencrypted electronic mail is not secure and the recipient of this message is expected to provide safeguards from viruses and pursue alternate means of communication where privacy or a binding message is desired.
Subject: Re: [rt.cpan.org #116570] potentially unnecessary dependency in Crypt-OpenSSL-ECDSA
Date: Sat, 30 Jul 2016 10:55:04 +1000
To: bug-Crypt-OpenSSL-ECDSA [...] rt.cpan.org
From: Mike McCauley <mikem [...] airspayce.com>
Hi Bill, thanks for reporting this. Quite right. I have now uploaded to CPAN version 0.08 with more lenient requirements. Cheers. On Friday, July 29, 2016 03:07:33 PM Bill Sirinek via RT wrote: Show quoted text
> Fri Jul 29 15:07:27 2016: Request 116570 was acted upon. > Transaction: Ticket created by Bill.Sirinek@theice.com > Queue: Crypt-OpenSSL-ECDSA > Subject: potentially unnecessary dependency in Crypt-OpenSSL-ECDSA > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: Bill.Sirinek@theice.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=116570 > > > > > It appears that the makefile for Crypt::OpenSSL::ECDSA forces the use of > perl 5.18 or newer. This is not realistic for most enterprise users. Even > the latest RHEL 7 uses perl 5.16. > > I changed the Makefile.PL file to say "use 5.016001" instead of 5.018001 and > the module built, tested and installed with no warnings or errors. I'm not > even sure 5.16 is required, you might consider removing the statement > entirely unless there's some feature only available in a more recent > version of perl that this module uses that I am not aware of. > > I have copied and pasted the output of perl -v as well as the build and test > output. I did not do a make install because I converted this to an rpm file > with the cpan2rpm software and installed the rpm. I installed the > resulting RPM and it seems to work okay. > > Thanks > > Bill > > > bsirinek@myhost $ perl -v > > This is perl 5, version 16, subversion 3 (v5.16.3) built for > x86_64-linux-thread-multi (with 28 registered patches, see perl -V for more > detail) > > Copyright 1987-2012, Larry Wall > > Perl may be copied only under the terms of either the Artistic License or > the GNU General Public License, which may be found in the Perl 5 source > kit. > > Complete documentation for Perl, including FAQ lists, should be found on > this system using "man perl" or "perldoc perl". If you have access to the > Internet, point your browser at http://www.perl.org/, the Perl Home Page. > > > > bsirinek@myhost $ head Makefile.PL > use 5.016001; > use ExtUtils::MakeMaker; > > # You might need to set this if your openssl headers and libs anre in an > unusual place: my $openssl_prefix = $ENV{OPENSSL_PREFIX} || '/usr'; > > # See lib/ExtUtils/MakeMaker.pm for details of how to influence > # the contents of the Makefile that is written. > WriteMakefile( > NAME => 'Crypt::OpenSSL::ECDSA', > > > bsirinek@myhost $ perl Makefile.PL > Checking if your kit is complete... > Looks good > Writing Makefile for Crypt::OpenSSL::ECDSA > Writing MYMETA.yml and MYMETA.json > > > bsirinek@myhost $ make > cp lib/Crypt/OpenSSL/ECDSA.pm blib/lib/Crypt/OpenSSL/ECDSA.pm > AutoSplitting blib/lib/Crypt/OpenSSL/ECDSA.pm > (blib/lib/auto/Crypt/OpenSSL/ECDSA) /usr/bin/perl > /usr/share/perl5/vendor_perl/ExtUtils/xsubpp -typemap > /usr/share/perl5/ExtUtils/typemap -typemap typemap ECDSA.xs > ECDSA.xsc && > mv ECDSA.xsc ECDSA.c gcc -c -I. -I/usr/include -D_REENTRANT -D_GNU_SOURCE > -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include > -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -O2 -g -pipe -Wall > -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong > --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic > -DVERSION=\"0.07\" -DXS_VERSION=\"0.07\" -fPIC "-I/usr/lib64/perl5/CORE" > -DPERL5 -DOPENSSL_NO_KRB5 ECDSA.c Running Mkbootstrap for > Crypt::OpenSSL::ECDSA () > chmod 644 ECDSA.bs > rm -f blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so > gcc -shared -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions > -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches > -m64 -mtune=generic -Wl,-z,relro ECDSA.o -o > blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so \ -L/usr/lib > -lcrypto \ > > chmod 755 blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.so > cp ECDSA.bs blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.bs > chmod 644 blib/arch/auto/Crypt/OpenSSL/ECDSA/ECDSA.bs > Manifying blib/man3/Crypt::OpenSSL::ECDSA.3pm > > > bsirinek@myhost $ make test > PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" > "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/Crypt-OpenSSL-ECDSA.t .. > ok > All tests successful. > Files=1, Tests=21, 0 wallclock secs ( 0.03 usr 0.00 sys + 0.03 cusr 0.00 > csys = 0.06 CPU) Result: PASS > bsirinek@myhost $ > > Bill Sirinek - Senior Systems Engineer > Intercontinental Exchange | ICE > 5660 New Northside Drive NW | 3rd Floor | Atlanta, GA 30328 > Tel: 770.857.2468 | Mobile: 770.851.9902 | Fax: 770.857.0010 > bill.sirinek@theice.com<mailto:bill.sirinek@theice.com> > > > ________________________________ > > This message may contain confidential information and is intended for > specific recipients unless explicitly noted otherwise. If you have reason > to believe you are not an intended recipient of this message, please delete > it and notify the sender. This message may not represent the opinion of > Intercontinental Exchange, Inc. (ICE), its subsidiaries or affiliates, and > does not constitute a contract or guarantee. Unencrypted electronic mail is > not secure and the recipient of this message is expected to provide > safeguards from viruses and pursue alternate means of communication where > privacy or a binding message is desired.
-- Mike McCauley VK4AMM mikem@airspayce.com Airspayce Pty Ltd 9 Bulbul Place Currumbin Waters QLD 4223 Australia http://www.airspayce.com Phone +61 7 5598-7474