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
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
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 ''
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
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
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