Bug 217592

Summary: databases/influxdb: add MANPAGES option
Product: Ports & Packages Reporter: Johannes Jost Meixner <xmj>
Component: Individual Port(s)Assignee: Jochen Neumeister <joneum>
Status: Closed FIXED    
Severity: Affects Some People CC: cheffo, girgen, joneum
Priority: --- Keywords: patch-ready
Version: LatestFlags: cheffo: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
add manpages knob
none
add manpages knob, v2 none

Description Johannes Jost Meixner freebsd_committer freebsd_triage 2017-03-06 20:28:46 UTC
Created attachment 180573 [details]
add manpages knob

Influxdb has beautiful manpages that are currently not shipped.

Add a knob and glue to do that.
Comment 1 Johannes Jost Meixner freebsd_committer freebsd_triage 2017-03-07 11:15:29 UTC
Created attachment 180596 [details]
add manpages knob, v2

Make patch apply to influx 1.2.
Comment 2 Palle Girgensohn freebsd_committer freebsd_triage 2017-03-08 23:04:58 UTC
Great!

There is a standard knob for DOCS, can we use that instead of a custom one?

Palle
Comment 3 Palle Girgensohn freebsd_committer freebsd_triage 2017-03-08 23:11:33 UTC
See https://www.freebsd.org/doc/en/books/porters-handbook/makefile-options.html, Section 5.12.1.3
Comment 4 Johannes Jost Meixner freebsd_committer freebsd_triage 2017-03-22 16:59:03 UTC
I'd like to stick with MANPAGES: Mk/bsd.options.desc.mk provides a description for it, and there are 63 other ports that define this option.
Comment 5 Johannes Jost Meixner freebsd_committer freebsd_triage 2017-06-03 11:48:09 UTC
Can we resolve this please? :-)
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-06-20 20:22:55 UTC
A commit references this bug:

Author: joneum
Date: Tue Jun 20 20:22:17 UTC 2017
New revision: 443999
URL: https://svnweb.freebsd.org/changeset/ports/443999

Log:
  - databases/influxdb: add MANPAGES option

  PR:		217592
  Reported by:	xmj
  Approved by:	cheffo at freebsd-bg.org (maintainer) miwi (mentor)
  Differential Revision:	https://reviews.freebsd.org/D11219

Changes:
  head/databases/influxdb/Makefile
  head/databases/influxdb/pkg-plist
Comment 7 Jochen Neumeister freebsd_committer freebsd_triage 2017-06-20 20:23:14 UTC
Committed, thanks :-)