Re: Auth providers and problems with our packaging

2013-11-20 Thread Ben Reser
On 6/14/13 5:20 AM, Ben Reser wrote: > Actually now that I think about it a bit more I'm not sure that helps. > It'll probably just shift it to a link time error because the defines > will be there so it'll build, SWIG will generate wrappers for them > that depend on the functions in our libraries

Re: Auth providers and problems with our packaging

2013-06-19 Thread Peter Samuelson
[Ben Reser] > Actually now that I think about it a bit more I'm not sure that > helps. It'll probably just shift it to a link time error because the > defines will be there so it'll build, SWIG will generate wrappers for > them that depend on the functions in our libraries, which won't be > avail

Re: Auth providers and problems with our packaging

2013-06-14 Thread Ben Reser
On Fri, Jun 14, 2013 at 2:09 PM, Ben Reser wrote: > Yes and I guess that's probably the easiest solution for this older > APIs. In 1.6. we introduced the platform specific APIs that make it > easier to deal with runtime detection of platform specific auth > runtimes. Given that I guess going bac

Re: Auth providers and problems with our packaging

2013-06-14 Thread Daniel Shahaf
Ben Reser wrote on Fri, Jun 14, 2013 at 14:09:29 +0200: > On Fri, Jun 14, 2013 at 1:30 PM, Daniel Shahaf wrote: > > Would it suffice to change the header file conditions from > > > > #if defined(DARWIN) || defined(DOXYGEN) > > > > to > > > > #if defined(DARWIN) || defined(DOXYGEN) || defined(SWIG)

Re: Auth providers and problems with our packaging

2013-06-14 Thread Ben Reser
On Fri, Jun 14, 2013 at 1:30 PM, Daniel Shahaf wrote: > Would it suffice to change the header file conditions from > > #if defined(DARWIN) || defined(DOXYGEN) > > to > > #if defined(DARWIN) || defined(DOXYGEN) || defined(SWIG) Yes and I guess that's probably the easiest solution for this older AP

Re: Auth providers and problems with our packaging

2013-06-14 Thread Daniel Shahaf
Ben Reser wrote on Fri, Jun 14, 2013 at 13:21:13 +0200: > I propose we stop hiding APIs based on platform. APIs that return an > svn_error_t should return a new NOT_IMPLEMENTED error code. APIs that > do not (as is the case with the auth APIs in this case) should provide > some appropriate noop b