Re: [Openexr-devel] Compilng in MSVC without Intel compiler now.

[Drew Hess]

Hi Lutz,

Good news.  As you can probably tell from lack of activity on the list and 
in the CVS, we haven't had much time for updates lately.  I've got a 
couple of bug fixes pending, and I'll check out your diff once I've got a 
copy of .NET 2003.

If this works, we'll probably make .NET 2003 and Intel C++ the required
compilers for Win32.  People using previous versions of Visual Studio/C++ 
will need to use the C interface.

By the way, README.win32 already tells you exactly where to put zlib and 
FLTK.


-dwh-


 On Wed, 4 Jun 2003, Lutz Latta wrote:

> Hi everyone,
> I managed to compile OpenEXR on Windows with the new version of MS 
> VisualStudio .NET 2003 (MSVC7.1). They've been catching up with ISO 
> conformance for compiling templates, and it was rather easy to get the 
> MSVC6/Intel Compiler version ported.
> 
> I've attached a diff with the changes to this mail. To sum it up:
> 
> - I commented out a "hack" from the previous windows version, where the 
> min/max functions where added. The new compiler/STL matches them 
> correctly to STL min/max, without the hack. Commenting out the stuff now 
> isn't nice either, but I don't know what policy you have for compiler 
> detection code.
> 
> - According to MS the ISO rules require dependent type names from 
> template classes to be prefixed with the "typename" keyword. Like:
>                                                   vvv
> inline Euler<T>::Euler( const Matrix33<T> &M, typename Euler::Order p )
> 
> I assume this should be accepted (if not required as in MSVC7.1) by 
> other compilers as well.
> 
> 
> Additionally, I found two errors that I didn't debug/fix yet:
> 
> - The ImathTest program assert when testing the euler-matrix conversion 
> in extractEulerZYX() with order 2001.
> 
> - The IlmImfTest does not compile at testXdr.cpp:214 where an Xdr::read 
> is done with a "const char*" template argument, that the compiler can't 
> find an implementation for. I can't either, so my initial assumption is 
> that the test code is out of date.
> 
> 
> Otherwise everything works fine, even exrdisplay.
> 
> I think you should add to the readme (at least for the Windows version) 
> what other packages (=zlib, FLTK) are required and where to put them 
> (=in folders on level with the main OpenEXR folder) since Windows 
> doesn't have standard include and library paths.
> 
> Another suggestion for the next release: do NOT give the .tar.gz and 
> .zip files the same name, when one is the source distribution and the 
> other only contains windows binaries with headers. Maybe call the zip 
> file "win32-bin".
> 
> 
> Keep up the good work,
>     Lutz Latta
> 
>