<div dir="auto">(passed presubmit so I'll hopefully get this in on Monday...)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 28, 2022, 16:14 enh <<a href="mailto:enh@google.com">enh@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jul 28, 2022 at 11:55 AM Rob Landley <<a href="mailto:rob@landley.net" target="_blank" rel="noreferrer">rob@landley.net</a>> wrote:<br>
><br>
> On 7/27/22 18:32, enh wrote:<br>
> >     Heh, interesting. This works:<br>
> ><br>
> >     $ ./diff <(echo -e '1\n2') <(echo -e '1\n3') --unchanged-line-format=U%l<br>
> >     --old-line-format=D%l --new-line-format=A%l<br>
> >     U1D2A3<br>
> ><br>
> >     But actually making the "aa" and "bb" files and sticking them at the end does<br>
> >     not? Hmmm... (Gimme a couple hours for dinner, I'll try to fix it this evening.)<br>
> ><br>
> ><br>
> > yeah, no hurry --- i've reverted us back to before the update, so no-one's<br>
> > annoyed that their tests are failing. (we've also added one of the affected<br>
> > tests to toybox's presubmit so we won't be able to submit a killall that broken<br>
> > again, whether or not we don't come up with a toybox test to cover this specific<br>
> > case.)<br>
><br>
> Hopefully fixed now, and I added a skeleton.test for it (which is where I've<br>
> been adding lib/args.c plumbing tests).<br>
<br>
thanks, seems okay locally, so i've kicked off a presubmit run (which<br>
now includes the killall-using test that broke before).<br>
<br>
given that postsubmit takes a day or two to catch up with a culprit<br>
change, i'll hold off submitting until monday [my time] though so that<br>
people whose mondays come first don't have to deal with any fallout<br>
:-)<br>
<br>
> (The problem was --longopt= with nothing after it was advancing to eat the next<br>
> argument instead of saving "" as its argument. The two modifier bits of ; and<br>
> " " means there are four modes of argument parsing, and then each one has a<br>
> -short and --long path, so eight codepaths to test and that's BEFORE you get<br>
> into --longopt abc vs --longopt=abc having this corner case I just fixed...)<br>
><br>
> Sigh, possibly I should make skeleton have an & prefix where the first argument<br>
> has an imaginary dash because that's the OTHER mode of argument parsing that's<br>
> not being tested by skeleton right now. (And I should wire it up to ps so "ps<br>
> ax" and friends get parsed properly. It's there in "tar"...)<br>
><br>
> Rob<br>
</blockquote></div>