ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  It doesn't compile:
  
    /home/nate/kde/src/kio/src/widgets/kpropertiesdialog.cpp:1389:72: error: 
‘QString::QString(const char*)’ is private within this context
     1389 |     KRun::runCommand(QStringLiteral("filelight"), directory, 
nullptr, QStringLiteral("/"));
          |                                                                     
   ^
  
  Anyway, should use `KRun::runApplication()` for this since that's made for 
launching GUI apps and handling arguments passed to the binary.

REPOSITORY
  R241 KIO

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

To: shubham, ngraham, #frameworks
Cc: kde-frameworks-devel, #frameworks, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to