[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: viewcvs not working anymore on windows with current version o f tr unk due to changes in apr.i

From: Marc Haesen <marc.haesen_at_telindus.com>
Date: 2004-02-26 07:44:54 CET

The latest commit of apr.i and svn_types.i solved the problems on windows.

I have 2 files attached: my version of apr.h and swigerror.txt, which
contains the error report when I pass -DWIN32 on the swig command line.

Regards,
Marc

-----Original Message-----
From: Ben Reser [mailto:ben@reser.org]
Sent: Wednesday, February 25, 2004 08:33 PM
To: 'dev@subversion.tigris.org'
Subject: Re: viewcvs not working anymore on windows with current version o f
tr unk due to changes in apr.i

On Wed, Feb 25, 2004 at 07:54:39PM +0100, Marc Haesen wrote:
> This patch not really solves the problem. The reason is that
APR_INT64_T_FMT
> is only defined if you pass -DWIN32 on the command line of swig, which is
> not the case in the automatically generated project files. But if I change
> this, swig gives a syntax error in apr.h for the following declaration:
>
> typedef unsigned __int64 apr_uint64_t;
>
> If I remove this line from apr.h, it is OK. (For swig at least, because
with
> this line removed apr itself will not build anymore)
> The syntax error occurs both in version 1.3.19 and 1.3.21 of swig.

So what you're saying is SWIG doesn't like APR's header files on
Windows. Which means there's also a SWIG bug.

Could you give the specific error SWIG is giving you?

Can you please post your apr.h file?

> I am also not really sure if swig handles the || operator in the #if
> statement correctly.

It should.

We should always just handling things like this:

#if <format conditions for 64-bit platforms we know about>
  /* Use long */
#elsif <format conditions for 32-bit platforms we know about>
  /* Use long long */
#else
  /* Use long long since it should always be 64-bit but will
     emit warnings on some platforms. */
#endif

That gives us a fall through case for other platforms with weird
formats.

> I saw that you already committed svn_types.i, but a similar problem exists
> in apr.i, which is not changed yet.

Yup I'll do that one myself here in a sec.

--
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Visit us at InfoSecurity - The Largest Security Fair in the world!
10 and 11 March - Brussels Kart Expo - Booth 1.B.050
Get your free tickets on www.telindus.be
    



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

  • application/octet-stream attachment: apr.h
Received on Thu Feb 26 07:44:25 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.