[redland-dev] Building Redland 0.9.17 on WinXP w/full Cygwin install

[Dave Beckett]

On Tue, 25 May 2004, Jasper van de Gronde wrote:
> Dave Beckett wrote:
> > On Mon, 24 May 2004, Jasper van de Gronde wrote:
> It now looks like this on my system:
> #ifdef WIN32
> #  ifdef RAPTOR_STATIC
> #    define RAPTOR_API
> #  else
> #    ifdef RAPTOR_INTERNAL
> #      define RAPTOR_API _declspec(dllexport)
> #    else
> #      define RAPTOR_API _declspec(dllimport)
> #    endif
> #  endif
> #else
> #  define RAPTOR_API
> #endif

OK, merged into cvs

> I just needed this because I wanted to link statically to Raptor.

Fine.  I assume you are aware of the license issues of statically
linking to a GPL/GLPed library.  If you distribute it, it is a derived
work and must be under the same license.  The MPL alternative you
can read yourself for the conditions.

> >>    * raptor_general.c: Added a const to the first parameter of 
> >>raptor_print_statement.
> > 
> > Hmm?  It already has that:
> > raptor_print_statement(const raptor_statement * statement, FILE *stream)
> 
> Yes, but in the header it says:
> raptor_print_statement(const raptor_statement const * statement, FILE 
> *stream)
> It only generated a warning.

OK, fixed the header to match the C.

> > <snip/>
> > 
> > Actually I noticed these lines in win32_config.h :
> > 
> > /* for access() which is POSIX but doesn't seem to have the defines in VC */
> > #ifndef R_OK
> > #define R_OK 4
> > #endif
> > 
> > so maybe can they be removed if io.h is included?  Jose/others verify?
> 
> No, I tried and for some reason MSVC doesn't seem to have these defines, 
> even the help simply uses the numerical values (from the docs):
> mode Value Checks File For
> 00 Existence only
> 02  Write permission
> 04 Read permission
> 06 Read and write permission

OK, thanks

Dave