Pádraig Brady <[email protected]> writes: > On 19/05/2026 03:43, Collin Funk wrote: >> Pádraig Brady <[email protected]> writes: >> >>> This looks good at a quick glance. >>> Do we need the uses_strace_ thing in the test? >> I used 'require_strace_ write' since I wanted to skip the test when >> strace is not available. >> When I added uses_strace_, I added a call to it in require_strace_ >> so >> that the strace call in the checks will not wait for input when invoking >> AIX's unrelated program which would hang the tests. >> I don't think it uses the correct number of arguments to do that at >> the >> moment, but I figured it may change in the future. I don't expect anyone >> to remember AIX specific programs, so might as well avoid running into >> that problem. > > We need to have the more involved check, > as strace 5.10 on Debian 11 at least fails due to strace > not supporting all the options. > > I've pushed the attached which calls uses_strace_ > and also skips if all the strace options are not supported.
Oh, I didn't realize the options were relatively new. I guess I should have checked. Thanks! Collin
