Re: [Tiger-devel] [PATCH] The $MESG check in check_root does not work for SuSE.

[Nicolas François]

Hello Ryan,

On Sat, Oct 18, 2003 at 11:30:01PM -0600, Ryan Bradetich wrote:
> Hello all,
> 
> The current method for checking mesg does not appear to work correctly
> on SuSE.  
> 
> Here is the error that is produced on SuSE:
> 
> # /usr/bin/mesg 2>/tmp/foo | grep "is y"
> # cat /tmp/foo
> mesg: ttyname: Inappropriate ioctl for device

It seems to be caused by a patch I submitted:
http://mail.nongnu.org/archive/html/tiger-devel/2003-09/msg00031.html

I wanted to redirect stderr because when run in a non interractive shell
(e.g. when run by tigercron), mesg (provided by debian's sysvinit_2.85-7
package) reports the following error:
stdin: is not a tty

When run by tigercron, this error was reported by cron, appart from the
tiger report.


If this redirection fails on Suse, I can give a try to a less intrusive
patch: before running mesg, test if stdin is a tty.

I can't reproduce the error you mentionned (Inappropriate ioctl for
device). Can you test the patch attached to this mail?
It works for me, but perhaps tty has the same behaviour as mesg.

> Applying this patch fixes the problem for me and works on HP-UX, RedHat,
> and SuSE.

Does it work with tigercron? (it doesn't here on a debian sid)

Kind Regards,
-- 
Nekral

check_root.patch
Description: Text document