Patched up cthon source?

Patched up cthon source?

[Thomas Haynes]

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?

Re: Patched up cthon source?

[Chuck Lever]

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

Re: Patched up cthon source?

[Benny Halevy]

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 <bhalevy@panasas.com>
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 <bhalevy@panasas.com>
---
 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

Re: Patched up cthon source?

[sfaibish]

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 <thomas@netapp.com>  
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 majordomo@vger.kernel.org
> 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 : sfaibish@emc.com

Re: Patched up cthon source?

[Jim Rees]

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.

Re: Patched up cthon source?

[Thomas Haynes]

Thanks Benny, it works like a charm now!

Re: Patched up cthon source?

[DENIEL Philippe]

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

Re: Patched up cthon source?

[Bryan Schumaker]

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

Re: Patched up cthon source?

[Jeff Layton]

On Tue, 25 Jan 2011 08:43:25 -0500
Bryan Schumaker <bjschuma@netapp.com> 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 <jlayton@redhat.com>
---
 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

Re: Patched up cthon source?

[Trond Myklebust]

On Tue, 2011-01-25 at 09:56 -0500, Jeff Layton wrote: 
> On Tue, 25 Jan 2011 08:43:25 -0500
> Bryan Schumaker <bjschuma@netapp.com> 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 <jlayton@redhat.com>
> ---
>  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
Trond.Myklebust@netapp.com
www.netapp.com

RE: Patched up cthon source?

[peter.staubach]

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

Re: Patched up cthon source?

[Jeff Layton]

On Tue, 25 Jan 2011 10:38:32 -0500
Trond Myklebust <Trond.Myklebust@netapp.com> wrote:

> On Tue, 2011-01-25 at 09:56 -0500, Jeff Layton wrote: 
> > On Tue, 25 Jan 2011 08:43:25 -0500
> > Bryan Schumaker <bjschuma@netapp.com> 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 <jlayton@redhat.com>
> > ---
> >  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 <jlayton@redhat.com>

Re: Patched up cthon source?

[Steve Dickson]

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 majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: Patched up cthon source?

[Halevy, Benny]

Great. Thanks!

Benny


-----Original Message-----
From: linux-nfs-owner@vger.kernel.org on behalf of Steve Dickson
Sent: Tue 2011-01-25 19:05
To: Thomas Haynes
Cc: linux-nfs@vger.kernel.org
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 majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Patched up cthon source?

[Jim Rees]

Jeff Layton wrote:

  On Tue, 25 Jan 2011 08:43:25 -0500
  Bryan Schumaker <bjschuma@netapp.com> 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.

Re: Patched up cthon source?

[Jeff Layton]

On Mon, 7 Feb 2011 14:13:21 -0500
Jim Rees <rees@umich.edu> wrote:

> Jeff Layton wrote:
> 
>   On Tue, 25 Jan 2011 08:43:25 -0500
>   Bryan Schumaker <bjschuma@netapp.com> 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 <jlayton@redhat.com>