brummett / Devel-Chitin

Abstract base class for Perl debuggers
1 stars 4 forks source link

Won't install on 5.8.9 #38

Closed brummett closed 6 years ago

brummett commented 6 years ago

Since the inclusion of

[Prereqs / TestRequires ]
experimental = 0

in dist.ini (because of the optree.t test), Devel::Chitin won't install with cpanm, since experimental.pm won't install on 5.8.

Work out some solution. Maybe a patch to experimental to allow it to install, even though it won't do anything. Maybe work out the install process so it won't try installing on 5.8, and ensure it won't be using when testing on 5.8.

brummett commented 6 years ago

This is probably why we had to take it off the list of perls tested by TravisCI

brummett commented 6 years ago

a 'make install' with all the other prereqs works fine

brummett commented 6 years ago

Turns out one of the dzil Git modules requires 5.10, so removing 'experimental' won't completely solve the problem.

Maybe try a different "install:" procedure in .travix.yml. We're using dzil to generate the list of prerequisites and install them. I suspect that not all the dzil authordeps are needed to generate this list. Try only installing what's needed to get 'dzil listdeps' working.

brummett commented 6 years ago

There are multiple issues going on here.

Tried using an older Dist::Zilla (The last of the 5.* series so it would install on 5.8.9), but then I couldn't get TravisCI to test with 5.8.9. Just specifying "5.8" in the test matrix means "5.8.8". But then even that old version of Dist::Zilla wouldn't work on 5.8.8.

Tried using the travis-perl helpers to get 5.8.9 installed. It wasn't available as one of the pre-built images and it didn't build it from source.

This might be doable if there was a separate installation procedure for 5.8.9 that included building perl from source and managing prerequisites by hand. I don't think it's worth it. I'll just keep testing 5.8.9 locally.

brummett commented 6 years ago

Actually got this working via #52