Re: [PATCH] xfstests: fix nr of args in open calls

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Sat, Oct 25, 2008 at 03:19:08PM -0500, Eric Sandeen wrote:
> Christoph Hellwig wrote:
> > On Sat, Oct 25, 2008 at 01:45:53PM -0500, Eric Sandeen wrote:
> >> --- xfstests.orig/ltp/iogen.c
> >> +++ xfstests/ltp/iogen.c
> >> @@ -1060,7 +1060,11 @@ int         nbytes;
> >>  
> >>      Oflags |= O_CREAT | O_WRONLY;
> >>  
> >> +#ifdef CRAY
> >>      if ((fd = open(path, Oflags, 0666, Ocbits, Ocblks)) == -1) {
> >> +#else
> >> +    if ((fd = open(path, Oflags, 0666)) == -1) {
> >> +#endif
> > 
> > Do we really need that ifdef cray here?
> 
> Not sure; Ocbits etc are set under the same #ifdef, xfsqa runs on irix,
> and I'm not sure what all gets set on that build ...

#ifdef CRAY is for Unicos, and I don't think xfsqa near it's current
version can run there.  I'd rather see all this striped out, but at
least no new occurances added.