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

Fix Perl 5.8.0 support #130

Merged
merged 3 commits into from Apr 16, 2019
Merged

Fix Perl 5.8.0 support #130

merged 3 commits into from Apr 16, 2019

Conversation

pali
Copy link
Contributor

@pali pali commented Apr 16, 2019

No description provided.

@pali pali force-pushed the perl-5.8.0 branch 2 times, most recently from 4ba46a9 to 8e09d0c Compare April 16, 2019 09:00
@rurban
Copy link
Owner

rurban commented Apr 16, 2019

yeah, 5.8.0 is basically just a broken 5.6.3

@pali
Copy link
Contributor Author

pali commented Apr 16, 2019

Seems that Travis has timeout problems with it... I will remove it from Travis configuration. But locally it is working.

@pali
Copy link
Contributor Author

pali commented Apr 16, 2019

Now Travis tests passed. But failing AppVeyor because it cannot download cperl.

@rurban rurban merged commit 809f17e into rurban:master Apr 16, 2019
@rurban
Copy link
Owner

rurban commented Apr 16, 2019

Yeah, I have to fix the YAML.

@pali pali deleted the perl-5.8.0 branch April 17, 2019 07:15
pali referenced this pull request Apr 18, 2019
try 5.8.5. also improve the -n $cperl test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants