On Wed, 2003-07-02 at 22:55, Patrick Devine wrote:
> 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.
Shouldn't be... changed in CVS accordingly
Jeremy
