Re: [PATCH] dd: add a flag to discard cached data

2011-02-24 Thread Pádraig Brady
On 24/02/11 07:52, Jim Meyering wrote: One quick question: does the test need something to make it skip (not fail) on systems that lack kernel support for the feature? Oops right. Attached is latest version. thanks for the review, Pádraig. From fe067f8f52defc54636ae862df03a2115ac6266f Mon Sep

Re: [PATCH] dd: add a flag to discard cached data

2011-02-24 Thread Jim Meyering
Pádraig Brady wrote: On 24/02/11 07:52, Jim Meyering wrote: One quick question: does the test need something to make it skip (not fail) on systems that lack kernel support for the feature? Oops right. Attached is latest version. ... +# Advise to drop cache for whole file +if ! dd

bug#8103: NUL terminated lines

2011-02-24 Thread Jim Meyering
Bjartur Thorlacius wrote: Maybe we should modify tac to add the -z option. Would you care to write a patch? It would be redundant, as tac -s $'\0' is equivalent. Are you using a non-GNU version of tac? If so, please tell us which one -- that may influence the decision of whether to make -s ''

bug#8103: NUL terminated lines

2011-02-24 Thread Bjartur Thorlacius
On 2/24/11, Jim Meyering j...@meyering.net wrote: Bjartur Thorlacius wrote: Maybe we should modify tac to add the -z option. Would you care to write a patch? It would be redundant, as tac -s $'\0' is equivalent. Are you using a non-GNU version of tac? I don't remember whether I was using

bug#8103: NUL terminated lines

2011-02-24 Thread Jim Meyering
Bjartur Thorlacius wrote: On 2/24/11, Jim Meyering j...@meyering.net wrote: Bjartur Thorlacius wrote: Maybe we should modify tac to add the -z option. Would you care to write a patch? It would be redundant, as tac -s $'\0' is equivalent. Are you using a non-GNU version of tac? I don't

bug#8103: NUL terminated lines

2011-02-24 Thread Alan Curry
Bjartur Thorlacius writes: On 2/24/11, Jim Meyering j...@meyering.net wrote: Bjartur Thorlacius wrote: Maybe we should modify tac to add the -z option. Would you care to write a patch? It would be redundant, as tac -s $'\0' is equivalent. Note that a $'\0' argument in a shell command