Re: [UPDATE] graphics/openimageio

2021-01-03 Thread Jeremie Courreges-Anglas
On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
> Update graphics/openimageio to 2.2.10.0
>
> Build & run tested blender with it.
> 14 out of 54 tests fail.
> Atleast 3 of them fail because of some
> optional dependencies not being turned on.
> Like OpenVDB,libheif and ptex.

> Removed the USE_DICOM option since it
> is optional now.

It may be optional, but it is still used if available:

(after a make configure)
--8<--
russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
dcmtk-3.6.3p0: ok
Read shared items: ok
russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee log2
===>  Building for openimageio-2.2.10.0
ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
-->8--

Optional deps for which we have ports should be explicitely enabled or
disabled, to avoid problems during bulk builds.  See the commit log:

  revision 1.44
  date: 2020/10/26 18:37:59;  author: jca;  state: Exp;  lines: +3 -2;  
commitid: tLqO00IGaKrpp8i9;
  Don't pick up dcmtk if available

  Force-disable the dep for now, can be reconsidered later if DICOM
  support is deemed valuable.

While here, openimageio is only used by blender, for which you're the
new maintainer.  Should you also take over maintainership of this port?

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: [UPDATE] graphics/openimageio

2021-01-04 Thread Stuart Henderson
On 2021/01/03 19:46, Jeremie Courreges-Anglas wrote:
> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
> > Update graphics/openimageio to 2.2.10.0
> >
> > Build & run tested blender with it.
> > 14 out of 54 tests fail.
> > Atleast 3 of them fail because of some
> > optional dependencies not being turned on.
> > Like OpenVDB,libheif and ptex.
> 
> > Removed the USE_DICOM option since it
> > is optional now.
> 
> It may be optional, but it is still used if available:
> 
> (after a make configure)
> --8<--
> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
> dcmtk-3.6.3p0: ok
> Read shared items: ok
> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee log2
> ===>  Building for openimageio-2.2.10.0
> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
> -->8--
> 
> Optional deps for which we have ports should be explicitely enabled or
> disabled, to avoid problems during bulk builds.  See the commit log:

yes - I'd go further actually, s/for which we have ports//. Then you
don't need to search the ports tree to see if they exist, and it also
covers the case where some library is added to ports later.

> While here, openimageio is only used by blender, for which you're the
> new maintainer.  Should you also take over maintainership of this port?

that would seem sensible!



Re: [UPDATE] graphics/openimageio

2021-01-04 Thread Pascal Stumpf
On Sun, 03 Jan 2021 19:46:29 +0100, Jeremie Courreges-Anglas wrote:
> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
> > Update graphics/openimageio to 2.2.10.0
> >
> > Build & run tested blender with it.
> > 14 out of 54 tests fail.
> > Atleast 3 of them fail because of some
> > optional dependencies not being turned on.
> > Like OpenVDB,libheif and ptex.
> 
> > Removed the USE_DICOM option since it
> > is optional now.
> 
> It may be optional, but it is still used if available:
> 
> (after a make configure)
> --8<--
> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
> dcmtk-3.6.3p0: ok
> Read shared items: ok
> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee log2
> ===>  Building for openimageio-2.2.10.0
> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
> -->8--
> 
> Optional deps for which we have ports should be explicitely enabled or
> disabled, to avoid problems during bulk builds.  See the commit log:
> 
>   revision 1.44
>   date: 2020/10/26 18:37:59;  author: jca;  state: Exp;  lines: +3 -2;  
> commitid: tLqO00IGaKrpp8i9;
>   Don't pick up dcmtk if available
> 
>   Force-disable the dep for now, can be reconsidered later if DICOM
>   support is deemed valuable.

Yes, this should be left in.

> While here, openimageio is only used by blender, for which you're the
> new maintainer.  Should you also take over maintainership of this port?

Yes, I think you should.  :)

> -- 
> jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: [UPDATE] graphics/openimageio

2021-01-05 Thread Jeremie Courreges-Anglas
On Mon, Jan 04 2021, Dimitri Karamazov  wrote:
> On Sun, Jan 03, 2021 at 07:46:29PM +0100, Jeremie Courreges-Anglas wrote:
>> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
>> > Update graphics/openimageio to 2.2.10.0
>> >
>> > Build & run tested blender with it.
>> > 14 out of 54 tests fail.
>> > Atleast 3 of them fail because of some
>> > optional dependencies not being turned on.
>> > Like OpenVDB,libheif and ptex.
>> 
>> > Removed the USE_DICOM option since it
>> > is optional now.
>> 
>> It may be optional, but it is still used if available:
>> 
>> (after a make configure)
>> --8<--
>> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
>> dcmtk-3.6.3p0: ok
>> Read shared items: ok
>> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee log2
>> ===>  Building for openimageio-2.2.10.0
>> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
>> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
>> -->8--
>> 
>> Optional deps for which we have ports should be explicitely enabled or
>> disabled, to avoid problems during bulk builds.  See the commit log:
>> 
>
> Oh, I should use a better method to search across ports.
> Couldn't find DICOM when I tried to.
>
> Hi pascal, I wonder if I can take maintainership of this port.
> It is only required by blender and the update to this are very
> frequent which I intend to take care of.
>
> Let me know what you think.
>
> Any comments,OK's?

LGTM ports-wise, I'll commit that later today unless someone objects or
beats me to it (ok jca@).

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: [UPDATE] graphics/openimageio

2021-01-05 Thread Jeremie Courreges-Anglas
On Mon, Jan 04 2021, Dimitri Karamazov  wrote:
> On Sun, Jan 03, 2021 at 07:46:29PM +0100, Jeremie Courreges-Anglas wrote:
>> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
>> > Update graphics/openimageio to 2.2.10.0
>> >
>> > Build & run tested blender with it.
>> > 14 out of 54 tests fail.
>> > Atleast 3 of them fail because of some
>> > optional dependencies not being turned on.
>> > Like OpenVDB,libheif and ptex.
>> 
>> > Removed the USE_DICOM option since it
>> > is optional now.
>> 
>> It may be optional, but it is still used if available:
>> 
>> (after a make configure)
>> --8<--
>> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
>> dcmtk-3.6.3p0: ok
>> Read shared items: ok
>> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee log2
>> ===>  Building for openimageio-2.2.10.0
>> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
>> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
>> -->8--
>> 
>> Optional deps for which we have ports should be explicitely enabled or
>> disabled, to avoid problems during bulk builds.  See the commit log:
>> 
>
> Oh, I should use a better method to search across ports.
> Couldn't find DICOM when I tried to.

pkglocate dicom does list dcmtk.  If that didn't help, looking at cvs
blame and then the relevant commit log usually does.

> Hi pascal, I wonder if I can take maintainership of this port.
> It is only required by blender and the update to this are very
> frequent which I intend to take care of.
>
> Let me know what you think.

Was there a particular reason for bringing robin-map as a separate
distfile?  Did you consider making a new port?  I'm not saying you
should, it doesn't seem to be widely used, I'm just being curious.

> Any comments,OK's?

Committed.

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: [UPDATE] graphics/openimageio

2021-01-06 Thread Dimitri Karamazov
On Tue, Jan 05, 2021 at 10:00:55PM +0100, Jeremie Courreges-Anglas wrote:
> On Mon, Jan 04 2021, Dimitri Karamazov  wrote:
> > On Sun, Jan 03, 2021 at 07:46:29PM +0100, Jeremie Courreges-Anglas wrote:
> >> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
> >> > Update graphics/openimageio to 2.2.10.0
> >> >
> >> > Build & run tested blender with it.
> >> > 14 out of 54 tests fail.
> >> > Atleast 3 of them fail because of some
> >> > optional dependencies not being turned on.
> >> > Like OpenVDB,libheif and ptex.
> >> 
> >> > Removed the USE_DICOM option since it
> >> > is optional now.
> >> 
> >> It may be optional, but it is still used if available:
> >> 
> >> (after a make configure)
> >> --8<--
> >> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
> >> dcmtk-3.6.3p0: ok
> >> Read shared items: ok
> >> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee 
> >> log2
> >> ===>  Building for openimageio-2.2.10.0
> >> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
> >> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
> >> -->8--
> >> 
> >> Optional deps for which we have ports should be explicitely enabled or
> >> disabled, to avoid problems during bulk builds.  See the commit log:
> >> 
> >
> > Oh, I should use a better method to search across ports.
> > Couldn't find DICOM when I tried to.
> 
> pkglocate dicom does list dcmtk.  If that didn't help, looking at cvs
> blame and then the relevant commit log usually does.
> 
This was useful. Thanks.

> > Hi pascal, I wonder if I can take maintainership of this port.
> > It is only required by blender and the update to this are very
> > frequent which I intend to take care of.
> >
> > Let me know what you think.
> 
> Was there a particular reason for bringing robin-map as a separate
> distfile?  Did you consider making a new port?  I'm not saying you
> should, it doesn't seem to be widely used, I'm just being curious.
> 

It is required only during build, if it was necessary at runtime
I'll have no choice but to make a port. So I took the easy path.
Even in some of the biggest package repos it is only required
by openimageio.

But do you see it as a useful addition to ports? If it is so it will
take me just a minute make one. I thought it was a submodule at first
since instead of dumping an error it tried to fetch during configure,
so adding it as distfile felt only natural to me then.

In the next update I'll turn this in as a port since it is now known
to me to be just another dep.

regards,
  Dimitri



Re: [UPDATE] graphics/openimageio

2021-01-07 Thread Jeremie Courreges-Anglas
On Wed, Jan 06 2021, Dimitri Karamazov  wrote:
> On Tue, Jan 05, 2021 at 10:00:55PM +0100, Jeremie Courreges-Anglas wrote:
>> On Mon, Jan 04 2021, Dimitri Karamazov  wrote:
>> > On Sun, Jan 03, 2021 at 07:46:29PM +0100, Jeremie Courreges-Anglas wrote:
>> >> On Sun, Jan 03 2021, Dimitri Karamazov  wrote:
>> >> > Update graphics/openimageio to 2.2.10.0
>> >> >
>> >> > Build & run tested blender with it.
>> >> > 14 out of 54 tests fail.
>> >> > Atleast 3 of them fail because of some
>> >> > optional dependencies not being turned on.
>> >> > Like OpenVDB,libheif and ptex.
>> >> 
>> >> > Removed the USE_DICOM option since it
>> >> > is optional now.
>> >> 
>> >> It may be optional, but it is still used if available:
>> >> 
>> >> (after a make configure)
>> >> --8<--
>> >> russell /usr/ports/graphics/openimageio$ doas pkg_delete dcmtk
>> >> dcmtk-3.6.3p0: ok
>> >> Read shared items: ok
>> >> russell /usr/ports/graphics/openimageio$ make all MAKE_JOBS=2 2>&1 | tee 
>> >> log2
>> >> ===>  Building for openimageio-2.2.10.0
>> >> ninja: error: '/usr/local/lib/libdcmimage.so.0.0', needed by
>> >> 'lib/libOpenImageIO.so.8.0', missing and no known rule to make it
>> >> -->8--
>> >> 
>> >> Optional deps for which we have ports should be explicitely enabled or
>> >> disabled, to avoid problems during bulk builds.  See the commit log:
>> >> 
>> >
>> > Oh, I should use a better method to search across ports.
>> > Couldn't find DICOM when I tried to.
>> 
>> pkglocate dicom does list dcmtk.  If that didn't help, looking at cvs
>> blame and then the relevant commit log usually does.
>> 
> This was useful. Thanks.
>
>> > Hi pascal, I wonder if I can take maintainership of this port.
>> > It is only required by blender and the update to this are very
>> > frequent which I intend to take care of.
>> >
>> > Let me know what you think.
>> 
>> Was there a particular reason for bringing robin-map as a separate
>> distfile?  Did you consider making a new port?  I'm not saying you
>> should, it doesn't seem to be widely used, I'm just being curious.
>> 
>
> It is required only during build, if it was necessary at runtime
> I'll have no choice but to make a port. So I took the easy path.
> Even in some of the biggest package repos it is only required
> by openimageio.

Header-only and indeed not widely used.  Or maybe other packagers took
the same approach as you so it's not widely visible.

> But do you see it as a useful addition to ports? If it is so it will
> take me just a minute make one. I thought it was a submodule at first
> since instead of dumping an error it tried to fetch during configure,
> so adding it as distfile felt only natural to me then.
>
> In the next update I'll turn this in as a port since it is now known
> to me to be just another dep.

While we usually prefer to have a single port to watch for problems,
instead of embedded copies, here the benefit to have a port appears
limited.  As far as I'm concerned, just pick the approach that you
prefer.

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: update: graphics/openimageio

2013-05-09 Thread Aaron Bieber
On Sun, May 05, 2013 at 01:01:19PM +0200, Pascal Stumpf wrote:
> Update to latest version 1.1.10.
> 

This fails for me on amd64:

cd /usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO && 
/usr/local/bin/cmake -E cmake_link_script CMakeFiles/atomic_test.dir/link.txt 
--verbose=1
/usr/bin/c++   -O2 -pipe  -pthread  -DNDEBUG
CMakeFiles/atomic_test.dir/atomic_test.cpp.o  -o atomic_test  -L/usr/local/lib  
-L/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO  -lOpenImageIO 
/usr/local/lib/libboost_filesystem-mt.so.5.0 
/usr/local/lib/libboost_regex-mt.so.5.0 
/usr/local/lib/libboost_system-mt.so.5.0 
/usr/local/lib/libboost_thread-mt.so.5.0 /usr/X11R6/lib/libfreetype.so.19.0 
/usr/local/lib/libHalf.so.4.0 /usr/local/lib/libIex.so.4.0 
/usr/local/lib/libImath.so.4.0 /usr/local/lib/libIlmThread.so.4.0 
/usr/local/lib/libpng.so.15.1 /usr/lib/libz.so.4.1 
/usr/local/lib/libtiff.so.39.2 /usr/local/lib/libjpeg.so.66.0 
/usr/lib/libz.so.4.1 /usr/local/lib/libtiff.so.39.2 
/usr/local/lib/libjpeg.so.66.0 /usr/local/lib/libopenjpeg.so.5.0 
/usr/local/lib/libwebp.so.1.0 /usr/local/lib/libopenjpeg.so.5.0 
/usr/local/lib/libwebp.so.1.0 /usr/local/lib/libIlmImf.so.4.0 
-Wl,-rpath,/usr/local/lib:/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO:/usr/X11R6/!
 lib -Wl,-rpath-link,/usr/X11R6/lib:/usr/local/lib 
/usr/local/lib/libopenjpeg.so.5.0: warning: vsprintf() is often misused, please 
use vsnprintf()
/usr/local/lib/libboost_regex-mt.so.5.0: warning: wcscpy() is almost always 
misused, please use wcslcpy()
/usr/local/lib/libOpenImageIO.so.0.0: warning: strcpy() is almost always 
misused, please use strlcpy()
/usr/local/lib/libOpenImageIO.so.0.0: warning: strcat() is almost always 
misused, please use strlcat()
/usr/local/lib/libOpenImageIO.so.0.0: warning: sprintf() is often misused, 
please use snprintf()
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x4d0): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::ArgParse(int, char 
const**)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x5e1): In function `main':
: undefined reference to `OpenImageIO::v1_1::ustring::make_unique(char const*)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x79b): In function `main':
: undefined reference to `OpenImageIO::v1_1::ustring::make_unique(char const*)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x944): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::options(char const*, 
...)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x95b): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::parse(int, char const**)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x979): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::~ArgParse()'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xaf4): In function `main':
: undefined reference to 
`OpenImageIO::v1_1::Strutil::timeintervalformat(double, int)'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xb95): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::geterror() const'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xbf8): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::usage() const'
CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xcca): In function `main':
: undefined reference to `OpenImageIO::v1_1::ArgParse::~ArgParse()'
collect2: ld returned 1 exit status
*** Error 1 in . (libOpenImageIO/CMakeFiles/atomic_test.dir/build.make:111 
'libOpenImageIO/atomic_test')
*** Error 1 in . (CMakeFiles/Makefile2:1017 
'libOpenImageIO/CMakeFiles/atomic_test.dir/all')
*** Error 1 in /usr/ports/pobj/openimageio-1.1.10/build-amd64 (Makefile:147 
'all')
*** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2641 
'/usr/ports/pobj/openimageio-1.1.10/build-amd64/.build_done')
*** Error 1 in /usr/ports/graphics/openimageio 
(/usr/ports/infrastructure/mk/bsd.port.mk:2360 'all')

> 
> Index: Makefile
> ===
> RCS file: /home/pascal/cvs/ports/graphics/openimageio/Makefile,v
> retrieving revision 1.5
> diff -u -p -r1.5 Makefile
> --- Makefile  11 Mar 2013 11:10:54 -  1.5
> +++ Makefile  3 May 2013 20:25:21 -
> @@ -6,14 +6,11 @@ COMMENT =   library for reading and writi
>  BROKEN-hppa =missing atomic ops
>  BROKEN-mips64 =  missing atomic ops
>  
> -MAJ =1.0
> -V =  ${MAJ}.4
> -REV =513d0dd
> -DISTNAME =   OpenImageIO-oiio-Release-${V}-0-g${REV}
> +V =  1.1.10
> +DISTNAME =   Release-${V}
>  PKGNAME =openimageio-${V}
> -REVISION =   0
>  
> -SHARED_LIBS +=   OpenImageIO 0.0 # 1.0
> +SHARED_LIBS +=   OpenImageIO 1.0 # 1.0
>  
>  CATEGORIES = graphics devel
>  
> @@ -28,8 +25,10 @@ WANTLIB += GL GLEW GLU Half ICE Iex IlmI
>  WA

Re: update: graphics/openimageio

2013-05-09 Thread Landry Breuil
On Thu, May 09, 2013 at 07:19:06AM -0600, Aaron Bieber wrote:
> On Sun, May 05, 2013 at 01:01:19PM +0200, Pascal Stumpf wrote:
> > Update to latest version 1.1.10.
> > 
> 
> This fails for me on amd64:

Most likely because you already have the previous version and it takes
preference over the newly built one ?

Landry

> cd /usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO && 
> /usr/local/bin/cmake -E cmake_link_script CMakeFiles/atomic_test.dir/link.txt 
> --verbose=1
> /usr/bin/c++   -O2 -pipe  -pthread  -DNDEBUG
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o  -o atomic_test  
> -L/usr/local/lib  
> -L/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO  
> -lOpenImageIO /usr/local/lib/libboost_filesystem-mt.so.5.0 
> /usr/local/lib/libboost_regex-mt.so.5.0 
> /usr/local/lib/libboost_system-mt.so.5.0 
> /usr/local/lib/libboost_thread-mt.so.5.0 /usr/X11R6/lib/libfreetype.so.19.0 
> /usr/local/lib/libHalf.so.4.0 /usr/local/lib/libIex.so.4.0 
> /usr/local/lib/libImath.so.4.0 /usr/local/lib/libIlmThread.so.4.0 
> /usr/local/lib/libpng.so.15.1 /usr/lib/libz.so.4.1 
> /usr/local/lib/libtiff.so.39.2 /usr/local/lib/libjpeg.so.66.0 
> /usr/lib/libz.so.4.1 /usr/local/lib/libtiff.so.39.2 
> /usr/local/lib/libjpeg.so.66.0 /usr/local/lib/libopenjpeg.so.5.0 
> /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libopenjpeg.so.5.0 
> /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libIlmImf.so.4.0 
> -Wl,-rpath,/usr/local/lib:/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO:/usr/X11R!
 6/!
>  lib -Wl,-rpath-link,/usr/X11R6/lib:/usr/local/lib 
> /usr/local/lib/libopenjpeg.so.5.0: warning: vsprintf() is often misused, 
> please use vsnprintf()
> /usr/local/lib/libboost_regex-mt.so.5.0: warning: wcscpy() is almost always 
> misused, please use wcslcpy()
> /usr/local/lib/libOpenImageIO.so.0.0: warning: strcpy() is almost always 
> misused, please use strlcpy()
> /usr/local/lib/libOpenImageIO.so.0.0: warning: strcat() is almost always 
> misused, please use strlcat()
> /usr/local/lib/libOpenImageIO.so.0.0: warning: sprintf() is often misused, 
> please use snprintf()
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x4d0): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::ArgParse(int, char 
> const**)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x5e1): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ustring::make_unique(char 
> const*)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x79b): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ustring::make_unique(char 
> const*)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x944): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::options(char const*, 
> ...)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x95b): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::parse(int, char 
> const**)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x979): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::~ArgParse()'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xaf4): In function `main':
> : undefined reference to 
> `OpenImageIO::v1_1::Strutil::timeintervalformat(double, int)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xb95): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::geterror() const'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xbf8): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::usage() const'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xcca): In function `main':
> : undefined reference to `OpenImageIO::v1_1::ArgParse::~ArgParse()'
> collect2: ld returned 1 exit status
> *** Error 1 in . (libOpenImageIO/CMakeFiles/atomic_test.dir/build.make:111 
> 'libOpenImageIO/atomic_test')
> *** Error 1 in . (CMakeFiles/Makefile2:1017 
> 'libOpenImageIO/CMakeFiles/atomic_test.dir/all')
> *** Error 1 in /usr/ports/pobj/openimageio-1.1.10/build-amd64 (Makefile:147 
> 'all')
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2641 
> '/usr/ports/pobj/openimageio-1.1.10/build-amd64/.build_done')
> *** Error 1 in /usr/ports/graphics/openimageio 
> (/usr/ports/infrastructure/mk/bsd.port.mk:2360 'all')
> 
> > 
> > Index: Makefile
> > ===
> > RCS file: /home/pascal/cvs/ports/graphics/openimageio/Makefile,v
> > retrieving revision 1.5
> > diff -u -p -r1.5 Makefile
> > --- Makefile11 Mar 2013 11:10:54 -  1.5
> > +++ Makefile3 May 2013 20:25:21 -
> > @@ -6,14 +6,11 @@ COMMENT = library for reading and writi
> >  BROKEN-hppa =  missing atomic ops
> >  BROKEN-mips64 =missing atomic ops
> >  
> > -MAJ =  1.0
> > -V =${MAJ}.4
> > -REV =  513d0dd
> > -DISTNAME = OpenImageIO-oiio-Release-${V}-0-g${R

Re: update: graphics/openimageio

2013-05-09 Thread Aaron Bieber
On Thu, May 09, 2013 at 03:48:03PM +0200, Pascal Stumpf wrote:
> On Thu, 9 May 2013 07:19:06 -0600, Aaron Bieber wrote:
> > On Sun, May 05, 2013 at 01:01:19PM +0200, Pascal Stumpf wrote: > Update
> > to latest version 1.1.10. >
> > 
> > This fails for me on amd64:
> > 
> > cd /usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO
> > && /usr/local/bin/cmake -E cmake_link_script
> > CMakeFiles/atomic_test.dir/link.txt --verbose=1 /usr/bin/c++ -O2
> > -pipe -pthread -DNDEBUG CMakeFiles/atomic_test.dir/atomic_test.cpp.o
> > -o atomic_test -L/usr/local/lib
> > -L/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO
> > -lOpenImageIO /usr/local/lib/libboost_filesystem-mt.so.5.0
> > /usr/local/lib/libboost_regex-mt.so.5.0
> > /usr/local/lib/libboost_system-mt.so.5.0
> > /usr/local/lib/libboost_thread-mt.so.5.0
> > /usr/X11R6/lib/libfreetype.so.19.0 /usr/local/lib/libHalf.so.4.0
> > /usr/local/lib/libIex.so.4.0 /usr/local/lib/libImath.so.4.0
> > /usr/local/lib/libIlmThread.so.4.0
> > /usr/local/lib/libpng.so.15.1 /usr/lib/libz.so.4.1
> > /usr/local/lib/libtiff.so.39.2 /usr/local/lib/libjpeg.so.66.0
> > /usr/lib/libz.so.4.1 /usr/local/lib/libtiff.so.39.2
> > /usr/local/lib/libjpeg.so.66.0 /usr/local/lib/libopenjpeg.so.5.0
> > /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libopenjpeg.so.5.0
> > /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libIlmImf.so.4.0
> > -Wl,-rpath,/usr/local/lib:/usr/ports/pobj/openimageio-1.1.10/build-amd64
> > /libOpenImageIO:/usr/X11R6/lib
> > -Wl,-rpath-link,/usr/X11R6/lib:/usr/local/lib
> > /usr/local/lib/libopenjpeg.so.5.0: warning: vsprintf() is often misused,
> > please use vsnprintf() /usr/local/lib/libboost_regex-mt.so.5.0:
> > warning: wcscpy() is almost always misused, please use
> > wcslcpy() /usr/local/lib/libOpenImageIO.so.0.0: warning:
> 
> Ugh, looks like it's picking up an already installed version.  Hmm, is
> it worth it to fix this kind of errors?

Uninstalled the previous version and it built fine. Seems like it should
be something that is fixed, but I am assuming bulk builds are something
that end users might do at some point.

Landry, what do you think? 

> 
> > strcpy() is almost always misused, please use strlcpy()
> > /usr/local/lib/libOpenImageIO.so.0.0: warning: strcat() is almost always
> > misused, please use strlcat() /usr/local/lib/libOpenImageIO.so.0.0:
> > warning: sprintf() is often misused, please use snprintf()
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x4d0):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::ArgParse(int, char const**)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x5e1):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x79b):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x944):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::options(char const*, ...)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x95b):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::parse(int, char const**)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x979):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::~ArgParse()'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xaf4):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::Strutil::timeintervalformat(double, int)'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xb95):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::geterror() const'
> > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xbf8): In function
> > `main': : undefined reference to `OpenImageIO::v1_1::ArgParse::usage()
> > const' CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xcca):
> > In function `main': : undefined reference to
> > `OpenImageIO::v1_1::ArgParse::~ArgParse()' collect2:
> > ld returned 1 exit status *** Error 1 in .
> > (libOpenImageIO/CMakeFiles/atomic_test.dir/build.make:111
> > 'libOpenImageIO/atomic_test') *** Error
> > 1 in . (CMakeFiles/Makefile2:1017
> > 'libOpenImageIO/CMakeFiles/atomic_test.dir/all') *** Error 1 in
> > /usr/ports/pobj/openimageio-1.1.10/build-amd64 (Makefile:147 'all')
> > *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2641
> > '/usr/ports/pobj/openimageio-1.1.10/build-amd64/.build_done')
> > *** Error 1 in /usr/ports/graphics/openimageio
> > (/usr/ports/infrastructure/mk/bsd.port.mk:2360 'all')
> > 
> > >
> > > Index: Makefile
> > > ===
> > > RCS file: /home/pascal/cvs/ports/graphics/openimageio/Makefile,v
> > > retrieving revision 1.5 diff -u -p -r1.5 Makefile --- Makefile 11 Mar
> > > 2013 11:10:54 - 1.5 +++ Makefile 3 May 2013 20:25:21 -

Re: update: graphics/openimageio

2013-05-09 Thread Pascal Stumpf
On Thu, 9 May 2013 07:19:06 -0600, Aaron Bieber wrote:
> On Sun, May 05, 2013 at 01:01:19PM +0200, Pascal Stumpf wrote: > Update
> to latest version 1.1.10. >
> 
> This fails for me on amd64:
> 
> cd /usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO
> && /usr/local/bin/cmake -E cmake_link_script
> CMakeFiles/atomic_test.dir/link.txt --verbose=1 /usr/bin/c++ -O2
> -pipe -pthread -DNDEBUG CMakeFiles/atomic_test.dir/atomic_test.cpp.o
> -o atomic_test -L/usr/local/lib
> -L/usr/ports/pobj/openimageio-1.1.10/build-amd64/libOpenImageIO
> -lOpenImageIO /usr/local/lib/libboost_filesystem-mt.so.5.0
> /usr/local/lib/libboost_regex-mt.so.5.0
> /usr/local/lib/libboost_system-mt.so.5.0
> /usr/local/lib/libboost_thread-mt.so.5.0
> /usr/X11R6/lib/libfreetype.so.19.0 /usr/local/lib/libHalf.so.4.0
> /usr/local/lib/libIex.so.4.0 /usr/local/lib/libImath.so.4.0
> /usr/local/lib/libIlmThread.so.4.0
> /usr/local/lib/libpng.so.15.1 /usr/lib/libz.so.4.1
> /usr/local/lib/libtiff.so.39.2 /usr/local/lib/libjpeg.so.66.0
> /usr/lib/libz.so.4.1 /usr/local/lib/libtiff.so.39.2
> /usr/local/lib/libjpeg.so.66.0 /usr/local/lib/libopenjpeg.so.5.0
> /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libopenjpeg.so.5.0
> /usr/local/lib/libwebp.so.1.0 /usr/local/lib/libIlmImf.so.4.0
> -Wl,-rpath,/usr/local/lib:/usr/ports/pobj/openimageio-1.1.10/build-amd64
> /libOpenImageIO:/usr/X11R6/lib
> -Wl,-rpath-link,/usr/X11R6/lib:/usr/local/lib
> /usr/local/lib/libopenjpeg.so.5.0: warning: vsprintf() is often misused,
> please use vsnprintf() /usr/local/lib/libboost_regex-mt.so.5.0:
> warning: wcscpy() is almost always misused, please use
> wcslcpy() /usr/local/lib/libOpenImageIO.so.0.0: warning:

Ugh, looks like it's picking up an already installed version.  Hmm, is
it worth it to fix this kind of errors?

> strcpy() is almost always misused, please use strlcpy()
> /usr/local/lib/libOpenImageIO.so.0.0: warning: strcat() is almost always
> misused, please use strlcat() /usr/local/lib/libOpenImageIO.so.0.0:
> warning: sprintf() is often misused, please use snprintf()
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x4d0):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::ArgParse(int, char const**)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x5e1):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x79b):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x944):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::options(char const*, ...)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x95b):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::parse(int, char const**)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x979):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::~ArgParse()'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xaf4):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::Strutil::timeintervalformat(double, int)'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xb95):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::geterror() const'
> CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xbf8): In function
> `main': : undefined reference to `OpenImageIO::v1_1::ArgParse::usage()
> const' CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xcca):
> In function `main': : undefined reference to
> `OpenImageIO::v1_1::ArgParse::~ArgParse()' collect2:
> ld returned 1 exit status *** Error 1 in .
> (libOpenImageIO/CMakeFiles/atomic_test.dir/build.make:111
> 'libOpenImageIO/atomic_test') *** Error
> 1 in . (CMakeFiles/Makefile2:1017
> 'libOpenImageIO/CMakeFiles/atomic_test.dir/all') *** Error 1 in
> /usr/ports/pobj/openimageio-1.1.10/build-amd64 (Makefile:147 'all')
> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2641
> '/usr/ports/pobj/openimageio-1.1.10/build-amd64/.build_done')
> *** Error 1 in /usr/ports/graphics/openimageio
> (/usr/ports/infrastructure/mk/bsd.port.mk:2360 'all')
> 
> >
> > Index: Makefile
> > ===
> > RCS file: /home/pascal/cvs/ports/graphics/openimageio/Makefile,v
> > retrieving revision 1.5 diff -u -p -r1.5 Makefile --- Makefile 11 Mar
> > 2013 11:10:54 - 1.5 +++ Makefile 3 May 2013 20:25:21 - @@
> > -6,14 +6,11 @@ COMMENT = library for reading and writi BROKEN-hppa =
> > missing atomic ops BROKEN-mips64 = missing atomic ops
> >
> > -MAJ = 1.0 -V = ${MAJ}.4 -REV = 513d0dd -DISTNAME =
> > OpenImageIO-oiio-Release-${V}-0-g${REV} +V = 1.1.10 +DISTNAME =
> > Release-${V} PKGNAME = openimageio-${V} -REVISION = 0
> >
> > -SHARED_LIBS += OpenImageIO 0 +SHARED_LIBS += OpenImageIO 1   .0 # 1.0
> >
> >  CATEGORIES = graphics devel
> >
> > @

Re: update: graphics/openimageio

2013-05-09 Thread Stuart Henderson
On 2013/05/09 08:12, Aaron Bieber wrote:
> 
> Uninstalled the previous version and it built fine. Seems like it should
> be something that is fixed, but I am assuming bulk builds are something
> that end users might do at some point.

It's not uncommon to have ports which won't build if an older version
is installed, I don't think it's a major problem. If it's easy to fix
without too much mess then why not, but I wouldn't go overboard patching
things to make it work..

> Landry, what do you think? 
> 
> > 
> > > strcpy() is almost always misused, please use strlcpy()
> > > /usr/local/lib/libOpenImageIO.so.0.0: warning: strcat() is almost always
> > > misused, please use strlcat() /usr/local/lib/libOpenImageIO.so.0.0:
> > > warning: sprintf() is often misused, please use snprintf()
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x4d0):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::ArgParse(int, char const**)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x5e1):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x79b):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ustring::make_unique(char const*)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x944):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::options(char const*, ...)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x95b):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::parse(int, char const**)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0x979):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::~ArgParse()'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xaf4):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::Strutil::timeintervalformat(double, int)'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xb95):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::geterror() const'
> > > CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xbf8): In function
> > > `main': : undefined reference to `OpenImageIO::v1_1::ArgParse::usage()
> > > const' CMakeFiles/atomic_test.dir/atomic_test.cpp.o(.text+0xcca):
> > > In function `main': : undefined reference to
> > > `OpenImageIO::v1_1::ArgParse::~ArgParse()' collect2:
> > > ld returned 1 exit status *** Error 1 in .
> > > (libOpenImageIO/CMakeFiles/atomic_test.dir/build.make:111
> > > 'libOpenImageIO/atomic_test') *** Error
> > > 1 in . (CMakeFiles/Makefile2:1017
> > > 'libOpenImageIO/CMakeFiles/atomic_test.dir/all') *** Error 1 in
> > > /usr/ports/pobj/openimageio-1.1.10/build-amd64 (Makefile:147 'all')
> > > *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2641
> > > '/usr/ports/pobj/openimageio-1.1.10/build-amd64/.build_done')
> > > *** Error 1 in /usr/ports/graphics/openimageio
> > > (/usr/ports/infrastructure/mk/bsd.port.mk:2360 'all')
> > > 
> > > >
> > > > Index: Makefile
> > > > ===
> > > > RCS file: /home/pascal/cvs/ports/graphics/openimageio/Makefile,v
> > > > retrieving revision 1.5 diff -u -p -r1.5 Makefile --- Makefile 11 Mar
> > > > 2013 11:10:54 - 1.5 +++ Makefile 3 May 2013 20:25:21 - @@
> > > > -6,14 +6,11 @@ COMMENT = library for reading and writi BROKEN-hppa =
> > > > missing atomic ops BROKEN-mips64 = missing atomic ops
> > > >
> > > > -MAJ = 1.0 -V = ${MAJ}.4 -REV = 513d0dd -DISTNAME =
> > > > OpenImageIO-oiio-Release-${V}-0-g${REV} +V = 1.1.10 +DISTNAME =
> > > > Release-${V} PKGNAME = openimageio-${V} -REVISION = 0
> > > >
> > > > -SHARED_LIBS += OpenImageIO 0 +SHARED_LIBS += OpenImageIO 1   .0 # 1.0
> > > >
> > > >  CATEGORIES = graphics devel
> > > >
> > > > @@ -28,8 +25,10 @@ WANTLIB += GL GLEW GLU Half ICE Iex IlmI WANTLIB
> > > > += QtOpenGL SM X11 Xext boost_filesystem-mt boost_python-mt WANTLIB
> > > > += boost_regex-mt boost_system-mt boost_thread-mt c jpeg WANTLIB += m
> > > > openjpeg png pthread ${MODPY_WANTLIB} stdc++ tiff webp z +WANTLIB +=
> > > > freetype
> > > >
> > > > -MASTER_SITES = https://github.com/OpenImageIO/oiio/tarball/RB-${MAJ}/
> > > > +MASTER_SITES = https://github.com/OpenImageIO/oiio/archive/
> > > > +DIST_SUBDIR = openimageio
> > > >
> > > >  MODULES = devel/cmake \ lang/python \ @@ -45,7 +44,10
> > > > @@ LIB_DEPENDS = devel/boost \ CONFIGURE_ARGS +=
> > > > -DPYLIB_INSTALL_DIR="lib/python${MODPY_VERSION}/site-packages"
> > > >
> > > >  CFLAGS += -pthread -WRKDIST = ${WRKDIR}/OpenImageIO-oiio-${REV}
> > > > +WRKDIST = ${WRKDIR}/oiio-Release-$V WRKSRC = ${WRKDIST}/src +
> > > > +post-install: + find ${PREFIX} -name '*.orig' -exec rm -f {} \;
> > > >
> > > >  .include  Index: distinfo
> > > > ===