[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug-diffutils] RE: DiffUtility: Test FAIL: function-line-vs-leading-spa
From: |
Peram, Sudhakara |
Subject: |
[bug-diffutils] RE: DiffUtility: Test FAIL: function-line-vs-leading-space |
Date: |
Mon, 24 Jan 2011 10:39:18 -0500 |
Thank you Jim for your information, where can I find the below patch so
that I will install on my machine.
Thank you once again
--Sudhakar
-----Original Message-----
From: Jim Meyering [mailto:address@hidden
Sent: Monday, January 24, 2011 10:36 AM
To: Peram, Sudhakara
Cc: address@hidden
Subject: Re: DiffUtility: Test FAIL: function-line-vs-leading-space
Peram, Sudhakara wrote:
> Below is the test failed on solaris when I was building 64 bit diff
utility to
> support large files.
>
> SunOS hostname 5.9 Generic_122300-47 sun4u sparc SUNW,Sun-Fire-V490
...
> FAIL: function-line-vs-leading-space (exit: 1)
...
> + tail -n+3 out
>
> usage: tail [+/-[n][lbc][f]] [file]
>
> tail [+/-[n][l][r|f]] [file]
>
> + fail=1
Thanks for the report.
I've pushed this fix:
>From 4e0d3664c7a1a7515309b140dbf8ff27538423cf Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Mon, 24 Jan 2011 16:32:00 +0100
Subject: [PATCH] tests: avoid newer but less-portable tail option syntax
* tests/function-line-vs-leading-space: Use sed -n '3,$p' rather than
tail -n+3. Older versions of tail do not accept that newer syntax.
Reported by Sudhakara Peram.
---
tests/function-line-vs-leading-space | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tests/function-line-vs-leading-space
b/tests/function-line-vs-leading-space
index 9143e02..1541365 100755
--- a/tests/function-line-vs-leading-space
+++ b/tests/function-line-vs-leading-space
@@ -55,7 +55,7 @@ fail=0
diff -u -F '^[[:space:]]*\(function\|procedure\)' in in2 > out 2> err
test $? = 1 || fail=1
-tail -n+3 out > k && mv k out || fail=1
+sed -n '3,$p' out > k && mv k out || fail=1
compare out exp || fail=1
--
1.7.3.5.38.gb312b