<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 10, 2023 at 11:20 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 7/10/23 18:58, enh via Toybox wrote:<br>
> Untested (because I don't know how to test such changes without merging<br>
> them), but seems plausible based on the documentation at<br>
> <a href="https://docs.github.com/en/actions/learn-github-actions/variables#about-environment-variables" rel="noreferrer" target="_blank">https://docs.github.com/en/actions/learn-github-actions/variables#about-environment-variables</a> <br>
> ---<br>
>  .github/workflows/toybox.yml | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
<br>
Applied, but who is this for? I'm not going to notice: I push from the command<br>
line after testing locally. I check the github web UI less often than I still<br>
check the busybox mailing list.<br></blockquote><div><br></div><div>yeah, probably mostly just me. (though obviously others can nag you :-) there's some magic you can add to the README.md that shows a badge that says whether or not the tests are passing at ToT that's pretty cool and surfaces this nicely, but i've no idea how that works.)</div><div><br></div><div>but, yes, i actually do usually look at the commits before i kick off a sync, if only to see "what's changed since last?" and "what commands do i need to be paying attention to over the next week?". so even if this _only_ helps me, it will help me :-)</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">
Between your first and second emails I was working to add ASAN=1 to the "make<br>
tests" build target, starting by cleaning up my dirty scripts/test.sh file so I<br>
could check it in (mostly comments, but I've been digging to try to get it<br>
running under mkroot). But adding ": ${ASAN=1}" to the start of that file so I<br>
could still test without it ala "ASAN= make tests" doesn't scripts/test.sh tests<br>
the existing toybox binary, and for "make tests" toybox is built by a make<br>
dependency which apparently CAN'T BE TOLD which target we actually called on the<br>
command line, because gnu make is horrific. By the time scripts/test.sh gets run<br>
the value of ASAN doesn't matter, and scripts/make.sh would have to do something<br>
disgusting like ps $PPID to try to figure out that it got called from "make<br>
tests". Hmmm...<br></blockquote><div><br></div><div>ah, that's an even better idea. in that case, maybe revert my change?</div><div><br></div><div>especially since we suspiciously haven't had any CI run at all since that change went in: <a href="https://github.com/landley/toybox/commits/master">https://github.com/landley/toybox/commits/master</a> (i'd expected as least an error, but not even a "ran and failed"!)</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>
_______________________________________________<br>
Toybox mailing list<br>
<a href="mailto:Toybox@lists.landley.net" target="_blank">Toybox@lists.landley.net</a><br>
<a href="http://lists.landley.net/listinfo.cgi/toybox-landley.net" rel="noreferrer" target="_blank">http://lists.landley.net/listinfo.cgi/toybox-landley.net</a><br>
</blockquote></div></div>