[Github-comments] [geany/geany] configure.ac:expand comment for AC_USE_SYSTEM_EXTENSIONS (#2057)

[Andy Alt]

andy5995 commented on this pull request.



> @@ -20,7 +20,11 @@ if test -n "${build}" -a -n "${target}"; then
 	GEANY_STATUS_ADD([Building Geany for], [${target}])
 fi
 
-# why do we use this?
+# If possible, enable extensions to C or Posix on hosts that normally
+# disable the extensions, typically due to standards-conformance
+# namespace issues. This should be called before any macros that run the
+# C compiler.
+# <https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Posix-Variants.html>

Oh, I misunderstood the intent of the commented question.

I couldn't say for sure.. I did a little checking using `git grep` and the only preprocessor macro (from the list shown in that autoconf doc) being used in the Geany repo is '_GNU_SOURCE`.

```
ctags/main/lxcmd.c:46:#ifndef _GNU_SOURCE
ctags/main/lxcmd.c:47:# define _GNU_SOURCE   /* for WIFEXITED and WEXITSTATUS */
ctags/main/options.c:15:#ifndef _GNU_SOURCE
ctags/main/options.c:16:# define _GNU_SOURCE   /* for asprintf */
plugins/export.c:294:#ifdef _GNU_SOURCE
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2057#discussion_r249083136
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190118/a93979fc/attachment.html>