Page MenuHomeFreeBSD

devel/p5-Pod-Coverage: Add missing dependency textproc/p5-Pod-Parser
ClosedPublic

Authored by loader on May 15 2021, 10:49 AM.

Details

Summary
# make -C /usr/ports/devel/p5-Pod-Coverage install
# perl -MPod::Coverage
Can't locate Pod/Find.pm in @INC (you may need to install the Pod::Find module) (@INC contains: /usr/local/lib/perl5/site_perl/mach/5.32 /usr/local/lib/perl5/site_perl /usr/local/lib/perl5/5.32/mach /usr/local/lib/perl5/5.32) at /usr/local/lib/perl5/site_perl/Pod/Coverage.pm line 6.
BEGIN failed--compilation aborted at /usr/local/lib/perl5/site_perl/Pod/Coverage.pm line 6.
Compilation failed in require.
BEGIN failed--compilation aborted.

Proposed commit log message:

devel/p5-Pod-Coverage: Add missing dependency

Add missing dependency on textproc/p5-Pod-Parser. Without it, the
following error is observed at runtime:

 Can't locate Pod/Find.pm in @INC (you may need to install the Pod::Find module)

Reviewed_by:    koobs
Approved by:    koobs (ports, mentor), portmgr (blanket: runtime fix)
Differential_Revision: D30274
MFH:            2021Q2 (missing dependency, runtime fix)
Test Plan
  • portlint: OK (looks fine.)
  • testport: OK (poudriere: 1400006, [amd64], tested)

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

loader created this revision.
loader edited the summary of this revision. (Show Details)
  • Include all context/why in the commit log message. The entire differential summary should only need to ever be what the full commit log message will be, which includes all necessary detail for someone with zero knowledge to grok the full context of the committed change. I've updated the commit log message as an example of what I mean.
  • fetch, build, run, package failures are always Approved by: portmgr (blanket: <type> fix). The same reason can be used for the MFH <reason>. Update commit log message accordingly
This revision is now accepted and ready to land.May 16 2021, 2:14 AM