<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 29 Jul 2019 at 14:27, Antony Antony <<a href="mailto:antony@phenome.org">antony@phenome.org</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">On Mon, Jul 29, 2019 at 11:31:30AM -0400, Andrew Cagney wrote:<br>
> On Mon, 29 Jul 2019 at 08:38, Andrew Cagney <<a href="mailto:andrew.cagney@gmail.com" target="_blank">andrew.cagney@gmail.com</a>> wrote:<br>
> <br>
> > These make variables should only be expanded when web pages are enabled?<br>
> > Per the comment:<br>
> ><br>
> > # shortcuts to use when web is enabled, set up to evaluate once as<br>
> > # they can be a little expensive.  These make variable can only be<br>
> > # used inside of WEB_ENABLED blocks.<br>
> ><br>
> <br>
> In fact it was working.  Here's what happens in my tree using the commit<br>
> just before the below:<br>
> <br>
> $ v=$(gmake showversion)<br>
> $ echo $v<br>
> 3.28-506-g423d0e04bd-HEAD<br>
> $ v=$(make showversion LSW_WEBDIR=..)<br>
> + cd .<br>
> + git show --no-patch --format=%H HEAD<br>
> + testing/web/gime-git-description.sh .<br>
> $ echo $v<br>
> 3.28-506-g423d0e04bd-HEAD<br>
> $<br>
> <br>
> so I'm stumped as to why the below behaviour occurred.<br>
<br>
I added LSW_WEBDIR this month, this explains why I started to see change in <br>
behaviour of 'make showversion'. the commit that added set -x is old and I <br>
wondered why didn't I notice set -x before. Thanks for the information.<br>
<br>
'make showversion' output should not change based on other variables are set <br>
or not! It is used by scripts.<br>
<br>
If you still want to see verbose, we a need different way without changing <br>
output of 'make showversion' or other targets used by scripts.<br>
<br></blockquote><div><br></div><div>The output as in stdout and, as illustrated above, can be captured in a shell variable, is unchanged.</div><div>Can you be more specific w.r.t., the problem you're seeing?<br></div><div><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-antony<br>
<br>
> <br>
> Andrew<br>
> <br>
> At one point they were very expensive, unconditionally expanded, and<br>
> > repeatedly making everything slow.  Hence their verbosity - so we can see<br>
> > if the bug has crept back in<br>
> > (from memory, most of the expense was in computing 'dirty' which is why<br>
> > the web page code doesn't do that)<br>
> ><br>
> > Andrew<br>
> ><br>
> > ---------- Forwarded message ---------<br>
> > From: Antony Antony <<a href="mailto:antony@vault.libreswan.fi" target="_blank">antony@vault.libreswan.fi</a>><br>
> > Date: Mon, 29 Jul 2019 at 08:01<br>
> > Subject: [Swan-commit] Changes to ref refs/heads/master<br>
> > To: <<a href="mailto:swan-commit@lists.libreswan.org" target="_blank">swan-commit@lists.libreswan.org</a>><br>
> ><br>
> ><br>
> > New commits:<br>
> > commit 367867e7124949f029c5bf0b3b0f527384770bad<br>
> > Author: Antony Antony <<a href="mailto:antony@phenome.org" target="_blank">antony@phenome.org</a>><br>
> > Date:   Sun Jul 21 11:08:10 2019 +0000<br>
> ><br>
> >     testing: remove set -x from <a href="http://web-targets.mk" rel="noreferrer" target="_blank">web-targets.mk</a><br>
> ><br>
> >     after:<br>
> >     make showversion<br>
> >     3.28-504-g5113b92b34-master<br>
> ><br>
> >     before:<br>
> >     make showversion<br>
> >     + cd .<br>
> >     + git show --no-patch --format=%H HEAD<br>
> >     + testing/web/gime-git-description.sh .<br>
> >     3.28-504-g5113b92b34-dirty-master<br>
> ><br>
> > _______________________________________________<br>
> > Swan-commit mailing list<br>
> > <a href="mailto:Swan-commit@lists.libreswan.org" target="_blank">Swan-commit@lists.libreswan.org</a><br>
> > <a href="https://lists.libreswan.org/mailman/listinfo/swan-commit" rel="noreferrer" target="_blank">https://lists.libreswan.org/mailman/listinfo/swan-commit</a><br>
> ><br>
<br>
> _______________________________________________<br>
> Swan-dev mailing list<br>
> <a href="mailto:Swan-dev@lists.libreswan.org" target="_blank">Swan-dev@lists.libreswan.org</a><br>
> <a href="https://lists.libreswan.org/mailman/listinfo/swan-dev" rel="noreferrer" target="_blank">https://lists.libreswan.org/mailman/listinfo/swan-dev</a><br>
<br>
</blockquote></div></div>