Skip to content

Remove workaround for distros needing dot in @INC #18394

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

Merged
merged 1 commit into from
Dec 23, 2020

Conversation

jkeenan
Copy link
Contributor

@jkeenan jkeenan commented Dec 10, 2020

In commit 19641fd (Apr 07 2017), as part of ceasing to pass '.' to
@inc in tests, we added a workaround to t/TEST for CPAN distributions
which ship with core which were not yet fully adapted to the new
regulation on @inc.

All such CPAN distributions have now been adapted. Hence, we can remove
the workaround.

In commit 19641fd (Apr 07 2017), as part of ceasing to pass '.' to
@inc in tests, we added a workaround to t/TEST for CPAN distributions
which ship with core which were not yet fully adapted to the new
regulation on @inc.

All such CPAN distributions have now been adapted.  Hence, we can remove
the workaround.
@jkeenan jkeenan requested a review from steve-m-hay December 23, 2020 00:04
@jkeenan jkeenan self-assigned this Dec 23, 2020
@jkeenan
Copy link
Contributor Author

jkeenan commented Dec 23, 2020

In commit 19641fd (Apr 07 2017), as part of ceasing to pass '.' to
@inc in tests, we added a workaround to t/TEST for CPAN distributions
which ship with core which were not yet fully adapted to the new
regulation on @inc.

All such CPAN distributions have now been adapted. Hence, we can remove
the workaround.

Are there any objections to this change to t/test.pl?

Thank you very much.
Jim Keenan

@Leont
Copy link
Contributor

Leont commented Dec 23, 2020

LGTM

@jkeenan jkeenan removed the request for review from steve-m-hay December 23, 2020 01:30
@jkeenan jkeenan merged commit 150d2b0 into blead Dec 23, 2020
@jkeenan jkeenan deleted the smoke-me/jkeenan/t-TEST-workarounds-20201210 branch December 24, 2020 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
close