Re: [PATCH] tests: Introduce test_seq
- From
Jeff King <peff@peff.net>
- Date
- Aug 3, 2012, 20:07 UTC
- Message-ID
- <20120803200718.GA10648@sigill.intra.peff.net>
- In-Reply-To
- <1344024290-9197-1-git-send-email-michal.kiedrowicz@gmail.com>
On Fri, Aug 03, 2012 at 10:04:50PM +0200, Michał Kiedrowicz wrote:
> Previous patch didn't support `test_seq 1 50` (I removed it accidentally).
Our emails just crossed paths. :)
Show 6 quoted lines
> +# test_seq is a portable replacement for seq(1). > +# It may be used like: > +# > +# for i in `test_seq 100`; do > +# echo $i > +# done
This should probably note that it is a subset of seq's behavior. You talked about it in the commit message, but the in-code comment is a much more likely thing for a potential user to read.
-Peff