On 2 Jul 2003, Jeremy Katz wrote: > On Mon, 2003-06-30 at 14:24, Patrick Devine wrote: > > I looked through the code and found there is a line which changes the > > Xauth file to point at the wrong place: > > > > if os.environ.has_key('HOME'): > > os.environ['XAUTHORITY'] = os.environ['HOME'] + '/.Xauthority' > When this happens, don't you have $XAUTHORITY set already? That's > probably the better thing to check for not existing (basically, > XAUTHORITY needs to be set and this is making sure it is). Yes, it's already set, so testing for it would probably be a better check. I wasn't sure if there was a case where you actually would want to overwrite $XAUTHORITY though so I was trying to be somewhat cautious. --Patrick.