-
Notifications
You must be signed in to change notification settings - Fork 23
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
testsuite fails on alpine linux #103
Comments
Thanks, @ncopa! I'm not all too familiar with Alpine. If I wanted to add a GitHub workflow that tested this on Alpine using Docker images, what would be the easiest way for me to replicate the above? |
Am adding workflows in #104. That may help find the issue. |
Looks like this is not specific to Alpine at all. |
Seems to be here: https://metacpan.org/source/OALDERS/WWW-Mechanize-Cached-1.53/lib/WWW/Mechanize/Cached.pm#L193 It's expecting the header to be an |
So the problem is in WWW-Mechanize-Cached and should be fixed there? |
Yes. I may get a fix in there today. |
@ncopa Tested successfully against MetaCPAN-Client-2.026000 on Alpine linux Still has an issue with HTTP::Tiny: but the test succeeds so it appears to be expected t/ua_trap.t ................... Use of uninitialized value in concatenation (.) or string at /usr/share/perl5/core_perl/HTTP/Tiny.pm line 602. |
Thanks @timlegge. I have opened this issue with HTTP::Tiny: chansen/p5-http-tiny#137. I can fix it on this end as well. So, I need to release a new WWW::Mechanize::Cached and then fix the undef warning here, bump the version of WWW::Mechanize::Cached and then release a new MetaCPAN::Client. :) |
I suspect some of the dependencies was updated and introduces a regression, but I don't know which or how to fix this.
Recent changes of dependencies in Alpine that may have introduced this:
The text was updated successfully, but these errors were encountered: