Bug when building SWIG wrappers on (at least) OSX

[Paul Wagland <pa...@kungfoocoder.org>]

Hi all,

While trying to install subversion using home-brew on OSX, I came across a few build issues. The patches for these can be seen here: https://github.com/mxcl/homebrew/pull/22668/files#r6449557

Is any further information required from me before an issue can be created? Or is there anything else that is needed to get this patch included?

Thanks in advance,
Paul

Re: Bug when building SWIG wrappers on (at least) OSX

[Paul Wagland <pa...@kungfoocoder.org>]

On 19 Sep 2013, at 08:31, Ben Reser <be...@reser.org> wrote:

> On Wed Sep 18 23:02:31 2013, Paul Wagland wrote:
>> I can see that for the svn_auth.h, however I don’t see how rerunning autogen.h will change the order of the includes in swigutil_pl.c? What should the proper fix for that be?
> 
> Why exactly did you need to do that?  I have no problem building the 
> Perl bindings on OS X with the existing order.

Correct! The bindings build, however when I was building in parallel, then it didn't work. Now that the parallel building has been disabled it appears to work without needing that patch.

Thanks for following up!

Cheers,
Paul

Re: Bug when building SWIG wrappers on (at least) OSX

[Ben Reser <be...@reser.org>]

On Wed Sep 18 23:02:31 2013, Paul Wagland wrote:
> I can see that for the svn_auth.h, however I don’t see how rerunning autogen.h will change the order of the includes in swigutil_pl.c? What should the proper fix for that be?

Why exactly did you need to do that?  I have no problem building the 
Perl bindings on OS X with the existing order.

Re: Bug when building SWIG wrappers on (at least) OSX

[Paul Wagland <pa...@kungfoocoder.org>]

On 19 Sep 2013, at 06:48, Ben Reser <be...@reser.org> wrote:

> On Wed Sep 18 16:00:22 2013, Paul Wagland wrote:
>> While trying to install subversion using home-brew on OSX, I came across a few build issues. The patches for these can be seen here: https://github.com/mxcl/homebrew/pull/22668/files#r6449557
> 
> This is a known issue with the auth system API.  While that patch fixes 
> compilation on OS X it can't be applied because it breaks the Gnome 
> keyring support on OSes that support it via the perl bindings.  It also 
> completely ignores the same issue that happens with the Ruby and Python 
> bindings.

Thanks for the update.

I can see that for the svn_auth.h, however I don’t see how rerunning autogen.h will change the order of the includes in swigutil_pl.c? What should the proper fix for that be?

Cheers,
Paul

Re: Bug when building SWIG wrappers on (at least) OSX

[Ben Reser <be...@reser.org>]

On Wed Sep 18 16:00:22 2013, Paul Wagland wrote:
> While trying to install subversion using home-brew on OSX, I came across a few build issues. The patches for these can be seen here: https://github.com/mxcl/homebrew/pull/22668/files#r6449557

This is a known issue with the auth system API.  While that patch fixes 
compilation on OS X it can't be applied because it breaks the Gnome 
keyring support on OSes that support it via the perl bindings.  It also 
completely ignores the same issue that happens with the Ruby and Python 
bindings.

The proper workaround for users that want to build bindings on OS X for 
the time being is to:
make extraclean
./autogen.sh
./configure
make
...

This situation means that OS X users that want to build SWIG based 
bindings need SWIG, libtool, and autoconf (i.e. the same additional 
things you'd need if you built from a checkout rather than the tarball).

The problem here is that our API should not be conditional on 
dependencies at build time and rather should simply have noop 
implementations when the required dependencies were not available at 
runtime.  It's on my radar to fix.