Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test script missing from CGI.pm CPAN distribution? [rt.cpan.org #76189] #97

Closed
leejo opened this issue May 22, 2014 · 4 comments
Closed

Test script missing from CGI.pm CPAN distribution? [rt.cpan.org #76189] #97

leejo opened this issue May 22, 2014 · 4 comments

Comments

@leejo
Copy link
Owner

@leejo leejo commented May 22, 2014

https://rt.cpan.org/Ticket/Display.html?id=76189

I have noticed that a change was applied to perl (maint-5.12) in Jan
2011 which modified CGI.pm itself and added two new test scripts:
headers.t and multipart_init.t:

http://perl5.git.perl.org/perl.git/commit/b7fa2aca51

The first two changes do seem to have found their way back to the CGI.pm
CPAN distro, but the latter change -- the addition of multipart_init.t
-- appears to have been missed.

I've checked through every CPAN release from 3.48 through 3.59 and don't
see it anywhere, despite the Changes file for 3.52 claiming to have
fixed a logic error in it!

The current bleadperl version is attached. Could you please consider
including it (or a later version if you have one, perhaps including
whatever fixes 3.52's Changes file was referring to?) in future releases.
@leejo

This comment has been minimized.

Copy link
Owner Author

@leejo leejo commented May 22, 2014

mark@summersault.com - 2012-04-02 14:26:07

I have noticed that a change was applied to perl (maint-5.12) in Jan
2011 which modified CGI.pm itself and added two new test scripts:
headers.t and multipart_init.t:

http://perl5.git.perl.org/perl.git/commit/b7fa2aca51

The first two changes do seem to have found their way back to the CGI.pm
CPAN distro, but the latter change -- the addition of multipart_init.t
-- appears to have been missed.

I've checked through every CPAN release from 3.48 through 3.59 and don't
see it anywhere, despite the Changes file for 3.52 claiming to have
fixed a logic error in it!

The current bleadperl version is attached. Could you please consider
including it (or a later version if you have one, perhaps including
whatever fixes 3.52's Changes file was referring to?) in future releases.

Thanks for the report. I'll see about making this consistent.

Mark

@leejo

This comment has been minimized.

Copy link
Owner Author

@leejo leejo commented May 22, 2014

MARKSTOS - 2012-08-16 01:23:39

This is being released today. Thanks.

@leejo

This comment has been minimized.

Copy link
Owner Author

@leejo leejo commented May 22, 2014

Steve.Hay@verosoftware.com - 2012-08-16 08:27:26

MARKSTOS via RT wrote on 2012-08-16:

<URL: https://rt.cpan.org/Ticket/Display.html?id=76189 >

This is being released today. Thanks.

Thanks, merged into blead in commit a506842e80.

@leejo

This comment has been minimized.

Copy link
Owner Author

@leejo leejo commented Jul 11, 2014

Closing as already resolved as per previous comments.

@leejo leejo closed this Jul 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.