Page MenuHomeFreeBSD

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

Authored by michaelo on Mar 12 2025, 9:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jul 28, 4:12 PM
Unknown Object (File)
Mon, Jul 28, 3:01 PM
Unknown Object (File)
Sun, Jul 27, 10:45 PM
Unknown Object (File)
Fri, Jul 25, 10:53 AM
Unknown Object (File)
Fri, Jul 25, 2:13 AM
Unknown Object (File)
Mon, Jul 21, 9:25 AM
Unknown Object (File)
Mon, Jul 21, 5:48 AM
Unknown Object (File)
Mon, Jul 21, 5:48 AM
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 62883
Build 59767: arc lint + arc unit

Event Timeline

michaelo created this revision.
devel/tevent/Makefile
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
50

Same comment.

devel/tevent/Makefile
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
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