On 08/08/11 12:45, Mark Kettenis wrote:
>> From: =?UTF-8?q?Mat=C4=9Bj=20Cepl?= <[email protected]>
>> Date: Mon,  8 Aug 2011 21:26:48 +0200
>>
>> I guess we should have corresponding sessreg -d call in config/TakeConsole
>> as well, right?
>>
>> diff --git a/config/GiveConsole b/config/GiveConsole
>> index f8a8c6d..d201a5c 100644
>> --- a/config/GiveConsole
>> +++ b/config/GiveConsole
>> @@ -7,3 +7,5 @@
>>  # causing serious grief.
>>  #
>>  chown $USER /dev/console
>> +exec /usr/bin/sessreg  -a -w "/var/log/wtmp" -u "/var/run/utmp" \
>> +        -x "/etc/X11/xdm/Xservers" -l $DISPLAY -h "" $USER
> 
> $ ls -l /usr/bin/sessreg
> ls: /usr/bin/sessreg: No such file or directory
> 
> Guess that's a NACK.

config/Xstartup.cpp & config/Xreset.cpp already call BINDIR/sessreg so doing
the same in another config file should be fine - but it shouldn't be hardcoded
to /usr/bin as this patch does.

-- 
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to