Opened 16 years ago

Last modified 16 years ago

#215 closed defect (fixed)

Windows build of CVS 10/08/02 broken

Reported by: dgraham@… Owned by: dron
Priority: high Milestone:
Component: default Version: unspecified
Severity: normal Keywords:
Cc:

Description

Dear Frank and Andrey Kiselev:

I was just trying to build GDAL CVS from yesturday on Windows 2000 using Visual 
Studio 6.0 and found several problems involving the integration of the new 
Jpeg2000 driver.  But I have fixed these problems and will upload the files to 
this bug.  I would like the two of you to look over the changes and then commit 
them to CVS.

Dave Graham

Files I have had to change are

gdal/nmake.opt
gdal/makefile.vc
gdal/frmts/makefile.vc
gdal/frmts/gdalallregister.cpp
gdal/frmts/jpeg2000/makefile.vc
gdal/frmts/jpeg2000/jpeg2000dataset.cpp

Attachments (7)

Copy of nmake.opt (2.2 KB) - added by dgraham@… 16 years ago.
gdal/nmake.opt
makefile.vc (2.7 KB) - added by dgraham@… 16 years ago.
gdal/makefile.vc
makefile.2.vc (1002 bytes) - added by dgraham@… 16 years ago.
gdal/frmts/makefile.vc
gdalallregister.cpp (8.8 KB) - added by dgraham@… 16 years ago.
gdal/frmts/gdalallregister.cpp
jpeg2000dataset.cpp (20.0 KB) - added by dgraham@… 16 years ago.
gdal/frmts/jpeg2000/makefile.vc
makefile.3.vc (216 bytes) - added by dgraham@… 16 years ago.
gdal/frmts/jpeg2000/makefile.vc <- the real one this time
jpeg2000dataset.2.cpp (20.0 KB) - added by dgraham@… 16 years ago.
gdal/frmts/jpeg2000/jpeg2000dataset.cpp

Download all attachments as: .zip

Change History (11)

Changed 16 years ago by dgraham@…

Attachment: Copy of nmake.opt added

gdal/nmake.opt

Changed 16 years ago by dgraham@…

Attachment: makefile.vc added

gdal/makefile.vc

Changed 16 years ago by dgraham@…

Attachment: makefile.2.vc added

gdal/frmts/makefile.vc

Changed 16 years ago by dgraham@…

Attachment: gdalallregister.cpp added

gdal/frmts/gdalallregister.cpp

Changed 16 years ago by dgraham@…

Attachment: jpeg2000dataset.cpp added

gdal/frmts/jpeg2000/makefile.vc

Changed 16 years ago by dgraham@…

Attachment: makefile.3.vc added

gdal/frmts/jpeg2000/makefile.vc <- the real one this time

Changed 16 years ago by dgraham@…

Attachment: jpeg2000dataset.2.cpp added

gdal/frmts/jpeg2000/jpeg2000dataset.cpp

comment:1 Changed 16 years ago by warmerdam

Andrey,

Can you look into integrating these changes as appropriate? 

Thanks,

comment:2 Changed 16 years ago by dron

All patches applied (with few corrections).
The IReadBlock() function in the jpeg2000dataset.cpp may be changed in the
future, currently used David's solution.

Concerning gdalallregister.cpp: Frank, do you have plans to add some logic in
the configure script to determine presence of the Kakadu library? For now I just
comment out GDALRegister_JP2KAK().

Thank you, David!

comment:3 Changed 16 years ago by warmerdam

I do not plan to include configure logic for Kakadu at this time.  

comment:4 Changed 16 years ago by dgraham@…

Andrey,

Yes, I agree about the IReadBloeck function.  Not sure what the problem with the
VC++ 6.0 compile is.  What you did looked fine to me.  Makes me wonder about the
compiler.  I simply installed a workaround as you saw.

Dave
Note: See TracTickets for help on using tickets.