On 17/04/16 08:45, Assaf Gordon wrote:
Hello Bernhard, Pádraig,

Attached an updated patch based on your comments and suggestions.
One difference from previous patch:
'seq' will not silently ignore EPIPE (if SIGPIPE is ignored by default), and 
will exit with 'broken pipe' message.
I split the tests into two: one with /dev/full (skipped unless /dev/full 
exists),
the other forcing a broken pipe (which can be tested anywhere).

Yes it's better not to change the EPIPE handling
so we're consistent with other utils.
That might be a change we consider separately.

Your patch looks good.

please push.

thanks!
Pádraig


Reply via email to