Re: [patch] fileio.exp FAILs if run as root

[Daniel Jacobowitz <drow@false.org>]

On Tue, Dec 11, 2007 at 06:46:26PM +0100, Jan Kratochvil wrote:
> There are various warnings around, at least I see now on the console
> 	WARNING: Couldn't find the global config file.
> 
> but apparently nobody cares (do you also see this warning?).  I had to start

It comes from DejaGNU.  There's nothing much we can do about it; set
DEJAGNU in your environment to point to a local site.exp, and it will
go away.

> There are multiple possibilities:
> 
> (1) Keeping there setuid() before somebody complains it does not work.
>     http://sources.redhat.com/ml/gdb-patches/2007-12/msg00135.html
> 
> (2) Pointing me/anyone at the system where this setuid() code does not work.

It will not work on any of the systems the testcase was designed for,
i.e. those which use the remote File I/O protocol for file operations.

> (3) Skipping (not FAILing) the two tests known they FAIL on the 'root' run.
> 
> (4) Disabling running the testsuite as 'root'.
> 
> (5) Giving just a warning during the start.
> 
> (6) I am too dumb to become a GDB user.
> 
> more?

Just ignoring the two failures if you need to run as root for some
reason, or xfailing them locally, IMO.  I think a warning is quite
sufficient.  Perhaps in the FAIL message.

-- 
Daniel Jacobowitz
CodeSourcery