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

[Colomban Wendling]

b4n 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>

Did you check we actually need it?  If so, do you know for what we do need it?

-- 
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#pullrequestreview-194141556
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190118/e48cbcb7/attachment.html>