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

Add AppVeyor config #13

Closed
wants to merge 29 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@zakame
Contributor

zakame commented Feb 5, 2018

This PR is a submission for my CPAN Pull Request Challenge this month.

This PR adds an AppVeyor config to try diagnosing issues on CPAN Testers
related to FAILing cygwin/Windows builds, such as these:

Add AppVeyor config
This adds an AppVeyor config to try diagnosing issues on CPAN Testers
related to FAILing cygwin/Windows builds.
@zakame

This comment has been minimized.

Contributor

zakame commented Feb 5, 2018

For builds please see for example https://ci.appveyor.com/project/zakame/perl-log-dispatch-filerotate/build/1.0.12

Note that this particular build seems to indicate a file locking race condition that wasn't detected in the CPAN Testers logs; this may be another issue entirely (as the CPAN Testers issues seem to stem from not finding Date::Manip correctly.)

zakame added some commits Feb 5, 2018

Add .appveyor.cmd
Prepare to set up builds in multiple environments (system, cygwin, and
strawberry.)
Remove `perl_type: system` on AppVeyor for now
Dist::Zilla dependencies don't build at all.
Try setting custom PERL_MM_OPT for cygwin
Module install failures seem to stem from stack smashing protection
being enabled, so try to disable that under Cygwin.
Set TZ variable in AppVeyor
dzil somehow needs the time.
Last tweaks, hopefully
Last build indicated missing Test::Pod, so install that.  Also reduce
cpanm output for dzil authordeps.
@mschout

This comment has been minimized.

Owner

mschout commented Feb 6, 2018

Let me know via a comment when you have completed your changes and I'll squash them onto my own repo

@zakame

This comment has been minimized.

Contributor

zakame commented Feb 7, 2018

Hi @mschout, thanks for considering this PR! 👍

I think this one is largely done, I'm already making use of this to find test failures on cygwin/strawberry (e.g. #14 and #15.)

Revert travis changes
This should not be removed...
@mschout

This comment has been minimized.

Owner

mschout commented Feb 7, 2018

I've squash/merged the changes from this branch into master. Thanks!

@mschout mschout closed this Feb 7, 2018

@zakame zakame deleted the zakame:appveyor-ci branch Feb 7, 2018

@zakame

This comment has been minimized.

Contributor

zakame commented Feb 7, 2018

Thanks too! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment