<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 30, 2021 at 12:39 PM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.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">On 8/30/21 10:56 AM, enh via Toybox wrote:<br>
> Found in github CI by the existing patch tests:<br>
> ```<br>
> patch: xsignal 0: Invalid argument<br>
> ```<br>
<br>
Yeah, I bumped into that right after I pushed your commit (it broke the mkroot<br>
linux kernel build) but got distracted trying to integrate<br>
scripts/record-commands into mkroot.sh instead of immediately fixing it.<br>
<br>
> When I grepped for places that would be affected by my `kill -0` change,<br>
> I grepped toys/, not lib/. (And I only ran the tests for the toys I<br>
> touched, not all the tests, and CI was already broken so we didn't<br>
> notice the new breakage until the old breakage was fixed :-( )<br>
<br>
Your commit changed list_signals() to start from 1 but didn't do the same for<br>
xsignal_all_killers(). One character fix. </blockquote><div><br></div><div>yeah, i sent you a more complicated fix in case someone was tempted to reorder the array (note that in list_signals() the loop variable is the signal number, but in xsignal_all_killers() the loop variable is the array index), but (a) your smaller fix works and (b) we know we have a test that will catch this if someone does break it, so lgtm :-)</div><div><br></div><div>thanks! (and sorry for not running all the tests! it's _just_ slow enough on my laptop to run them all that i tend to only run the specific toy's tests.)</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">Pushed now, sorry for the delay. (I've<br>
been mostly AFK this week.)<br></blockquote><div><br></div><div>no worries --- me too, and likely to be worse for the next few weeks thanks to perf season starting! i'll get AOSP back in sync with you as soon as i can so we don't drift any further behind...</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">
Rob<br>
</blockquote></div></div>