Bug 194076 - [maintainer update] for graphics/openimageio
Summary: [maintainer update] for graphics/openimageio
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Marcus von Appen
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2014-10-01 17:31 UTC by Shane
Modified: 2014-11-23 14:44 UTC (History)
0 users

See Also:

Attachments
openimageio update to 1.4.13 (2.65 KB, text/plain)
2014-10-01 17:31 UTC, Shane 		no flags Details
openimageio update to 1.4.14 (2.55 KB, text/plain)
2014-10-22 04:30 UTC, Shane 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shane 2014-10-01 17:31:16 UTC 
Created attachment 147890 [details]
openimageio update to 1.4.13

update graphics/openimageio to v1.4.13
Includes changes for USE_PYTHON and @dirrm
Comment 1 Marcus von Appen freebsd_committer freebsd_triage 2014-10-02 08:06:53 UTC 
The PORTREVISION in py-openimageio should be reset, not?

Also, @dir is only necessary for empty directories (to be created) or, if the directories have different permissions/ownerships (so that @chown/@perm entries are necessary).
Comment 2 Shane 2014-10-02 18:47:57 UTC 
py-openimageio PORTREVISION - yes, it should use the master PORTREVISION but keeps getting automated entries on dependency changes.

As for @dir, make check-plist says it's ok, but during install I get

pkg-static: Warning: @dirrm is deprecated please use @dir

Seems there is no error when removing @dirrm entries, so we don't swap @dirrm and @dirrmtry entries for @dir we just remove them completely and use @dir just for special cases?
Comment 3 Marcus von Appen freebsd_committer freebsd_triage 2014-10-02 18:57:51 UTC 
(In reply to FreeBSD from comment #2)
> py-openimageio PORTREVISION - yes, it should use the master PORTREVISION but
> keeps getting automated entries on dependency changes.

Which is fine and correct. It should be reset on updates of the master port, though.
 
> As for @dir, make check-plist says it's ok, but during install I get
> 
> pkg-static: Warning: @dirrm is deprecated please use @dir
> 
> Seems there is no error when removing @dirrm entries, so we don't swap
> @dirrm and @dirrmtry entries for @dir we just remove them completely and use
> @dir just for special cases?

Exactly. It's only needed, if a port needs empty directories (e.g. under /var), or directories need to have speciall permissions or changes in the owner/group.
Comment 4 Shane 2014-10-22 04:30:12 UTC 
Created attachment 148559 [details]
openimageio update to 1.4.14
Comment 5 Shane 2014-10-22 04:33:35 UTC 
There was a minor update yesterday, only version and distinfo changes since previous patch.
Comment 6 Marcus von Appen freebsd_committer freebsd_triage 2014-10-22 05:04:01 UTC 
Looks good, I'll get it committed soon. Keep in mind that py-openimageio also needs to be reset (PORTREVISION) on updates.
Comment 7 commit-hook freebsd_committer freebsd_triage 2014-11-23 14:34:23 UTC 
A commit references this bug:

Author: mva
Date: Sun Nov 23 14:33:39 UTC 2014
New revision: 373153
URL: https://svnweb.freebsd.org/changeset/ports/373153

Log:
  - Update to version 1.4.14

  PR:		194076
  Submitted by:	FreeBSD@ShaneWare.Biz (maintainer)

Changes:
  head/graphics/openimageio/Makefile
  head/graphics/openimageio/distinfo
  head/graphics/openimageio/pkg-plist
  head/graphics/openimageio/pkg-plist-pybind
  head/graphics/py-openimageio/Makefile
Comment 8 Marcus von Appen freebsd_committer freebsd_triage 2014-11-23 14:44:16 UTC 
Committed, thanks.