Logic error in (4.4b2) Xsetup

[Jeremy Wilkins]

This Xsetup file actually exists prior to KDE 4.4b2 on 9.10. I have KDE
4.3.4 with the very 
same lines. I believe it was a way to make the KDM greeter seem like a part
of the startup 
process, hide the long KDM startup delay and after login the splash
continues. KDM is supposed to 
kill the splash screen automatically once it starts. Correct me if I'm
wrong. I don't think 
autologin requires the spash to be run since it shows the splash on its own.
On the other hand, I think It is also a very hackish way of testing for
AutoLoginEnable 
since any case/character or space mismatch will fail to evaluate entirely.
For instance if 
the true were True that wouldn't work. It would also fail if the line
started with a space. 
Theoretically neither would happen as long as you are using the current
programs to set auto login, but 
the rc file should be parsed as variables properly to ensure a future proof
setup where a mild 
space wouldn't clow up the whole setup.

----------------original message-----------------
From: "Ike Sumera" ike.sumera at gmail.com To: "Kubuntu Developer Discussion" 
kubuntu-devel at lists.ubuntu.com Date: Sun, 03 Jan 2010 16:22:24 -0600
----------------------------------------------------------

>
>If it could not find AutoLoginEnable=true then show ksplash...
>
>
>On Sat, Jan 2, 2010 at 11:38 AM, Zorael <zorael at gmail.com zorael at gmail.com
wrote:
>Hiho,
>
>I'm running 9.10 with 4.4b2 packages from the kubuntu-ppa/beta ppa. I
>have my machine set up to not login automatically, but kdm still
>displays ksplashx before showing the greeter. So I had a look at
>/etc/kde4/kdm/Xsetup, which looks like the following after removing
>comments;
>
> if ! grep -q ^AutoLoginEnable=true /etc/kde4/kdm/kdmrc; then
> ksplashx Default +
> fi
>
>The logic there translates to "if autologin is disabled, display
>ksplashx". (^ doesn't negate)
>
>Shouldn't it be the other way around? To make it only display ksplashx
>if autologin is enabled?
>
>
>Zorael
>
>--
>kubuntu-devel mailing list
>kubuntu-devel at lists.ubuntu.com kubuntu-devel at lists.ubuntu.com Modify 
>settings or unsubscribe at:
[https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel 
>-> https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel]
>
>
>
>__________________________________________________
>
>
>-- 
>kubuntu-devel mailing list
>kubuntu-devel at lists.ubuntu.com Modify settings or unsubscribe at: 
>https://lists.ubuntu.com/mailman/listinfo/kubuntu-devel
>
>
>
-- 

Jeremy Wilkins
(306) 382-8977

-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20100103/cbed3320/attachment.htm