On 20020226 Frederic Lepied wrote:
>"J.A. Magallon" <[EMAIL PROTECTED]> writes:
>
>> Hi.
>> 
>> This patch makes msec work as could be expected wrt the allow_user_list()
>> function, ie, 'allow(1)' means respect user wishes or manual config,
>> not 'force'. And 'allow(0)' means 'force no user list'.
>> 
>> --- /usr/share/msec/libmsec.py.orig  Tue Feb 26 00:44:02 2002
>> +++ /usr/share/msec/libmsec.py       Tue Feb 26 00:46:10 2002
>> @@ -229,8 +229,6 @@
>>  
>>      if arg:
>>          _interactive and log(_('Allowing the listing of users in display 
>managers'))
>> -        kdmrc.exists() and kdmrc.set_shell_variable('ShowUsers', 'All')
>> -        gdmconf.exists() and gdmconf.set_shell_variable('Browser', 'true')
>>      else:
>>          _interactive and log(_('Disabling the listing of users in display 
>managers'))
>>          kdmrc.exists() and kdmrc.set_shell_variable('ShowUsers', 'None')
>> 
>> I think this is the correct fix. I am beginning to hate msec re-enabling
>> user list each time I upgrade it. It is no more fun.
>
>I can't take this fix because it breaks level switch. The correct way
>to fix your problem is either to remove msec if you consider it
>doesn't fit your need or to add the following to
>/etc/security/msec/level.local to change the default behaviour:
>
>from mseclib import *
>
>allow_user_list(0)

Well, then at least s/allow/enable/, to make names consistent with behaviour.

-- 
J.A. Magallon                           #  Let the source be with you...        
mailto:[EMAIL PROTECTED]
Mandrake Linux release 8.2 (Cooker) for i586
Linux werewolf 2.4.18-jam1 #1 SMP Tue Feb 26 00:06:55 CET 2002 i686

Reply via email to