Bug 166858 - New port graphics/openimageio
Summary: New port graphics/openimageio
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Pawel Pekala
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-11 21:00 UTC by Shane
Modified: 2012-04-17 17:10 UTC (History)
0 users

See Also:


Attachments
openimageio.shar (6.47 KB, text/plain)
2012-04-12 03:58 UTC, Shane
no flags Details
openimageio-v2.shar (6.96 KB, application/x-shar)
2012-04-16 17:52 UTC, Pawel Pekala
no flags Details
iv-patch.shar (812 bytes, text/plain)
2012-04-17 11:55 UTC, Shane
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Shane 2012-04-11 21:00:40 UTC
This new port adds the openimageio library - the main influence is to be used by graphics/blender to enable the new cycles ray tracing render engine. I don't know what other programs can also make use of this yet.

I have used tinderbox locally to test this and also built on redports tinderbox - http://redports.org/buildarchive/20120410074515-62402/

It uses at least one posix function that isn't available in 7.x so I have marked it as ignore for OSVERSION < 8.0. My opinion is that users of this (or more specifically blender's cycles render engine) would be more likely to have newer machines with recent system versions so haven't spent time in looking at patching it for 7.x

The only other issue that I used USE_GCC=4.6+ to get around is an asm instruction for the i386 build that breaks with both base gcc and clang builds.
Comment 1 Shane 2012-04-12 03:58:29 UTC
it may have been helpful if I remembered to add the shar file.
Comment 2 Pawel Pekala freebsd_committer freebsd_triage 2012-04-15 00:16:22 UTC
Responsible Changed
From-To: freebsd-ports-bugs->pawel

I'll take it.
Comment 3 Pawel Pekala freebsd_committer freebsd_triage 2012-04-15 13:49:02 UTC
Hello,

Thanks for your submission to make FreeBSD better.

Everything looks almost ready to commit, I only fixed some missing deps
and small errors. I`m not sure about all options, during reviewing I
got impression that OpenGL and Qt libs are required only for building
image viewer part and are not used by library. They are separate options
which don't make sense to me, so my proposition is to merge them into
one IMAGE_VIEWER. Or I`m missing something here ?

--=20
pozdrawiam / with regards
Pawe=B3 P=EAkala
Comment 4 Shane 2012-04-16 04:33:54 UTC
On 15/04/2012 22:19, Pawel Pekala wrote:
> Hello,
>
> Thanks for your submission to make FreeBSD better.
>
> Everything looks almost ready to commit, I only fixed some missing deps
> and small errors. I`m not sure about all options, during reviewing I
> got impression that OpenGL and Qt libs are required only for building
> image viewer part and are not used by library. They are separate options
> which don't make sense to me, so my proposition is to merge them into
> one IMAGE_VIEWER. Or I`m missing something here ?
>

I'm sure your right about qt/gl only used in viewer - sounds like a good 
idea to combine into one option. The docs only mention that turning off 
the opengl and qt options "Skip anything that needs xxx"
- I can only see the two options used within the image viewer code and 
doubt that would change.

Now that I look over it again I see image viewer isn't built within 
tinderbox so was also missed in the pkg-plist - that would be me missing 
the QT_COMPONENTS ? Is that the deps I missed?

Was there anything else I got wrong?
Comment 5 Pawel Pekala freebsd_committer freebsd_triage 2012-04-16 17:52:42 UTC
HDF5, webp, QT_COMPONENTS as you already know and boost-python-libs for
python bindings. That's what I have found from reading build logs.

I'm attaching what I have so far for you to see. In this form I think
it looks good, I just need to test couple more things and should commit
tomorrow. If you have any questions/improvements let me know.

-- 
pozdrawiam / with regards
Pawe³ Pêkala
Comment 6 Shane 2012-04-17 11:55:16 UTC
For 8.0 - 8.2 another patch is needed for the image viewer (for log2f).
Otherwise I think it's good to go.
Comment 7 dfilter service freebsd_committer freebsd_triage 2012-04-17 17:04:38 UTC
pawel       2012-04-17 16:04:20 UTC

  FreeBSD ports repository

  Modified files:
    graphics             Makefile 
  Added files:
    graphics/openimageio Makefile distinfo pkg-descr pkg-plist 
    graphics/openimageio/files patch-src_iv_imageviewer.cpp 
                               patch-src_ptex.imageio_ptex_Ptexture.h 
  Log:
  OpenImageIO is a library for reading and writing images, and a bunch of
  related classes, utilities, and applications.
  
  There is a particular emphasis on formats and functionality used
  in professional, large-scale animation and visual effects work for film.
  
  OpenImageIO is used extensively in animation and VFX studios all over the world,
  and is also incorporated into several commercial products.
  
  WWW: http://www.openimageio.org
  
  PR:             ports/166858
  Submitted by:   Shane Ambler
  
  Revision  Changes    Path
  1.1491    +1 -0      ports/graphics/Makefile
  1.1       +92 -0     ports/graphics/openimageio/Makefile (new)
  1.1       +2 -0      ports/graphics/openimageio/distinfo (new)
  1.1       +14 -0     ports/graphics/openimageio/files/patch-src_iv_imageviewer.cpp (new)
  1.1       +17 -0     ports/graphics/openimageio/files/patch-src_ptex.imageio_ptex_Ptexture.h (new)
  1.1       +10 -0     ports/graphics/openimageio/pkg-descr (new)
  1.1       +71 -0     ports/graphics/openimageio/pkg-plist (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 8 Pawel Pekala freebsd_committer freebsd_triage 2012-04-17 17:09:19 UTC
State Changed
From-To: open->closed

New port added. Thanks!
Comment 9 dfilter service freebsd_committer freebsd_triage 2012-04-17 17:09:25 UTC
pawel       2012-04-17 16:08:57 UTC

  FreeBSD doc repository (ports committer)

  Modified files:
    en_US.ISO8859-1/articles/contributors contrib.additional.sgml 
  Log:
  For graphics/openimageio
  
  PR:             ports/166858
  
  Revision  Changes    Path
  1.1080    +5 -0      doc/en_US.ISO8859-1/articles/contributors/contrib.additional.sgml
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"