You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Branko Čibej <br...@xbc.nu> on 2005/08/28 09:39:17 UTC

Re: Pytnon biindings build hosed on Windows

David James wrote:

>On 8/27/05, Branko Čibej <br...@xbc.nu> wrote:
>  
>
>>The Python bindings on HEAD of trunk don't compile on windows. Part of
>>this seems to be a bug in the runtime generator:
>>
>>Compiling...
>>swigutil_py.c
>>..\..\..\subversion\bindings\swig\proxy\swig_python_external_runtime.swg(1313) : warning C4003: not enough actual parameters for macro 'SWIG_GetModule'
>>..\..\..\subversion\bindings\swig\proxy\swig_python_external_runtime.swg(1319) : warning C4003: not enough actual parameters for macro 'SWIG_GetModule'
>>    
>>
>
>These warnings look like they were caused by a SWIG bug. The
>SWIG_GetModule macro is defined as follows by SWIG 1.3.25 on my
>machine:
>   #define SWIG_GetModule(clientdata) SWIG_Python_GetModule()
>
>Later in the file, SWIG uses the SWIG_GetModule macro as a zero-argument macro:
>  swig_module_info *module = SWIG_GetModule();
>
>GCC parses the above as follows:
>  swig_module_info *module = SWIG_Python_GetModule();
>
>Can Microsoft Visual C++ handle the above code? You said that Visual
>C++ reports warnings; does the code still work? If not, I can modify
>our runtime generator to work around this SWIG bug.
>  
>
Well, they're warnings, so the code does compile. I've no idea if it 
works, though, since I can't compile the bindings. :)

I'd be much happier if we got rid of the warnings, even if they're benign.

>>The other issue seems to be s SWIG problem:
>>
>>Performing Custom Build Step on ..\..\..\subversion\bindings\swig\core.i
>>C:\Home\brane\src\svn\httpd-2.0.54\srclib\apr\include\apr.h(326): Error: Syntax error in input.
>>
>>
>>This is the offending line in apr.h:
>>
>>typedef  unsigned __int64  apr_uint64_t;
>>    
>>
>
>It looks like SWIG can't handle the __int64 type. Perhaps we can fool
>SWIG by defining __int64 to "long long".
>  
>
I though about this, but I was confused because the line _before_ that 
one in apr.h reads:

    typedef  __int64  apr_int64_t;

and SWIG doesn't complain about it.
i'll try your patch anyway, will let you know how it goes.

-- Brane


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