[PATCH 2/4] Properly register session with utmp.

Alan Coopersmith alan.coopersmith at oracle.com
Mon Aug 8 12:57:45 PDT 2011


On 08/08/11 12:45, Mark Kettenis wrote:
>> From: =?UTF-8?q?Mat=C4=9Bj=20Cepl?= <mcepl at redhat.com>
>> 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-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list