On Thu, Jul 28, 2011 at 12:21 AM, Sebastian Elsner <[email protected]> wrote:
> I sent this email to users mailing list, but found in the meantime, that
> some issues got resolved and the users does not get much attention. So
> please ignore the other one.
> I went through the procedure of compiling oiio (for win 64).

I'm not familiar with the windows build but I'll answer what I can.

First, which procedure is it that you're following exactly (where is it
written down, somewhere on the wiki?)

> * tbb - there is a checkbox in cmake for this to use but no way to specify a
> library path, so I have to do this for every project of my the VS 2008
> solution. What is tbb used for right now? There are no headers included...

If the USE_TBB option is activated, oiio will build with the bundled TBB
for atomic support and a few other things.  If not, it'll try to use
some simple atomic support built into OIIO.  You shouldn't have to set
any path here, and either option should work I think.

> * gtest - is it used?

gtest isn't used any more.

> * For VS 2008 it would make sense to add the /MP (multithreading) compiler
> flag, so we get multi processor speedup (VS 2005 does not support this
> though)

Good idea.  Does adding something like the snippet

if(MSVC AND NOT MSVC_VERSION LESS 1500)  # 1500 == VS 2008 according to web.
    add_definitions("/MP")
endif()

to the main CmakeLists.txt work?  I'm not sure if add_definitions is the
right way to do it, or whether we should modify CMAKE_CXX_FLAGS.

> * link also asks for /NODEFAULTLIB:LIBCMTD to be set

Huh, I've no idea about this is :)  Any windows people want to comment
on it?

> * I have been using OpenExr 1.7 trunk - the ilm library version cmake
> displays is not correctly read (finds 1.6.0). If this is of any
> importance....

Again no idea, sorry.  Is it just deducing the wrong version number somehow?

> * On windows "strtpf()" is not available, we should use "strtod()" instead.
> In oiiotool.cpp line 351

Easy to fix - we should add a strtpf() to fmath.h.  Please open an issue
on the github tracker (https://github.com/OpenImageIO/oiio/issues).

> * I would like to update some of the windows build info in the wiki, for
> example that you should use boost::filesystem with boost version <= 1.45
> (because filesystemv3 is default with newer versions and oiio relies on
> filesystemv2)

Yeah we need to fix OIIO for more recent boost versions I suppose.  I
recently fixed the same problem in another project, in that case I made
it compatible with both versions but it was a bit of a pain.  Please
open an issue on the tracker.

> * I could provide the 64bit "external" libraries I used, so they can be
>  posted alongside the existing 32bit libs.

I imagine this would be welcome, though someone else will have to chime
in.

Thanks,
~Chris
_______________________________________________
Oiio-dev mailing list
[email protected]
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

Reply via email to