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 tests for '.' in @INC #62

Closed
wants to merge 1 commit into from

Conversation

kentfredric
Copy link
Contributor

This is the smallest change that can be made that preserves original
semantics

Bug: https://bugs.gentoo.org/614342
Closes: #58

This is the smallest change that can be made that preserves original
semantics

Bug: https://bugs.gentoo.org/614342
Closes: tokuhirom#58
kentfredric added a commit to kentnl-gentoo/Test-TCP that referenced this pull request Jan 10, 2018
This is the smallest change that can be made that preserves original
semantics

Bug: https://bugs.gentoo.org/614342
Bug: tokuhirom#58
Bug: tokuhirom#62
@karenetheridge
Copy link

The perl 5.28.0 release is coming up fast; this issue is blocking testing of Plack, Catalyst, and everything else downstream. Please could it be merged soon? thank you!

@tokuhirom
Copy link
Owner

hm the test case is failing

@kentfredric
Copy link
Contributor Author

Clearly the tests are broken in the most serious way:

cpanm --version
Can't locate strict.pm in @INC (@INC contains: /home/travis/perl5/perlbrew/perls/5.14/lib/site_perl/5.14.4/x86_64-linux /home/travis/perl5/perlbrew/perls/5.14/lib/site_perl/5.14.4 /home/travis/perl5/perlbrew/perls/5.14/lib/5.14.4/x86_64-linux /home/travis/perl5/perlbrew/perls/5.14/lib/5.14.4 .) at /home/travis/perl5/perlbrew/bin/cpanm line 463.
BEGIN failed--compilation aborted at /home/travis/perl5/perlbrew/bin/cpanm line 463.

Yeah .... that's uh... not gonna work.

@nponeccop
Copy link
Collaborator

nponeccop commented Jul 19, 2018

Fixed by #69

@kentfredric don't forget to delete the branch - there are too many forks :)

@nponeccop nponeccop closed this Jul 19, 2018
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

4 participants