[kde-windows] [Bug 315040] New: "there are kde applications running" message on updating

[Mihai Sorin Dobrescu]

https://bugs.kde.org/show_bug.cgi?id=315040

            Bug ID: 315040
           Summary: "there are kde applications running" message on
                    updating
    Classification: Unclassified
           Product: kde-windows
           Version: 4.8.0
          Platform: MS Windows
                OS: MS Windows
            Status: UNCONFIRMED
          Severity: major
          Priority: NOR
         Component: installer
          Assignee: kde-windows at kde.org
          Reporter: msdobrescu at gmail.com

I have installed KDE on Windows 7 Enterprise 64bit, msvc build, and I try to
update it.
There are no KDE processes except the KDE installer itself, but the "there are
kde applications running" message appears and KDE can't be updated.



Reproducible: Always

Steps to Reproduce:
Follow the update procedure: 
1. Start the installer only (nu other KDE apps running)
2. Let it detect the updates
3. Start update
4. A dialog box appears saying: "there are kde applications running"
5. The installation stops as the installer closes.
Actual Results:  
The upgrading fails.

Expected Results:  
The KDE processes should be identified correctly and the installation
(upgrading) should continue.

The issue is in the code:

bool InstallerEngine::isAnyKDEProcessRunning()
{
QString cmd = m_root +""/bin/kdeinit4.exe"";
QProcess p;
QStringList args = QStringList() << ""--list"";
p.start(cmd,args);
if (!p.waitForStarted())
{
qCritical() << ""could not start"" << cmd << args;
return false;
}
if (!p.waitForFinished())
{
qCritical() << ""failed to run"" << cmd << args;
return false;
}
QByteArray _stderr = p.readAllStandardError();
qDebug() << ""run"" << cmd << args << ""without errors"" << _stderr;
QList<QByteArray> lines = _stderr.split('\n');
int ret = lines.size() - 1; // because of trailing '\n'
// one line means ony kdeinit4 is running
return ret > 1;
}

M:\KDE\bin>kdeinit4 --list
path: M:/KDE/SETUP name: kdewin-installer-gui-latest pid: 3936
path: M:/KDE/bin name: kdeinit4 pid: 7732

M:\KDE\bin>

So, there is an additional line, because kdeinit4 reports itself, the installer
process is also there and the \n too.

-- 
You are receiving this mail because:
You are the assignee for the bug.