github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
github ([personal profile] github) wrote in [site community profile] changelog2026-02-08 11:13 pm

[dreamwidth/dreamwidth] ace801: Fix Apache2::Const load-order issue in compile tests

Branch: refs/heads/main Home: https://github.com/dreamwidth/dreamwidth Commit: ace801e57d9f8b5cb4fe833aacca10c6eae6a2ab https://github.com/dreamwidth/dreamwidth/commit/ace801e57d9f8b5cb4fe833aacca10c6eae6a2ab Author: Mark Smith mark@dreamwidth.org Date: 2026-02-08 (Sun, 08 Feb 2026)

Changed paths: M .github/workflows/ci.yml M cgi-bin/DW/Request/Apache2.pm

Log Message:


Fix Apache2::Const load-order issue in compile tests

When modules like LJ::URI load Apache2::Const with :common first, a subsequent use Apache2::Const -compile => qw/ :common :http / is a no-op because Perl skips the import for already-loaded modules. This leaves the :http constants uncompiled, failing as barewords under strict subs.

Use an explicit BEGIN { Apache2::Const->compile(...) } which works regardless of prior load state. Also revert the unnecessary PERL5LIB workaround from the CI workflow.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

To unsubscribe from these emails, change your notification settings at https://github.com/dreamwidth/dreamwidth/settings/notifications