Bug 192740

Summary: [patch] mail/milter-greylist - fix some Makefile issues
Product: Ports & Packages Reporter: Daniel Austin <freebsd-ports>
Component: Individual Port(s)Assignee: John Marino <marino>
Status: Closed FIXED    
Severity: Affects Some People CC: marino
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
update/simplify Makefile and plist none

Description Daniel Austin 2014-08-17 13:08:12 UTC 
Created attachment 145917 [details]
update/simplify Makefile and plist

- update and simplify Makefile
- ensure manpages are always installed
- remove an absolute assignment from BUILD_DEPENDS into RUN_DEPENDS
- move PORTDOCS and PORTEXAMPLES install into do-install from post-install
- update an include from absolute to relative path

poudriere testport logs at:

http://poudriere.dan.tm/latest-per-pkg/milter-greylist/4.4.3_3/
Comment 1 John Marino freebsd_committer freebsd_triage 2014-08-17 13:15:25 UTC 
I'll take this.
Comment 2 commit-hook freebsd_committer freebsd_triage 2014-08-17 13:37:49 UTC 
A commit references this bug:

Author: marino
Date: Sun Aug 17 13:37:47 UTC 2014
New revision: 365186
URL: http://svnweb.freebsd.org/changeset/ports/365186

Log:
  mail/milter-greylist: Fix and simplify makefile

  This port was conditionally installing manpages per option, which is
  not allowed.  While fixing this, align it with the recently modernized
  milter-greylist-devel port, which includes:

   * Remove assignment of BUILD_DEPENDS to RUN_DEPENDS (illegal).  The
     BUILD_DEPENDS is empty by default
   * Use options to set dependencies and configure arguments
   * Use PORTDOCS and PORTEXAMPLES
   * Wrap to 80 columns
   * Combine post-install target with do-install target
   * include local .mk with relative path instead of hardcoded PORTSDIR

  PR:		192740
  Submitted by:	Daniel Austin (per my request)
  Approved by:	Just Fix It

Changes:
  head/mail/milter-greylist/Makefile
  head/mail/milter-greylist/pkg-plist
Comment 3 John Marino freebsd_committer freebsd_triage 2014-08-17 13:38:46 UTC 
Thanks for doing that, Daniel!