<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 24 Apr 2021 at 10:19, D. Hugh Redelmeier <<a href="mailto:hugh@mimosa.com">hugh@mimosa.com</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">| From: Andrew Cagney <<a href="mailto:andrew.cagney@gmail.com" target="_blank">andrew.cagney@gmail.com</a>><br>
<br>
| Around the 4.5 release I'd like to put forward making  a few pervasive<br>
| changes.<br>
<br>
I don't know all the logisticsl<br>
<br>
| When I say around I mean that period around the release during which the<br>
| mainline is pretty much frozen.<br>
<br>
Makes sense.<br>
<br>
| before:<br>
| - s/TRUE/true/ s/FALSE/false/<br>
|   If this is done just before the release patches to 4.5 won't have to deal<br>
| with it.<br>
<br>
Yes!<br>
<br>
If a tool will handle it, be a little more careful:<br>
<br>
        s/\<TRUE\>/true/g<br>
        s/\<FALSE\>/false/g<br>
<br>
GNU sed seems to handle this:<br>
<br>
        $ echo 'x x xx' | sed -e 's/\<x\>/y/g'<br>
        y y xx<br>
<br></blockquote><div>Yes, it isn't trivial.  For instance, TRUE and FALSE embedded in strings and comments probably shouldn't be touched.</div><div>The hack I tried anchored things a little by using ' = TRUE;' for instance.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
| after:<br>
| - add ikev1 to all the ikev1 test names; and ikev2 to all the ikev2<br>
| test names<br>
<br>
(Broken Record) "ikev1_" is too long.  We should systematically use<br>
"v1_" or something else concise.  For test names, "1_" should be OK.<br>
<br>
When can we ditch ikev1 code and make this moot?  I thought we'd long<br>
passed our original target.<br></blockquote><div><br></div><div>I consider the test names to be public facing.   Hence, I don't think using ambiguous abbreviations is useful.</div><div>Besides being able to glob *ikev1* is going to be far more robust than globbing *1_*.</div><div><br></div><div>Oh, my money is on ikev3 arriving before we eliminate ikev1 :-)</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">
_______________________________________________<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>
</blockquote></div></div>