Re: PATCH: add dejagnu(1) multi-launcher

Andreas Schwab wrote:
> On Dez 18 2018, Jacob Bachmeyer <[email protected]> wrote:
>   
>> Andreas Schwab wrote:
>>     
>>> On Dez 18 2018, Jacob Bachmeyer <[email protected]> wrote:
>>>   
>>>       
>>>> +if expr "$verbose" \> 0 > /dev/null ; then
>>>>     
>>>>         
>>> Why not test $verbose -gt 0?
>>>   
>>>       
>> The existing "runtest" script uses expr for those tests.  I decided to
>> follow suit on the assumption that that choice was made for a reason,
>> presumably portability, although the Autoconf portable shell guidelines do
>> not mention that concern with "test".
>>     
>
> Your script is already using $( ) and $(( )) which isn't supported by
> some pre-POSIX shells.
>   

As does the "runtest" script.  The "dejagnu" script is simply following 
suit.


-- Jacob