Bug 226464

Summary: graphics/p5-Image-ExifTool: Update to 10.80
Product: Ports & Packages Reporter: Tod McQuillin <devin>
Component: Individual Port(s)Assignee: Sunpoet Po-Chuan Hsieh <sunpoet>
Status: Closed FIXED    
Severity: Affects Some People CC: devin
Priority: --- Keywords: needs-qa
Version: LatestFlags: koobs: maintainer-feedback? (devin)
Hardware: Any   
OS: Any   
Description Flags
Update to 10.80 devin: maintainer-approval+

Description Tod McQuillin 2018-03-09 01:16:03 UTC
Created attachment 191322 [details]
Update to 10.80

Update to latest production release 10.80

Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2018-03-09 04:13:45 UTC
Thanks David

Please set the maintainer-approval attachment flag to + to signify maintainer approval on patches for ports you maintainer. Attachment -> Details -> maintainer-approval [+]

Also, can you please confirm this change passes QA (portlint, poudriere) or not?
Comment 2 Tod McQuillin 2018-03-09 04:22:19 UTC
This patch has passed portlint and 'port test' from ports-mgmt/porttools
Comment 3 Sunpoet Po-Chuan Hsieh freebsd_committer 2018-03-09 20:56:51 UTC

Regarding the DOCS installation, I would like to simplify Makefile as follows:
@@ -23,23 +23,11 @@

 PORTDOCS=      *

-DOC_FILES1=    Changes README
-DOCSDIR2=      ${DOCSDIR}/html
-DOC_FILES2=    *.css *.html *.pdf
-DOCSDIR3=      ${DOCSDIR2}/TagNames
-DOC_FILES3=    *.css *.html

-       @${MKDIR} ${STAGEDIR}${DOCSDIR3}
+       cd ${WRKSRC}/ && ${COPYTREE_SHARE} html ${STAGEDIR}${DOCSDIR}

 .include <bsd.port.mk>

It also fix a problem that ${WRKSRC}/html/overview.png is not installed.
% cd `make -V WRKSRC`
% ls html/overview.png
% grep overview.png html/*
html/under.html:<center><img src="overview.png" alt="ExifTool Overview" width="680" height="460"></center>
Comment 4 commit-hook freebsd_committer 2018-03-10 23:02:18 UTC
A commit references this bug:

Author: sunpoet
Date: Sat Mar 10 23:01:38 UTC 2018
New revision: 464115
URL: https://svnweb.freebsd.org/changeset/ports/464115

  Update to 10.80

  - While I'm here:
    - Simplify DOCS installation
    - Convert to options target helper

  Changes:	http://search.cpan.org/dist/Image-ExifTool/Changes
  PR:		226464
  Submitted by:	Tod McQuillin <devin@spamcop.net> (maintainer)

Comment 5 Sunpoet Po-Chuan Hsieh freebsd_committer 2018-03-10 23:05:38 UTC
Committed. Thanks!