[DTrace-devel] [PATCH v2 18/20] test, usdt, pid: remove arg count check

Nick Alcock nick.alcock at oracle.com
Thu Oct 6 13:39:27 UTC 2022


On 22 Sep 2022, Kris Van Hees said:

> On Wed, Sep 07, 2022 at 02:00:05PM +0100, Nick Alcock via DTrace-devel wrote:
>> This check is obsolescent and these days is only testing the *test
>> runner*.  Tests don't need to test runtest.sh: the chain of blame is
>> meant to run the other way :)
>
> That check does not exist to test runtest.sh but rather is a user friendly
> check for poeple who might execute the test script without runtest.sh.  I don't
> think we really need to remove them from all the scripts - it can be convenient
> (and has reminded me a couple of times in the past that I needed to supply that
> argument).

OK, if it's actually useful... I don't plan to add more of these checks,
though. Dropped.



More information about the DTrace-devel mailing list