Page MenuHomeFreeBSD

archivers/upx: Fix build due to missing perl (pod2*) tools
ClosedPublic

Authored by koobs on Dec 14 2018, 8:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 26, 9:14 AM
Unknown Object (File)
Tue, May 26, 9:08 AM
Unknown Object (File)
Tue, May 19, 4:11 AM
Unknown Object (File)
Tue, May 19, 4:07 AM
Unknown Object (File)
Sun, May 17, 1:23 PM
Unknown Object (File)
Sun, May 17, 1:15 PM
Unknown Object (File)
Apr 28 2026, 9:34 AM
Unknown Object (File)
Apr 28 2026, 5:24 AM
Subscribers

Details

Summary
archivers/upx: Fix build due to missing perl (pod2*) tools

upx unconditionally runs its docs target, which relies on multiple [1] pod2*
tools provided by perl.

USES=perl and USE_PERL=build were removed in ports r478489, resulting in those
tools not being available at build time, and the following errors:

  /bin/sh: pod2man: not found
  /bin/sh: pod2html: not found
  /bin/sh: pod2text: not found

This change, accordingly, re-adds perl as an unconditional build dependency,
including a comment about what its for.

While I'm here:

  * Make COMMENT portlint compliant [2]
  
[1] pod2man, pod2html, pod2text
[2] Wikipedia, MajorGeeks and others aslso describe UPX without a "The" prefix

PR: 234002
Reported by: <mail ozzmosis com>
Approved by: portmgr (blanket: build fix, JFI)
Differential_Revision: D18549
MFH: 2018Q4
Test Plan
  • portlint: OK (looks fine.)
  • testport: OK (poudriere: with/without DOCS)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 21559
Build 20863: arc lint + arc unit

Event Timeline

koobs edited the summary of this revision. (Show Details)
This revision was not accepted when it landed; it landed in state Needs Review.Dec 14 2018, 8:44 AM
This revision was automatically updated to reflect the committed changes.