<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Dec 17, 2018 at 1:18 PM Eric Anholt <<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Eero Tamminen <<a href="mailto:eero.t.tamminen@intel.com" target="_blank">eero.t.tamminen@intel.com</a>> writes:<br>
<br>
> Hi,<br>
><br>
> On 17.12.2018 8.08, Marek Olšák wrote:<br>
> [...]<br>
>> I think one of the serious usability issues is that environment <br>
>> variables such as CFLAGS, CXXFLAGS, LDFLAGS, and PKG_CONFIG_PATH are not <br>
>> saved by meson for future reconfigures.<br>
><br>
> I don't know what Meson is supposed to do, but to me that would be<br>
> a bug in a build tool.<br>
><br>
> Re-configure is supposed to adapt SW to the changes in the build<br>
> environment, and environment variables are part of that (along with<br>
> command line options and SW installed to to the system).  Build<br>
> configure tool deciding to "remember" some of those things instead<br>
> of checking the new situation, seems like a great opportunity for<br>
> confusion.<br>
<br>
A user-triggered reconfigure, sure.  Recapture env vars then.  But "git<br>
pull; ninja -C build" losing track of the configuration state is broken.<br>
We don't have to specify all of your meson -Doption=state configuration<br>
on every build, why should you need to specify your PKG_CONFIG_PATH<br>
configure options on every build?<br></blockquote><div><br></div>Thanks, Eric.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Yes, meson behaves such that users have to set all environment variables for every "ninja" command that might reconfigure.</div><div class="gmail_quote"><br></div><div class="gmail_quote">I see 2 solutions:</div><div class="gmail_quote">1) meson needs to remember the relevant env vars</div><div class="gmail_quote">2) meson should FAIL to configure if any of the env vars are set (if it wants to ignore them)<br></div><div class="gmail_quote"><br></div><div class="gmail_quote">Marek<br></div></div>