bug-idutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [bug-idutils] idutils-4.5 released [stable]


From: Jim Meyering
Subject: Re: [bug-idutils] idutils-4.5 released [stable]
Date: Fri, 18 Jun 2010 12:02:33 +0200

Claudio Fontana wrote:
> 4.5 seems fine here (I built 4.5 on a SunOS 5.10 Sparc box), and will
> test and use on other machines as well in the coming days.
>
> I get an error at make check time (on the SunOS Sparc), which follows.
>
...
> Making check in man
> make  check-local
> PATH=../src:$PATH; export PATH;         \
> t=ls-files.$$;                                          \
> (cd . && ls -1 *.x) | sed 's/\.x$//' | LC_ALL=C sort > $t;\
> echo mkid.1 lid.1 fid.1 fnid.1 xtokid.1 eid.1 aid.1 gid.1 defid.1 |
> fmt -w1 | sed 's/\.1$//' | LC_ALL=C sort    \
>   | diff -u - $t || { rm $t; exit 1; };                         \
> rm $t
> fmt:  bad width: 0

Hi Claudio,

Thanks for the report.
I didn't notice it when I tested on Solaris 10
because I used the "fmt" from coreutils.
When I use the solaris version I see the same problem:

    $ echo a b c|/bin/fmt -w1
    fmt:  bad width: 0
    [Exit 1]
    $ echo a b c|/bin/fmt -w 1
    a
    b
    c

Inserting a space fixes it:

>From 0bcb904f52dcdcfaed70f7b131481a68353e36b3 Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Fri, 18 Jun 2010 11:59:54 +0200
Subject: [PATCH] tests: avoid "make check" failure with Solaris 10's "fmt"

* man/Makefile.am (check-x-vs-1): Use "fmt -w 1", not "fmt -w1",
since the latter fails on Solaris 10 with "fmt: bad width: 0".
Reported by Claudio Fontana.
---
 man/Makefile.am |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/man/Makefile.am b/man/Makefile.am
index 2d108ec..14dca9d 100644
--- a/man/Makefile.am
+++ b/man/Makefile.am
@@ -48,7 +48,7 @@ check-x-vs-1:
        PATH=../src$(PATH_SEPARATOR)$$PATH; export PATH;                \
        t=ls-files.$$$$;                                                \
        (cd $(srcdir) && ls -1 *.x) | sed 's/\.x$$//' | $(ASSORT) > $$t;\
-       echo $(dist_man1_MANS) | fmt -w1 | sed 's/\.1$$//' | $(ASSORT)  \
+       echo $(dist_man1_MANS) | fmt -w 1 | sed 's/\.1$$//' | $(ASSORT) \
          | diff -u - $$t || { rm $$t; exit 1; };                               
\
        rm $$t

--
1.7.1.618.g863be



reply via email to

[Prev in Thread] Current Thread [Next in Thread]