Re: [Toybox] [PATCH] Enable ASan for github CI.

2023-07-11 Thread Rob Landley
On 7/11/23 17:15, enh wrote: > especially since we suspiciously haven't had any CI run at all since that > change > went in: https://github.com/landley/toybox/commits/master > (i'd expected as least an error, but not even a "ran and failed"!) I hit that problem back when we added the mac tests

Re: [Toybox] [PATCH] Enable ASan for github CI.

2023-07-11 Thread enh via Toybox
On Mon, Jul 10, 2023 at 11:20 PM Rob Landley wrote: > On 7/10/23 18:58, enh via Toybox wrote: > > Untested (because I don't know how to test such changes without merging > > them), but seems plausible based on the documentation at > > >

Re: [Toybox] [PATCH] sh: pass "\" to the later app

2023-07-11 Thread Chet Ramey
On 7/10/23 5:57 PM, enh wrote: > iirc we even _tried_ -1/ENOSYS to test the predictions that (a) most c > programmers don't check for failures (b) even those that do don't log > enough to be able to debug these problems It penalizes those who do. yeah, but we optimize

Re: [Toybox] [PATCH] sh: pass "\" to the later app

2023-07-11 Thread Rob Landley
On 7/10/23 16:57, enh wrote: > and i think that's the cross-purpose we're talking at here ... the 99% case is > app code... > the kind of collaborative command-line debugging over a mailing list that > you're > familiar with doesn't exist for the TikToks of this world. There's no amount of

Re: [Toybox] [PATCH] Enable ASan for github CI.

2023-07-11 Thread Rob Landley
On 7/11/23 01:21, Rob Landley wrote: > On 7/10/23 18:58, enh via Toybox wrote: >> Untested (because I don't know how to test such changes without merging >> them), but seems plausible based on the documentation at >>