2011-01-24 22:19:24

by Haynes, Tom

[permalink] [raw]
Subject: Patched up cthon source?

I'm being evil here on purpose. :->

I'm trying to run cthon04 as downloaded from the connectathon.org repository
and I'm getting this:


Nroff
./stat: no data in nroff.time
general tests failed
Tests failed, leaving /cthon04/loghyr mounted


As I kinda recall several people fixing this already, where would I go to get patches?


2011-01-25 14:56:58

by Jeff Layton

[permalink] [raw]
Subject: Re: Patched up cthon source?

On Tue, 25 Jan 2011 08:43:25 -0500
Bryan Schumaker <[email protected]> wrote:

> On 01/24/2011 05:33 PM, Benny Halevy wrote:
> > this is a problem with the version of nroff on newer distributions.
>
> My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.
>
> Bryan
>

Yep, this workaround stopped working around for me on my rawhide box
broke yesterday for the same reason.

I think this patch may be a better approach than trying to grep out
random stuff, but I don't know whether non-GNU tbl programs will barf
on it. If anyone has a solaris box handy, could you let me know if
this breaks on it?

----------------------[snip]------------------
[PATCH] cthon04: add "nowarn" option to nroff.in tables

This prevents the tbl preprocessor from stuffing warnings into the
resulting file, which subsequently can make nroff spew warnings like
this on stderr:

warning: file `nroff.in', around line 47:
table wider than line width

Unfortunately, I'm not sure how non-gnu tbl programs will react to
this option, so YMMV.

Signed-off-by: Jeff Layton <[email protected]>
---
general/nroff.in | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/general/nroff.in b/general/nroff.in
index 522ac80..a8acb3a 100644
--- a/general/nroff.in
+++ b/general/nroff.in
@@ -18,7 +18,7 @@ benchmarks.
.LP
I ran the benchmarks on the following configurations:
.TS
-center;
+center nowarn;
l l l l.
CPU Unix Fortran Memory
.sp 4p
@@ -38,7 +38,7 @@ A bubble sort of integers.
A prime number generator.
.LP
.TS
-center;
+center nowarn;
c|c|c s|c s|
c|c|c s|c s|
c|c|c s|c s|
--
1.7.3.4


2011-01-25 15:53:10

by Jeff Layton

[permalink] [raw]
Subject: Re: Patched up cthon source?

On Tue, 25 Jan 2011 10:38:32 -0500
Trond Myklebust <[email protected]> wrote:

> On Tue, 2011-01-25 at 09:56 -0500, Jeff Layton wrote:
> > On Tue, 25 Jan 2011 08:43:25 -0500
> > Bryan Schumaker <[email protected]> wrote:
> >
> > > On 01/24/2011 05:33 PM, Benny Halevy wrote:
> > > > this is a problem with the version of nroff on newer distributions.
> > >
> > > My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.
> > >
> > > Bryan
> > >
> >
> > Yep, this workaround stopped working around for me on my rawhide box
> > broke yesterday for the same reason.
> >
> > I think this patch may be a better approach than trying to grep out
> > random stuff, but I don't know whether non-GNU tbl programs will barf
> > on it. If anyone has a solaris box handy, could you let me know if
> > this breaks on it?
> >
> > ----------------------[snip]------------------
> > [PATCH] cthon04: add "nowarn" option to nroff.in tables
> >
> > This prevents the tbl preprocessor from stuffing warnings into the
> > resulting file, which subsequently can make nroff spew warnings like
> > this on stderr:
> >
> > warning: file `nroff.in', around line 47:
> > table wider than line width
> >
> > Unfortunately, I'm not sure how non-gnu tbl programs will react to
> > this option, so YMMV.
> >
> > Signed-off-by: Jeff Layton <[email protected]>
> > ---
> > general/nroff.in | 4 ++--
> > 1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/general/nroff.in b/general/nroff.in
> > index 522ac80..a8acb3a 100644
> > --- a/general/nroff.in
> > +++ b/general/nroff.in
> > @@ -18,7 +18,7 @@ benchmarks.
> > .LP
> > I ran the benchmarks on the following configurations:
> > .TS
> > -center;
> > +center nowarn;
> > l l l l.
> > CPU Unix Fortran Memory
> > .sp 4p
> > @@ -38,7 +38,7 @@ A bubble sort of integers.
> > A prime number generator.
> > .LP
> > .TS
> > -center;
> > +center nowarn;
> > c|c|c s|c s|
> > c|c|c s|c s|
> > c|c|c s|c s|
>
> Here's a radical idea: what say we change the nroff.in file so that it
> no longer tries to fit a 130ish character wide table into an 80
> character wide page?
>

Agreed...

I actually gave that a try first, but after wasting a half hour or so
on it, I just went "nowarn". I'd be all for fixing it the right way if
someone can suggest a patch...

An even more radical idea that Peter S. suggested a while back would be
to update this test to use something more modern than nroff (XML
parsing maybe?)...

--
Jeff Layton <[email protected]>

2011-01-25 17:05:40

by Steve Dickson

[permalink] [raw]
Subject: Re: Patched up cthon source?

I just set up a git tree on linux-nfs.org

git://linux-nfs.org/~steved/cthon04.git

I'll start applying the patches to that tree so we all
will have one common place to pull from... At least for now...

steved.


On 01/24/2011 05:19 PM, Thomas Haynes wrote:
> I'm being evil here on purpose. :->
>
> I'm trying to run cthon04 as downloaded from the connectathon.org repository
> and I'm getting this:
>
>
> Nroff
> ./stat: no data in nroff.time
> general tests failed
> Tests failed, leaving /cthon04/loghyr mounted
>
>
> As I kinda recall several people fixing this already, where would I go to get patches?--
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2011-01-25 10:29:30

by DENIEL Philippe

[permalink] [raw]
Subject: Re: Patched up cthon source?

Thomas Haynes a ?crit :
> I'm being evil here on purpose. :->
>
> I'm trying to run cthon04 as downloaded from the connectathon.org repository
> and I'm getting this:
>
>
> Nroff
> ./stat: no data in nroff.time
> general tests failed
> Tests failed, leaving /cthon04/loghyr mounted
>
>
I had a similar behavior and it was a bug on my side that I solved
during the past bake-a-thom : the 'nroff' step does lots of garbage in
the file, so there is a call to 'grep -v' to get rid off all this
garbage. The result is sent to a secondary file that is moved to the
original file. So there is a 'mv -f a b' were both a and b exist. This
should lead to a situation were the original "b" disappears and is
replaced by "a". This was the bug I had. May be you are in the same
situation. I hope this would help you.

Regards

Philippe

2011-01-24 22:33:40

by Benny Halevy

[permalink] [raw]
Subject: Re: Patched up cthon source?

On 2011-01-24 17:24, Chuck Lever wrote:
>
> On Jan 24, 2011, at 5:19 PM, Thomas Haynes wrote:
>
>> I'm being evil here on purpose. :->
>>
>> I'm trying to run cthon04 as downloaded from the connectathon.org repository
>> and I'm getting this:
>>
>>
>> Nroff
>> ./stat: no data in nroff.time
>> general tests failed
>> Tests failed, leaving /cthon04/loghyr mounted
>>
>>
>> As I kinda recall several people fixing this already, where would I go to get patches?
>
> What are your CFLAGS settings in tests.init?
>

this is a problem with the version of nroff on newer distributions.
The following patch fixed it for me last bakeathon:

>From f44b150809e43b129fad34e3e4ec288e4dd36e6c Mon Sep 17 00:00:00 2001
From: Benny Halevy <[email protected]>
Date: Wed, 6 Oct 2010 11:37:27 -0400
Subject: [PATCH] cthon nroff general test workaround

Newer versions of nroff spew out a warning that cause the cthon test
to fail in the following way:

Nroff
./stat: no data in nroff.time
general tests failed

The following patch filters out the warning from the output file
generated during the test.

Signed-off-by: Benny Halevy <[email protected]>
---
general/runtests.wrk | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/general/runtests.wrk b/general/runtests.wrk
index f3a5a4e..8247e2f 100644
--- a/general/runtests.wrk
+++ b/general/runtests.wrk
@@ -89,6 +89,8 @@ $TIME nroff < nroff.tbl > nroff.out 2>> nroff.time || cat nroff.time
$TIME nroff < nroff.tbl > nroff.out 2>> nroff.time || cat nroff.time
rm nroff.out nroff.tbl
set -e
+egrep -v '^warning:.*$' <nroff.time >nroff.new
+mv -f nroff.new nroff.time
./stat nroff.time
set +e

--
1.7.3.4


2011-01-25 13:43:46

by Anna Schumaker

[permalink] [raw]
Subject: Re: Patched up cthon source?

On 01/24/2011 05:33 PM, Benny Halevy wrote:
> this is a problem with the version of nroff on newer distributions.

My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.

Bryan


--- a/general/runtests.wrk

+++ a/general/runtests.wrk

@@ -88,6 +88,12 @@

$TIME nroff < nroff.tbl > nroff.out 2>> nroff.time || cat nroff.time

$TIME nroff < nroff.tbl > nroff.out 2>> nroff.time || cat nroff.time

rm nroff.out nroff.tbl

+# Filter excessive noise from GNU nroff. Should be harmless for other

+# versions of nroff.

+egrep -v '^warning:.*$' <nroff.time >nroff.new

+mv -f nroff.new nroff.time

+egrep -v '^ table wider.*$' <nroff.time >nroff.new

+mv -f nroff.new nroff.time

set -e

./stat nroff.time

set +e




2011-01-25 15:51:23

by peter.staubach

[permalink] [raw]
Subject: RE: Patched up cthon source?

VGhhdCdzIGEgZ3JlYXQgaWRlYS4NCg0KUGVyaGFwcyB3ZSBjb3VsZCBhbHNvIGZpeCB0aGUgc3Rh
dCBwcm9ncmFtIHRvIGlnbm9yZSBsaW5lcyB0aGF0IGl0IGlzbid0IGxvb2tpbmcgZm9yLi4uDQoN
CgkJcHMNCg0KDQotLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogbGludXgtbmZzLW93
bmVyQHZnZXIua2VybmVsLm9yZyBbbWFpbHRvOmxpbnV4LW5mcy1vd25lckB2Z2VyLmtlcm5lbC5v
cmddIE9uIEJlaGFsZiBPZiBUcm9uZCBNeWtsZWJ1c3QNClNlbnQ6IFR1ZXNkYXksIEphbnVhcnkg
MjUsIDIwMTEgMTA6MzkgQU0NClRvOiBKZWZmIExheXRvbg0KQ2M6IEJyeWFuIFNjaHVtYWtlcjsg
QmVubnkgSGFsZXZ5OyBDaHVjayBMZXZlcjsgVGhvbWFzIEhheW5lczsgbGludXgtbmZzQHZnZXIu
a2VybmVsLm9yZw0KU3ViamVjdDogUmU6IFBhdGNoZWQgdXAgY3Rob24gc291cmNlPw0KDQpPbiBU
dWUsIDIwMTEtMDEtMjUgYXQgMDk6NTYgLTA1MDAsIEplZmYgTGF5dG9uIHdyb3RlOiANCj4gT24g
VHVlLCAyNSBKYW4gMjAxMSAwODo0MzoyNSAtMDUwMA0KPiBCcnlhbiBTY2h1bWFrZXIgPGJqc2No
dW1hQG5ldGFwcC5jb20+IHdyb3RlOg0KPiANCj4gPiBPbiAwMS8yNC8yMDExIDA1OjMzIFBNLCBC
ZW5ueSBIYWxldnkgd3JvdGU6DQo+ID4gPiB0aGlzIGlzIGEgcHJvYmxlbSB3aXRoIHRoZSB2ZXJz
aW9uIG9mIG5yb2ZmIG9uIG5ld2VyIGRpc3RyaWJ1dGlvbnMuDQo+ID4gDQo+ID4gTXkgdmVyc2lv
biBvZiBncm9mZiAoMS4yMSkgaXMgc3BsaXR0aW5nIHRoZSBlcnJvciBpbnRvIHR3byBsaW5lcywg
c28gSSBhbSB1c2luZyB0aGUgZm9sbG93aW5nIHBhdGNoLg0KPiA+IA0KPiA+IEJyeWFuDQo+ID4g
DQo+IA0KPiBZZXAsIHRoaXMgd29ya2Fyb3VuZCBzdG9wcGVkIHdvcmtpbmcgYXJvdW5kIGZvciBt
ZSBvbiBteSByYXdoaWRlIGJveA0KPiBicm9rZSB5ZXN0ZXJkYXkgZm9yIHRoZSBzYW1lIHJlYXNv
bi4NCj4gDQo+IEkgdGhpbmsgdGhpcyBwYXRjaCBtYXkgYmUgYSBiZXR0ZXIgYXBwcm9hY2ggdGhh
biB0cnlpbmcgdG8gZ3JlcCBvdXQNCj4gcmFuZG9tIHN0dWZmLCBidXQgSSBkb24ndCBrbm93IHdo
ZXRoZXIgbm9uLUdOVSB0YmwgcHJvZ3JhbXMgd2lsbCBiYXJmDQo+IG9uIGl0LiBJZiBhbnlvbmUg
aGFzIGEgc29sYXJpcyBib3ggaGFuZHksIGNvdWxkIHlvdSBsZXQgbWUga25vdyBpZg0KPiB0aGlz
IGJyZWFrcyBvbiBpdD8NCj4gDQo+IC0tLS0tLS0tLS0tLS0tLS0tLS0tLS1bc25pcF0tLS0tLS0t
LS0tLS0tLS0tLS0NCj4gW1BBVENIXSBjdGhvbjA0OiBhZGQgIm5vd2FybiIgb3B0aW9uIHRvIG5y
b2ZmLmluIHRhYmxlcw0KPiANCj4gVGhpcyBwcmV2ZW50cyB0aGUgdGJsIHByZXByb2Nlc3NvciBm
cm9tIHN0dWZmaW5nIHdhcm5pbmdzIGludG8gdGhlDQo+IHJlc3VsdGluZyBmaWxlLCB3aGljaCBz
dWJzZXF1ZW50bHkgY2FuIG1ha2UgbnJvZmYgc3BldyB3YXJuaW5ncyBsaWtlDQo+IHRoaXMgb24g
c3RkZXJyOg0KPiANCj4gICAgIHdhcm5pbmc6IGZpbGUgYG5yb2ZmLmluJywgYXJvdW5kIGxpbmUg
NDc6DQo+ICAgICAgIHRhYmxlIHdpZGVyIHRoYW4gbGluZSB3aWR0aA0KPiANCj4gVW5mb3J0dW5h
dGVseSwgSSdtIG5vdCBzdXJlIGhvdyBub24tZ251IHRibCBwcm9ncmFtcyB3aWxsIHJlYWN0IHRv
DQo+IHRoaXMgb3B0aW9uLCBzbyBZTU1WLg0KPiANCj4gU2lnbmVkLW9mZi1ieTogSmVmZiBMYXl0
b24gPGpsYXl0b25AcmVkaGF0LmNvbT4NCj4gLS0tDQo+ICBnZW5lcmFsL25yb2ZmLmluIHwgICAg
NCArKy0tDQo+ICAxIGZpbGVzIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMo
LSkNCj4gDQo+IGRpZmYgLS1naXQgYS9nZW5lcmFsL25yb2ZmLmluIGIvZ2VuZXJhbC9ucm9mZi5p
bg0KPiBpbmRleCA1MjJhYzgwLi5hOGFjYjNhIDEwMDY0NA0KPiAtLS0gYS9nZW5lcmFsL25yb2Zm
LmluDQo+ICsrKyBiL2dlbmVyYWwvbnJvZmYuaW4NCj4gQEAgLTE4LDcgKzE4LDcgQEAgYmVuY2ht
YXJrcy4NCj4gIC5MUA0KPiAgSSByYW4gdGhlIGJlbmNobWFya3Mgb24gdGhlIGZvbGxvd2luZyBj
b25maWd1cmF0aW9uczoNCj4gIC5UUw0KPiAtY2VudGVyOw0KPiArY2VudGVyIG5vd2FybjsNCj4g
IGwgbCBsIGwuDQo+ICBDUFUJVW5peAlGb3J0cmFuCU1lbW9yeQ0KPiAgLnNwIDRwDQo+IEBAIC0z
OCw3ICszOCw3IEBAIEEgYnViYmxlIHNvcnQgb2YgaW50ZWdlcnMuDQo+ICBBIHByaW1lIG51bWJl
ciBnZW5lcmF0b3IuDQo+ICAuTFANCj4gIC5UUw0KPiAtY2VudGVyOw0KPiArY2VudGVyIG5vd2Fy
bjsNCj4gICBjfGN8YyBzfGMgc3wNCj4gICBjfGN8YyBzfGMgc3wNCj4gICBjfGN8YyBzfGMgc3wN
Cg0KSGVyZSdzIGEgcmFkaWNhbCBpZGVhOiB3aGF0IHNheSB3ZSBjaGFuZ2UgdGhlIG5yb2ZmLmlu
IGZpbGUgc28gdGhhdCBpdA0Kbm8gbG9uZ2VyIHRyaWVzIHRvIGZpdCBhIDEzMGlzaCBjaGFyYWN0
ZXIgd2lkZSB0YWJsZSBpbnRvIGFuIDgwDQpjaGFyYWN0ZXIgd2lkZSBwYWdlPw0KDQpDaGVlcnMN
CiAgIFRyb25kDQotLSANClRyb25kIE15a2xlYnVzdA0KTGludXggTkZTIGNsaWVudCBtYWludGFp
bmVyDQoNCk5ldEFwcA0KVHJvbmQuTXlrbGVidXN0QG5ldGFwcC5jb20NCnd3dy5uZXRhcHAuY29t
DQoNCi0tDQpUbyB1bnN1YnNjcmliZSBmcm9tIHRoaXMgbGlzdDogc2VuZCB0aGUgbGluZSAidW5z
dWJzY3JpYmUgbGludXgtbmZzIiBpbg0KdGhlIGJvZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9yZG9t
b0B2Z2VyLmtlcm5lbC5vcmcNCk1vcmUgbWFqb3Jkb21vIGluZm8gYXQgIGh0dHA6Ly92Z2VyLmtl
cm5lbC5vcmcvbWFqb3Jkb21vLWluZm8uaHRtbA0KDQoT77+977+97Lm7HO+/vSbvv71+77+9Ju+/
vRjvv73vv70rLe+/ve+/vd22F++/ve+/vXfvv73vv73Lm++/ve+/ve+/vW3vv71i77+977+9Z37I
p++/vRfvv73vv73cqH3vv73vv73vv73GoHrvv70majordu+/ve+/ve+/vQfvv73vv73vv73vv716
Wivvv73vv70rembvv73vv73vv71o77+977+977+9fu+/ve+/ve+/ve+/vWnvv73vv73vv71677+9
Hu+/vXfvv73vv73vv70/77+977+977+977+9Ju+/vSnfohtm

2011-01-25 15:38:34

by Myklebust, Trond

[permalink] [raw]
Subject: Re: Patched up cthon source?

On Tue, 2011-01-25 at 09:56 -0500, Jeff Layton wrote:
> On Tue, 25 Jan 2011 08:43:25 -0500
> Bryan Schumaker <[email protected]> wrote:
>
> > On 01/24/2011 05:33 PM, Benny Halevy wrote:
> > > this is a problem with the version of nroff on newer distributions.
> >
> > My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.
> >
> > Bryan
> >
>
> Yep, this workaround stopped working around for me on my rawhide box
> broke yesterday for the same reason.
>
> I think this patch may be a better approach than trying to grep out
> random stuff, but I don't know whether non-GNU tbl programs will barf
> on it. If anyone has a solaris box handy, could you let me know if
> this breaks on it?
>
> ----------------------[snip]------------------
> [PATCH] cthon04: add "nowarn" option to nroff.in tables
>
> This prevents the tbl preprocessor from stuffing warnings into the
> resulting file, which subsequently can make nroff spew warnings like
> this on stderr:
>
> warning: file `nroff.in', around line 47:
> table wider than line width
>
> Unfortunately, I'm not sure how non-gnu tbl programs will react to
> this option, so YMMV.
>
> Signed-off-by: Jeff Layton <[email protected]>
> ---
> general/nroff.in | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/general/nroff.in b/general/nroff.in
> index 522ac80..a8acb3a 100644
> --- a/general/nroff.in
> +++ b/general/nroff.in
> @@ -18,7 +18,7 @@ benchmarks.
> .LP
> I ran the benchmarks on the following configurations:
> .TS
> -center;
> +center nowarn;
> l l l l.
> CPU Unix Fortran Memory
> .sp 4p
> @@ -38,7 +38,7 @@ A bubble sort of integers.
> A prime number generator.
> .LP
> .TS
> -center;
> +center nowarn;
> c|c|c s|c s|
> c|c|c s|c s|
> c|c|c s|c s|

Here's a radical idea: what say we change the nroff.in file so that it
no longer tries to fit a 130ish character wide table into an 80
character wide page?

Cheers
Trond
--
Trond Myklebust
Linux NFS client maintainer

NetApp
[email protected]
http://www.netapp.com


2011-01-24 23:01:15

by Jim Rees

[permalink] [raw]
Subject: Re: Patched up cthon source?

Benny Halevy wrote:

this is a problem with the version of nroff on newer distributions.
The following patch fixed it for me last bakeathon:

From f44b150809e43b129fad34e3e4ec288e4dd36e6c Mon Sep 17 00:00:00 2001

Maybe one of our Oracle friends (Chuck?) can push this upstream? The
Connectathon web site still lists cthon04 as the official test suite.

2011-01-24 22:52:41

by Sorin Faibish

[permalink] [raw]
Subject: Re: Patched up cthon source?

Unfortunatelly I got similar complains from our QA running cthon04 in
preparation for the Connectathon. Nothing evil here. :)

/Sorin



On Mon, 24 Jan 2011 17:19:17 -0500, Thomas Haynes <[email protected]>
wrote:

> I'm being evil here on purpose. :->
>
> I'm trying to run cthon04 as downloaded from the connectathon.org
> repository
> and I'm getting this:
>
>
> Nroff
> ./stat: no data in nroff.time
> general tests failed
> Tests failed, leaving /cthon04/loghyr mounted
>
>
> As I kinda recall several people fixing this already, where would I go
> to get patches?--
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>



--
Best Regards

Sorin Faibish
Corporate Distinguished Engineer
Unified Storage Division
EMC?
where information lives

Phone: 508-249-5745
Cellphone: 617-510-0422
Email : [email protected]

2011-01-24 22:25:33

by Chuck Lever III

[permalink] [raw]
Subject: Re: Patched up cthon source?


On Jan 24, 2011, at 5:19 PM, Thomas Haynes wrote:

> I'm being evil here on purpose. :->
>
> I'm trying to run cthon04 as downloaded from the connectathon.org repository
> and I'm getting this:
>
>
> Nroff
> ./stat: no data in nroff.time
> general tests failed
> Tests failed, leaving /cthon04/loghyr mounted
>
>
> As I kinda recall several people fixing this already, where would I go to get patches?

What are your CFLAGS settings in tests.init?

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com





2011-01-25 21:36:37

by Benny Halevy

[permalink] [raw]
Subject: RE: Patched up cthon source?

Great. Thanks!

Benny


-----Original Message-----
From: [email protected] on behalf of Steve Dickson
Sent: Tue 2011-01-25 19:05
To: Thomas Haynes
Cc: [email protected]
Subject: Re: Patched up cthon source?

I just set up a git tree on linux-nfs.org

git://linux-nfs.org/~steved/cthon04.git

I'll start applying the patches to that tree so we all
will have one common place to pull from... At least for now...

steved.


On 01/24/2011 05:19 PM, Thomas Haynes wrote:
> I'm being evil here on purpose. :->
>
> I'm trying to run cthon04 as downloaded from the connectathon.org repository
> and I'm getting this:
>
>
> Nroff
> ./stat: no data in nroff.time
> general tests failed
> Tests failed, leaving /cthon04/loghyr mounted
>
>
> As I kinda recall several people fixing this already, where would I go to get patches?--
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2011-01-24 23:32:56

by Haynes, Tom

[permalink] [raw]
Subject: Re: Patched up cthon source?

Thanks Benny, it works like a charm now!


2011-02-07 22:00:33

by Jeff Layton

[permalink] [raw]
Subject: Re: Patched up cthon source?

On Mon, 7 Feb 2011 14:13:21 -0500
Jim Rees <[email protected]> wrote:

> Jeff Layton wrote:
>
> On Tue, 25 Jan 2011 08:43:25 -0500
> Bryan Schumaker <[email protected]> wrote:
>
> > On 01/24/2011 05:33 PM, Benny Halevy wrote:
> > > this is a problem with the version of nroff on newer distributions.
> >
> > My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.
> >
> > Bryan
> >
>
> Yep, this workaround stopped working around for me on my rawhide box
> broke yesterday for the same reason.
>
> I think this patch may be a better approach than trying to grep out
> random stuff, but I don't know whether non-GNU tbl programs will barf
> on it. If anyone has a solaris box handy, could you let me know if
> this breaks on it?
>
> Your patch does not work for me on Ubuntu with groff 1.20.1.

Ahh yeah, looks like "nowarn" is a very new option. All the more reason
for Steve to drop that patch. He had some other patches to the code that
should make it superfluous anyway.

--
Jeff Layton <[email protected]>

2011-02-07 19:13:23

by Jim Rees

[permalink] [raw]
Subject: Re: Patched up cthon source?

Jeff Layton wrote:

On Tue, 25 Jan 2011 08:43:25 -0500
Bryan Schumaker <[email protected]> wrote:

> On 01/24/2011 05:33 PM, Benny Halevy wrote:
> > this is a problem with the version of nroff on newer distributions.
>
> My version of groff (1.21) is splitting the error into two lines, so I am using the following patch.
>
> Bryan
>

Yep, this workaround stopped working around for me on my rawhide box
broke yesterday for the same reason.

I think this patch may be a better approach than trying to grep out
random stuff, but I don't know whether non-GNU tbl programs will barf
on it. If anyone has a solaris box handy, could you let me know if
this breaks on it?

Your patch does not work for me on Ubuntu with groff 1.20.1.