[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: KDE 4.1 login fails for new user
From: David Jarvie <djarvie () kde ! org>
Date: 2008-07-15 21:32:11
Message-ID: 200807152232.18670.djarvie () kde ! org
[Download RAW message or body]
On Tue 15 July 2008 16:09:58 you wrote:
> On Saturday 12 July 2008, David Jarvie wrote:
> > I removed the ~/.kde directory and tried to log in using KDE 4.1 (SVN
> > yesterday). It showed the error message "Could not start kstartupconfig4.
> > Check your installation." before aborting the login. The error code being
> > returned by kstartupconfig4 was 3. Repeating the login attempt resulted
> > in the same error.
> >
> > By my reading of the code, kdostartupconfig.cpp calls
> >
> > KStandardDirs::locateLocal( "config", "startupconfigkeys" );
> >
> > and then tries to open the resultant file name in read-only mode - hence
> > the error. When you start with an empty ~/.kde, that file won't exist, so
> > AFAICS kstartupconfig4 will inevitably return the error code 3. (The
> > directory ~/.kde/share/config was created with correct user permissions,
> > presumably by that call, so there's no problem with write access.)
>
> So basically this patch should help?
>
> --- kdostartupconfig.cpp (revision 829274)
> +++ kdostartupconfig.cpp (working copy)
> @@ -80,6 +80,8 @@ int main( int argc, char **argv )
> kDebug() << "Running kdostartupconfig.";
> KCmdLineArgs::init( argc, argv, &about ); // for KLocale not to
> complain about encoding QString keysname = KStandardDirs::locateLocal(
> "config", "startupconfigkeys" ); + if (!QFile::exists(keysname))
> + return 0; // nothing to do
> QFile keys( keysname );
> if( !keys.open( QIODevice::ReadOnly ))
> return 3;
Yes, the patch fixes the problem. I didn't know whether the startupconfigkeys
file should be created empty, or whether the fault should just be ignored, as
in your patch.
--
David Jarvie.
KAlarm author and maintainer.
http://www.astrojar.org.uk/kalarm
["signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic