Skip to content

fix: remove hard dependency on deprecated Test2::Harness::Util::IPC#399

Draft
Koan-Bot wants to merge 1 commit into
cpan-authors:mainfrom
atoomic:koan.atoomic/fix-build-requires-ipc
Draft

fix: remove hard dependency on deprecated Test2::Harness::Util::IPC#399
Koan-Bot wants to merge 1 commit into
cpan-authors:mainfrom
atoomic:koan.atoomic/fix-build-requires-ipc

Conversation

@Koan-Bot
Copy link
Copy Markdown
Contributor

What

Remove the hard BUILD_REQUIRES/cpanfile dependency on Test2::Harness::Util::IPC (deprecated, now a broken shim) and make t/lib/Test/TMF.pm handle both old and new module names gracefully.

Why

Test2::Harness::Util::IPC was renamed to Test2::Harness::IPC::Util. The old module is now a deprecation shim that dies on load, causing t/import.t, t/plugin.t, and t/trace.t to fail with exit 255. This is the root cause of #325 (cpanel-perl build failure) and affects any installation where only the new module is present.

How

  • Makefile.PL: removed the Test2::Harness::Util::IPC entry from BUILD_REQUIRES
  • cpanfile: removed the matching requires line
  • t/lib/Test/TMF.pm: try new module first (Test2::Harness::IPC::Util), fall back to old, skip tests gracefully if neither is available

Testing

The 3 affected tests (import.t, plugin.t, trace.t) will now skip cleanly when neither IPC module is available, instead of dying. All other tests are unaffected.

Closes #325. Supersedes #338 (includes the same TMF.pm fix plus the missing Makefile.PL/cpanfile changes).


🤖 Generated with Claude Code

Test2::Harness::Util::IPC was renamed to Test2::Harness::IPC::Util.
The old module is now a broken deprecation shim that dies on load,
causing t/import.t, t/plugin.t, and t/trace.t to fail (GH cpan-authors#325).

Fix all three dependency declaration points:
- Makefile.PL: remove BUILD_REQUIRES entry
- cpanfile: remove test requires entry
- t/lib/Test/TMF.pm: try new module first, fall back to old,
  skip gracefully if neither is available

Closes cpan-authors#325. Supersedes cpan-authors#338.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build fail on cpanel-perl

1 participant