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

Re: Auth providers and problems with our packaging

From: Ben Reser <ben_at_reser.org>
Date: Fri, 14 Jun 2013 14:09:29 +0200

On Fri, Jun 14, 2013 at 1:30 PM, Daniel Shahaf <danielsh_at_elego.de> 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
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 back and mucking with the older
APIs doesn't seem necessary to deal with the runtime issues.

But in general outside of the existing auth cases we should avoid ever
doing this again. We should simply design our APIs so that any
platform specific ones have some sort of not implemented return for
platforms where they are missing.
Received on 2013-06-14 14:10:06 CEST

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