[openssl-dev] [openssl.org #4083] possible fix to make test failure with openssl-1.0.2d on MinGW...

christian fafard via RT rt at openssl.org
Mon Oct 12 11:11:58 UTC 2015


Hi Peter,
 
You are completely right! Windows carriage return is the real problem.
 
I should have done better testing before posting a ticket.
 
I'll use your patch until it get commited to the 1.0.2 branch.
 
Thank you
Christian
 
> Subject: Re: [openssl-dev] [openssl.org #4083] possible fix to make test failure with openssl-1.0.2d on MinGW...
> From: rt at openssl.org
> To: cfaf at hotmail.com
> CC: openssl-dev at openssl.org
> Date: Mon, 12 Oct 2015 06:03:23 +0000
> 
> 
> Hi Christian,
> 
> A similar patch was already applied to the master branch - see
> https://rt.openssl.org/Ticket/Display.html?id=3346 and commit
> 028bac0670c167f154438742eb4d0fbed73df209
> 
> You could cherry-pick the commit and apply it to the 1.0.2 branch.
> 
> 
> Cheers,
> 
> 
> Peter Mosmans
> 
> On 12-10-2015 12:03, christian fafard via RT wrote:
> > I'm sorry for that mess with the previous message.There was no CRLF because it was copy-pasted from emacs.
> > What i tried to say basically is that the "negate regex match" (!/^0$$/) constructused in the line 244 of 'test/Makefile' does not work with some versions of perl.Like for exemple, perl v5.8.8 in the MinGW/msys distibution.
> > That's the reason why 'make test' fail on that platform.
> > My proposal is to invert the if/else actions to get rid of the negation in the expression (/^0$$/).
> > So the line:
> > @<tmp.bntest sh -c "`sh ./bctest ignore`" | $(PERL) -e '$$i=0; while (<STDIN>) {if (/^test (.*)/) {print STDERR "\nverify $$1";} elsif (!/^0$$/) {die "\nFailed! bc: $$_";} else {print STDERR "."; $$i++;}} print STDERR "\n$$i tests passed\n"'
> > would became:
> > @<tmp.bntest sh -c "`sh ./bctest ignore`" | $(PERL) -e '$$i=0; while (<STDIN>) {if (/^test (.*)/) {print STDERR "\nverify $$1";} elsif (/^0$$/) {print STDERR "."; $$i++;} else {die "\nFailed! bc: $$_";}} print STDERR "\n$$i tests passed\n"'
> > I attached a patch.
> > ThanksChristian 		 	   		  
> >
> >
> > _______________________________________________
> > openssl-dev mailing list
> > To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-dev
> 
> 
 		 	   		  


More information about the openssl-dev mailing list