D5122: Allow users to install via a custom root installation command

[Friedrich W. H. Kossebau]

kossebau added inline comments.

INLINE COMMENTS

> zhigalin wrote in makeconfig.ui:122
> sudo just doesn't work as we don't have an interactive terminal here, so I removed it
> However, if a user has previously selected sudo, it will be kept as transition code in `makejob.cpp` still works

Ah, now I see, okay, so that is for backward compat support.

Please add a comment to the `if(suCommandIsDigit)` code section to make that clear,
as otherwise some future reader of the code could be confused as well by the handling of those magic numbers.

REPOSITORY
  R32 KDevelop

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

To: geetamc, kfunk, zhigalin
Cc: kossebau, zhigalin, ltoscano, ematirov, kfunk, brauch, kdevelop-devel, #kdevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170507/6554eae9/attachment.html>