From e4b2242f3a29832e1df389ceea6ec4ca41c06306 Mon Sep 17 00:00:00 2001 From: Ben Kaduk Date: Tue, 26 Mar 2013 17:42:38 -0400 Subject: [PATCH] Fix DARWIN build with clang In 1d8937b86050 we added a function call to kauth_cred_unref in the DARWIN100 case (replacing a macro), but added the inclusion of sys/kauth.h only when using versions older than DARWIN80. On DARWIN100 and above, clang detects that the now-implicit function declaration is in conflict with the actual prototype, which is included later through afs/sysincludes.h when compiling the kernel rx code. Since including sys/kauth.h seems to have been harmless for old versions, just include it always. Reviewed-on: http://gerrit.openafs.org/9676 Tested-by: BuildBot Reviewed-by: Derrick Brashear (cherry picked from commit 6ad2cc0275cd1074316d6bc70cd8758d25f94467) Change-Id: I7e3bfdee65791828fe6ee68e3dc125be2311cd86 Reviewed-on: http://gerrit.openafs.org/10549 Tested-by: BuildBot Reviewed-by: Andrew Deason Reviewed-by: Derrick Brashear Reviewed-by: Stephan Wiesand --- src/afs/DARWIN/osi_machdep.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/afs/DARWIN/osi_machdep.h b/src/afs/DARWIN/osi_machdep.h index ae6e02eb9b..b0edbf9c2e 100644 --- a/src/afs/DARWIN/osi_machdep.h +++ b/src/afs/DARWIN/osi_machdep.h @@ -33,8 +33,8 @@ typedef unsigned short etap_event_t; #include #include #include -#include #endif +#include #include #ifdef AFS_DARWIN80_ENV