-
Notifications
You must be signed in to change notification settings - Fork 28
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 Perl version 5.003 - 5.005 on Travis #98
Comments
suggestion, if we can confirm that it s not passing with versions below 5.6.0 maybe we should change the wording there... as I think during our last discussion we were targeting 5.6.0 |
5.5.3 is passing |
Anyway, I did some modification to perlbrew and travis-perl-helpers and seems that it allows me to add also older perl versions to Travis... I will try to create a pull request in next days. |
Support for 5.005_01 - 5.005_05 into travis-perl-helpers is in this pull request: 5.005 cannot be compiled with current patchperl, compilation fails on error:
And older versions (5.004) stuck at configure phase in:
(without no more output) |
How many distributions are on cpan that use ppport.h that don't have a minimum of 5.6? I'm guessing very few. |
On 6/14/19 8:47 PM, Karen Etheridge wrote:
do you we should provide support below 5.6.0?
How many distributions are on cpan that use ppport.h that don't have a
minimum of 5.6? I'm guessing very few.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#98?email_source=notifications&email_token=AAA2DH2GK3Z5QZTO56BKCCLP2RJ5HA5CNFSM4HYFSXH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXYOY5Y#issuecomment-502328439>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAA2DH4WW52TVDLHDWTA5ZLP2RJ5HANCNFSM4HYFSXHQ>.
If pali can provide patches before 5.6, I think it's fine to push them.
But I don't think we need to "support" earlier than really 5.8.
|
PR for Perl 5.5 testing: #178 |
Documentation says
ppport.h supports Perl versions from 5.003
, but on Travis is configured minimal version 5.6.0.The text was updated successfully, but these errors were encountered: