[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace/kcontrol/kfontinst
From:       David Faure <faure () kde ! org>
Date:       2009-10-02 20:46:33
Message-ID: 200910022246.33924.faure () kde ! org
[Download RAW message or body]

On Thursday 24 September 2009, Craig Drummond wrote:
> SVN commit 1027819 by craig:
> 
> Copy over re-worked fontinst from branches/work/kfontinst
> 
> Main changes:
> 
> * Create a dbus daemon for handling font installtion, and have the kcm
> and ioslave use this.
> A             dbus/org.kde.fontinst.service.cmake

Hi,

this change makes fonts:/System just hang for me.
It's blocks in QCoreApplication::exec (1), waiting for dbus signals from
"fontinst" which is not running. I see that you installed a \
org.kde.fontinst.service file for dbus, but this doesn't work when  install \
KDE into a custom prefix (which is the recommended way for development...)

Can you manually start the service when it's not already running?
It should be as simple as
   if (!QDBusConnection::sessionBus().interface()->isServiceRegistered("org.kde.fontinst")) \
{  // start it (via QProcess or KToolInvocation)
   }

(1) the on-demand QCoreApplication creation reads a bit strange, how about \
just doing that in the kioslave main? That's an okay thing to do, other \
slaves do it too. And then you can use QEventLoop exec() + quit() rather \
than QCoreApplication exec() + exit() (which reads funky because it doesn't \
actually exit the application). For an example, see \
kioslave/thumbnail/htmlcreator.cpp

./thumbnail/htmlcreator.cpp:68:    \
                m_eventLoop.exec(QEventLoop::ExcludeUserInputEvents);
./thumbnail/htmlcreator.h:26:#include <QEventLoop>
./thumbnail/htmlcreator.h:47:    QEventLoop m_eventLoop;
void HTMLCreator::slotCompleted()
{
    m_eventLoop.quit();
}

-- 
David Faure, faure@kde.org, sponsored by Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic