diff options
author | Alberto Villa <avilla@FreeBSD.org> | 2011-07-17 16:07:23 +0000 |
---|---|---|
committer | Alberto Villa <avilla@FreeBSD.org> | 2011-07-17 16:07:23 +0000 |
commit | c2f28f7e83083cd9486b9e3df8cff36d87da4507 (patch) | |
tree | 07cad62e878509241ca7463f244860d1033a6f9f /security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp | |
parent | - Update to 1.6.0. (diff) |
- Backport patches to fix KCalc issues.
Submitted by: Raphael Kubo da Costa <kubito@gmail.com> via area51
Notes
Notes:
svn path=/head/; revision=277858
Diffstat (limited to 'security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp')
-rw-r--r-- | security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp b/security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp new file mode 100644 index 000000000000..f2298e0f4493 --- /dev/null +++ b/security/kgpg-kde4/files/patch-ark-kerfuffle-cliiterface.cpp @@ -0,0 +1,35 @@ +SVN commit 1240907 by rkcosta: + +cliinterface: Do not assume a process does not exist in runProcess(). + +The Q_ASSERT(!m_process) in the beginning of runProcess() was wrong, as when +one is adding files to an archive (or creating an archive) processFinished() +calls list(), which in turn calls runProcess() as well, thus making the assert +fail. + +The best thing to do is just wait for the existing process to finish before +launching the new one. + +Backport of r1240907, in case the 4.6 ever gets packaged again. + +CCBUG: 277393 + +http://websvn.kde.org/?view=revision&revision=1240908 +-- ark/kerfuffle/cliinterface.cpp 2011/06/01 20:35:06 1234597 ++++ ark/kerfuffle/cliinterface.cpp 2011/07/11 03:23:39 1240908 +@@ -325,10 +325,13 @@ + return false; + } + +- Q_ASSERT(!m_process); +- + kDebug() << "Executing" << programPath << arguments; + ++ if (m_process) { ++ m_process->waitForFinished(); ++ delete m_process; ++ } ++ + #ifdef Q_OS_WIN + m_process = new KProcess(); + #else |