Page MenuHomeFreeBSD

*/*: Rework manpages handling in Samba-related ports
ClosedPublic

Authored by michaelo on Mar 12 2025, 9:16 AM.
Tags
None
Referenced Files
F134244904: D49327.id152169.diff
Fri, Oct 31, 9:34 PM
Unknown Object (File)
Wed, Oct 29, 1:37 AM
Unknown Object (File)
Tue, Oct 28, 12:51 PM
Unknown Object (File)
Tue, Oct 28, 10:10 AM
Unknown Object (File)
Tue, Oct 28, 7:37 AM
Unknown Object (File)
Tue, Oct 28, 7:22 AM
Unknown Object (File)
Tue, Oct 28, 2:50 AM
Unknown Object (File)
Mon, Oct 27, 2:44 PM
Subscribers
None

Details

Summary
  • Remove outdated, incorrect, precompiled manpages from ${FILESDIR}/files/man.
  • Properly disable manpages generation since XSLTPROC was set to true(1) WAF always assumed xsltproc(1) to work and if any automatic dependency pulled in textproc/docbook-xsl, manpages were generated even if explicitly disabled.

Approved by: jrm (mentor), otis (mentor), ...

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 62896
Build 59780: arc lint + arc unit

Event Timeline

michaelo created this revision.
devel/tevent/Makefile
48–49

I'm confused by this comment. If there are no man pages for this port, why is there an OPTION and the associated logic?

devel/tevent016/Makefile
49–50

Same comment.

devel/tevent/Makefile
48–49

The actual block has been there, I added the comment on purpose for the following reason:

  • consistency with other Samba-related ports
  • if a manpage is added only pkg-plist needs to be modified.
devel/tevent/Makefile
48–49

Let's remove it. If necessary, it's not that hard to copy a couple of lines from the other ports in the future. A MANPAGES option and the associated logic for ports that don't install any manpages is kinda weird.

This revision is now accepted and ready to land.Mar 12 2025, 9:55 PM