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

New test file name is too long to ship with perl core #65

Closed
khwilliamson opened this issue Apr 5, 2022 · 1 comment
Closed

New test file name is too long to ship with perl core #65

khwilliamson opened this issue Apr 5, 2022 · 1 comment

Comments

@khwilliamson
Copy link
Contributor

I took the newly merged pul requests to try out on an EBCDIC box to make sure they worked in that exotic situation. They did! However, the file rt_122270_is_bool_for_obsolete_xs_boolean.t needs to have its name shortened. There is a test in the perl core it is failing, which restricts file name lengths to 39 characters. Someone wrote this because some system had that restriction. Perhaps those systems are gone, so the test is obsolete; I don't know, but it seems like the path of least effort is to just shorten the file name; otherwise there will be complications when the next release of this module is attempted to be imported and shipped with Perl core.

charsbar added a commit that referenced this issue Apr 5, 2022
@charsbar
Copy link
Collaborator

charsbar commented Apr 5, 2022

I made the filename a little shorter. Thanks.

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

No branches or pull requests

2 participants