Mesa (master): docs/meson: Recommend not using CFLAGS and friends

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 0ff7eed289dcb1be62a7ed6973200a2e8420372c
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0ff7eed289dcb1be62a7ed6973200a2e8420372c

Author: Dylan Baker <dylan at pnwbakers.com>
Date:   Wed Dec 19 13:27:27 2018 -0800

docs/meson: Recommend not using CFLAGS and friends

Because of the many caveats involved, using -Dc_args instead of CFLAGS
is recommended both by meson upstream and by us.

v2: - Fix typo

Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl> (v1)
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 docs/meson.html | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/docs/meson.html b/docs/meson.html
index 21e15ad1c1..63c106147d 100644
--- a/docs/meson.html
+++ b/docs/meson.html
@@ -110,14 +110,14 @@ to invoke non-default targets for ninja to update them:
 <dl>
 <dt><code>Environment Variables</code></dt>
 <dd><p>Meson supports the standard CC and CXX environment variables for
-changing the default compiler, and CFLAGS, CXXFLAGS, and LDFLAGS for setting
-options to the compiler and linker during the initial configuration.
-
-These arguments are consumed and stored by meson when it is initialized. To
-change these flags after the build is initialized (or when doing a first
-initialization), consider using <code>-D${lang}_args</code> and
-<code>-D${lang}_link_args</code> instead. Meson will never change compiler in a
-configured build directory.
+changing the default compiler. Meson does support CFLAGS, CXXFLAGS, etc. But
+their use is discouraged because of the many caveats in using them. Instead it
+is recomended to use <code>-D${lang}_args</code> and
+<code>-D${lang}_link_args</code> instead. Among the benefits of these options
+is that they are guaranteed to persist across rebuilds and reconfigurations.
+
+Meson does not allow changing compiler in a configured builddir, you will need
+to create a new build dir for a different compiler.
 </p>
 
 <pre>