Message ID | 20210819183652.7750-1-carenas@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | fixup! fsmonitor-fs-listen-macos: implement FSEvent listener on MacOS | expand |
On 8/19/21 2:36 PM, Carlo Marcelo Arenas Belón wrote: > clang defines __GNUC__ for compatibility with gcc (indeed it misrepresents > itself as gcc 4.2.1), so instead use the __clang__ specific macro to > distinguish between them. > > tested with Apple clang 12.0.5, clang 14 and gcc 11.2.0 > > Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> > --- > compat/fsmonitor/fsmonitor-fs-listen-macos.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/compat/fsmonitor/fsmonitor-fs-listen-macos.c b/compat/fsmonitor/fsmonitor-fs-listen-macos.c > index 02f89de216..c55ec35b5d 100644 > --- a/compat/fsmonitor/fsmonitor-fs-listen-macos.c > +++ b/compat/fsmonitor/fsmonitor-fs-listen-macos.c > @@ -1,4 +1,4 @@ > -#if defined(__GNUC__) > +#ifndef __clang__ > /* > * It is possible to #include CoreFoundation/CoreFoundation.h when compiling > * with clang, but not with GCC as of time of writing. > thanks! Jeff
diff --git a/compat/fsmonitor/fsmonitor-fs-listen-macos.c b/compat/fsmonitor/fsmonitor-fs-listen-macos.c index 02f89de216..c55ec35b5d 100644 --- a/compat/fsmonitor/fsmonitor-fs-listen-macos.c +++ b/compat/fsmonitor/fsmonitor-fs-listen-macos.c @@ -1,4 +1,4 @@ -#if defined(__GNUC__) +#ifndef __clang__ /* * It is possible to #include CoreFoundation/CoreFoundation.h when compiling * with clang, but not with GCC as of time of writing.
clang defines __GNUC__ for compatibility with gcc (indeed it misrepresents itself as gcc 4.2.1), so instead use the __clang__ specific macro to distinguish between them. tested with Apple clang 12.0.5, clang 14 and gcc 11.2.0 Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> --- compat/fsmonitor/fsmonitor-fs-listen-macos.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)