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)
Mon, Aug 24, 12:10 PM
Unknown Object (File)
Fri, Aug 21, 12:13 PM
Unknown Object (File)
Fri, Aug 21, 12:13 PM
Unknown Object (File)
Fri, Aug 21, 12:11 PM
Unknown Object (File)
Tue, Aug 18, 2:36 AM
Unknown Object (File)
Sun, Aug 16, 2:14 PM
Unknown Object (File)
Sun, Aug 16, 12:10 PM
Unknown Object (File)
Thu, Aug 13, 11:17 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.