@tomhughes commented on this pull request.
> @@ -8,6 +8,7 @@ use Pod::Usage;
use MediaWiki::API;
use Test::More qw(no_plan);
use YAML::XS qw(Dump);
+# NOTE: on macos, LWP::Protocol::https also needs to be installed
There's nothing macos specific about that - the https protocol handler is in a
separate CPAN module and is in a separate package for most if not all linux
distributions as well.
But because the protocol handlers are loaded dynamically they don't need to be
explicitly required here and won't be a dependency of LWP and hence of anything
that requires LWP.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5818#pullrequestreview-2695829326
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5818/review/2695829...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
