D27863: Revert "Make it compile against last qt5.15 without deprecated method. QProcess::execute(QString) is deprecated"

[Laurent Montel]

mlaurent added inline comments.

INLINE COMMENTS

> kserviceactioncomponent.cpp:72
>          } else {
> -            
> QProcess::startDetached(m_desktopFile.desktopGroup().readEntry(QStringLiteral("Exec"),
>  QString()), QStringList());
> +            
> QProcess::startDetached(m_desktopFile.desktopGroup().readEntry(QStringLiteral("Exec"),
>  QString()));
>          }

Hum... interesting:

#if QT_VERSION < QT_VERSION_CHECK(5, 15, 0)
QProcess::startDetached(m_desktopFile.desktopGroup().readEntry(QStringLiteral("Exec"),
 QString()));
#else
const QString commandLine = 
m_desktopFile.desktopGroup().readEntry(QStringLiteral("Exec"), QString());
if (!commandLine.isEmpty()) {

  QStringList arguments = QProcess::splitCommand(commandLine);
  const QString prog = arguments.takeFirst();
  QProcess::startDetached(prog, arguments);

}   
#endif

REPOSITORY
  R268 KGlobalAccel

REVISION DETAIL
  https://phabricator.kde.org/D27863

To: fvogt, #frameworks, mlaurent
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns