Date: Sat, 25 Jul 1998 00:43:35 +1000 From: Bruce Evans <bde@zeta.org.au> To: phk@critter.freebsd.dk, thepish@FreeBSD.ORG Cc: cvs-committers@FreeBSD.ORG Subject: Re: cvs commit: src/usr.bin/time time.1 time.c Message-ID: <199807241443.AAA05108@godzilla.zeta.org.au>
next in thread | raw e-mail | index | archive | help
>In message <Pine.BSF.3.96.980724231128.2327E-100000@dana.clari.net.au>, Peter H
>awkins writes:
>>> Modified files:
>>> usr.bin/time time.1 time.c
>>
>>There is no situation where this patch is required and it's quite a departure
>>from the norms elsewhere. The consensus on the list seems to have been that
>>this shouldn't be committed.
>
>Quite the contrary: The weird exploitations of weird shell syntax
>needed to do what should really be very simple, clearly indicates
>the need to make it easier to do these things.
I didn't notice any weird shell syntax, except for non-real shells of
course.
>Furthermore, the energies wasted in this discussion should be redirected
>to more productive activities, like reviewing and handling PRs.
OK, the patch was not suitable for committing verbatim, due to spelling
errors ("premissions"), style bugs (redeclaration of opt* ...) and
inconsistency of the new args with truss(1) and gnu time(1).
[truss]
usage: truss [-S] [-o file] -p pid
truss [-S] [-o file] command [args]
^^^^^^^^^
[gnu time]
Usage: time [-apvV] [-f format] [-o file] [--append] [--verbose]
^^^^^^^^^
[--portability] [--format=format] [--output=file] [--version]
[--help] command [arg...]
Bruce
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199807241443.AAA05108>