[openstreetmap/openstreetmap-website] update-wiki-pages: process ChangesetKey template (PR #5818)

Tom Hughes notifications at github.com
Tue Mar 18 18:35:35 UTC 2025


@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/2695829326 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250318/49c9fe39/attachment.htm>


More information about the rails-dev mailing list