D5777: Fix check for empty su command for installing as root

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon May 8 20:16:13 UTC 2017


apol requested changes to this revision.
apol added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> makejob.cpp:218
> +        QRegExp regEx(" ");
> +        suCommandWithArg = suCommand.split(regEx);
>          return suCommandWithArg;

Use `KShell::splitArgs`

REPOSITORY
  R32 KDevelop

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

To: geetamc, apol
Cc: apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170508/98a40772/attachment-0001.html>


More information about the KDevelop-devel mailing list