[GRASS-dev] winGRASS not built/broken

Markus Metz markus.metz.giswork at gmail.com
Tue Nov 14 10:56:25 PST 2017


On Tue, Nov 14, 2017 at 3:13 PM, Even Rouault <even.rouault at spatialys.com>
wrote:
>
> > -->
>
> > Index: port/cpl_port.h
>
> > ===================================================================
>
> > --- port/cpl_port.h (revision 40701)
>
> > +++ port/cpl_port.h (working copy)
>
> > @@ -219,7 +219,7 @@
>
> > /* 64bit support */
>
> > /* --------------------------------------------------------------------
*/
>
> >
>
> > -#if defined(WIN32) && defined(_MSC_VER)
>
> > +#if defined(WIN32) && (defined(_MSC_VER) || defined(__MINGW32__))
>
> >
>
> > #define VSI_LARGE_API_SUPPORTED
>
> > typedef __int64 GIntBig;
>
> > <--
>
> >
>
> > to account for different compile environments on Windows that have
__int64
>
> Didn't know that mingw had __int64 as wel.
>
According to the documentation, yes. I tested, and __int64 is indeed
available with MinGW.
>
> Could you submit that as a pull request against
https://github.com/OSGeo/gdal so we can check that this doesn't break our 2
mingw targets at
> https://travis-ci.org/OSGeo/gdal ?

Done:
https://github.com/OSGeo/gdal/pull/262

Regarding integer types with MSC: why is HAVE_LONG_LONG not defined in
cpl_config.h for WIN32? There is a long long in WIN32 (equivalent to
__int64).

Markus M
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20171114/f64f45c1/attachment.html>


More information about the grass-dev mailing list