Commited change from int to bool type and also would like to clarify regarding why static bool construction is used: the reason is that __builtin_available() can not be used in combination with other checks inside if.

can't use traditional weak-linking here instead of __builtin_available()?

If you mean by weak-linking the dlsym operation then it is undefined whether it will load clock_gettime symbol or its stub with internal assertion, to my view. It depends on Apple's implementation which they can change anytime at their sole discretion. Apple SDK is relying on LLVM/Clang and its __builtin_available functionality thus for a robust, always working binaries it makes sense to use the same functionality.

we can't just tell them off, especially when the rest of the library builds just fine with GCC.

Proposed changes do not affect GCC users in any sense, so there is no limitation, code compiles and works successfully. If GCC user defines HAVE_CLOCK_GETTIME_MONOTONIC then clock_gettime will be used without any problem.

But the main problem with GCC on Apple platform - it is depreciated by Apple and thus it does not make sense to try to use GCC + recent Apple SDK based on LLVM/Clang to compile binaries for production. I do not think anybody will do that. And older Apple SDK, which is using/compatible with GCC. does not have clock_gettime defined anyway (HAVE_CLOCK_GETTIME_MONOTONIC will be undefined). So, all is ok with GCC users to my view either.

To avoid confusion with public defines of HAVE_XXX style I propose to rename HAVE_CLOCK_GETTIME_CHECK to _USE_CLOCK_GETTIME_CHECK.

But since the variable is always updated by the if(), why does it need to be static? Surely removing static from it will keep it working just as good and just remove potential problems?

@bagder, agree that variable may be local. Also replaced check for __builtin_available with existing HAVE_BUILTIN_AVAILABLE define (did not notice that it is used by curl already, not defined in CMake configs?) as advised by @nickzman. Applied necessary changes.

I used HAVE_BUILTIN_AVAILABLE in the way darwinssl.c uses it, but connect.c uses HAVE_BUILTIN_AVAILABLE as #if defined(HAVE_BUILTIN_AVAILABLE) both checks will work unless HAVE_BUILTIN_AVAILABLE is defined as

#define HAVE_BUILTIN_AVAILABLE 0

It appears that __builtin_available is also available on other platforms in LLVM/Clang, for example Android (NDK r17b), thus limited check to only Apple OS.

I suppose this is fine then. By "traditional" weak linking, I meant the way it was done prior to __builtin_available(), which was to check at run-time if the symbol was defined or not. The darwinssl.c code still does this for backward compatibility with older compilers, and it hasn't caused any problems. But using __builtin_available() will solve this for the vast majority of users.

Thanks!