Add /usr/bin/ts, a command that timestamps each line of its stdin
before it is being printed to stdout.
A typical use case is to profile shell scripts.
Obtained from: OpenBSD
Differential D35694
Add ts(1) command otis on Jul 2 2022, 3:34 PM. Authored by Tags None Referenced Files
Details
Add /usr/bin/ts, a command that timestamps each line of its stdin A typical use case is to profile shell scripts. Obtained from: OpenBSD
Diff Detail
Event TimelineComment Actions I cant immediately spot any other nits with the mdoc, but perhaps you should add a note about when ts(1) was added to FreeBSD in the history section header? Also, I can't wait for this one to land, it's super handy and I can't believe I didn't know about it until now. :)
Comment Actions Other than the question, I'm still happy with the mdoc, so will accept once that's been answered.
Comment Actions IMHO, that's a neat tool that perfectly fits original UNIX concepts. I'd vote for adding it. @otis , you got commit bit, right? If you are concerned about possible objections, and want attract more attention to this review, maybe send a note to current@? Comment Actions This seems to me too small to deserve its own vendor branch. Moreover, there are some local changes (subtle ones) to manual page. Comment Actions Vendor branches are how we manage the changes to upstreams so they don't get lost. But at 180 lines, 3 of which are added, it's hard to argue for using it. Comment Actions Importing this via a vendor branch would indeed be the "correct" way to do it, but it doesn't much matter as the upstream is unlikely to see significant change from this point. I imagine this program is essentially complete. This (not using a vendor branch) is the way we've handled makefs(8) -- OpenBSD, NetBSD and FreeBSD all have copies with independent development and at this point OpenBSD and NetBSD are not really upstreams that we could use for contrib/. Do note in the commit message the OpenBSD version that it came from, though.
|