-
Notifications
You must be signed in to change notification settings - Fork 581
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
One t-test fails, Perl 5.38 on Windows 10 #2105
Comments
If it matters, this is the new Strawberry Perl build ("portable" version). I took a quick look in file.t (list/list_tree), and there were many places mentioning "right files", so I'm not sure which was failing. |
It tells you the lines right in the fail messages. 😄 And we test with Strawberry Perl in our GitHub Actions. Everything looks fine there. |
Well, I see that CPAN Testers failed exactly the same way on the same Perl and Windows level, so perhaps it would be a good idea to look at it again. If you don't have a Strawberry Perl to install on (not the GitHub Actions version), I would be happy to try installing an instrumented version of Mojolicious, or force an install and try running file.t and other debug on it, and report back the results to you. Let me know. Update: I have found GitHub Actions to be buggy and unreliable. My advice is not to rely on it as the Gospel Truth of whether your package runs OK on a given platform. If you don't have access to that platform to test on, it's better than nothing, but be sure to periodically check the Testers Matrix on CPAN to see what has happened on a wide variety of platforms. For instance, some of my packages were failing t-tests because some platforms had installed extended precision Perl floating point math packages (I had to round down FP results to single precision in t-tests). Also, I can't test some parts of my packages on GA because they can't or won't install some necessary libraries and packages. |
9.35 fails on the same test. CPAN testers report same problem. Since the Mojolicious developer(s) apparently don't have a Windows/Strawberry Perl 5.38 system to test on, please feel free to contact me if I can be of any help in testing for you. |
Ditto release 9.36. CPAN testers apparently haven't gotten to it yet. Update: fails on CPAN Testers Windows when 5.34 and up. |
Same here. Build fails using strawberry perl. I suspect that the reason for that is that File::Find works with slashes, while canonpath returns backslashes on windows. So when doing the delete of the current directory on the new sub, it fails because that key does not exist. |
That would be quite a "gotcha" (using backslash \ on one hand and slash / on the other). Should be a simple fix, if it is. Note that in Windows, only the command line interpreter requires backslashes, while slashes can normally be used elsewhere. |
Latest release (9.38) same failure. |
This problem still persists. Strawberry Perl 5.40.0, Mojo 9.38. |
Strawberry Perl 5.38 + Mojolicious 9.39 still fails in the same way. I will leave this ticket open, but as it is evident that the owner does not intend to fix the issue, and I have no use for Mojolicious anyway, I will stop attempting to update it on this Perl level. Good bye. |
You're welcome to contribute a fix. |
I don't use it (it was included with Strawberry Perl, and tries to update), and I know nothing about its internals. I would probably be more of a menace than a help in trying to contribute a fix. |
Steps to reproduce the behavior
Installation via 'cpan' to update from CPAN
Expected behavior
Expect clean install
Actual behavior
Failed one t-test: t/mojo/file.t 'right file' (several occurrences listed at 285, 288, 293, 299, 308)
The text was updated successfully, but these errors were encountered: