Bug 203579

Summary: multimedia/ffmpeg: simplify manpages and html docs installation
Product: Ports & Packages Reporter: Jan Beich <jbeich>
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Only Me CC: riggs
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (multimedia)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
v0
none
QA with bug 203477: poudriere bulk -t -j 93i386
none
v0 rebased after bug 203477 none

Description Jan Beich freebsd_committer 2015-10-06 09:59:37 UTC
Created attachment 161754 [details]
v0

Let's take advantage of the following code in doc/Makefile:

  install-doc: install-html install-man

  install-html:

  install-man:

  ifdef CONFIG_HTMLPAGES
  install-progs-$(CONFIG_DOC): install-html

  install-html: $(HTMLPAGES)
          $(Q)mkdir -p "$(DOCDIR)"
          $(INSTALL) -m 644 $(HTMLPAGES) "$(DOCDIR)"
  endif

  ifdef CONFIG_MANPAGES
  #install-progs-$(CONFIG_DOC): install-man

  install-man: $(MANPAGES)
          $(Q)mkdir -p "$(MANDIR)/man1"
          $(INSTALL) -m 644 $(MANPAGES1) "$(MANDIR)/man1"
          $(Q)mkdir -p "$(MANDIR)/man3"
          $(INSTALL) -m 644 $(MANPAGES3) "$(MANDIR)/man3"
  endif
Comment 1 Jan Beich freebsd_committer 2015-10-06 10:12:44 UTC
Created attachment 161755 [details]
QA with bug 203477: poudriere bulk -t -j 93i386
Comment 2 Jan Beich freebsd_committer 2015-10-06 10:14:52 UTC
Created attachment 161756 [details]
v0 rebased after bug 203477
Comment 3 commit-hook freebsd_committer 2015-10-12 19:24:35 UTC
A commit references this bug:

Author: riggs
Date: Mon Oct 12 19:24:22 UTC 2015
New revision: 399150
URL: https://svnweb.freebsd.org/changeset/ports/399150

Log:
  Simplify DOCS handling, install manpages section 3 for ffmpeg libraries

  PR:		203579
  Submitted by:	jbeich@FreeBSD.org

Changes:
  head/multimedia/ffmpeg/Makefile
  head/multimedia/ffmpeg/pkg-plist
Comment 4 Thomas Zander freebsd_committer 2015-10-12 19:27:17 UTC
Very good catch! Thanks for this one, Jan!