sitter created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. sitter requested review of this revision.
REVISION SUMMARY as recently mused over on the plasma-devel list. nobody really knows why it's not installed to bindir and at the same time various distributions manually symlink it there because they think it is useful. from a scripting POV it certainly makes more sense to have the tool in bin as libexec may not be easily resolvable (32bit vs 64bit vs architecture triplets on various distributions). by default we'll now symlink kdesu to the bindir under the name kdesu5 (so as to not conflict with kdelibs4's kdesu). this feature can be opted out of by setting KDESU_SYMLINK=OFF TEST PLAN - cmake && make && make install DESTDIR=x - ls -l x/usr/local/bin/kdesu5 - symlinks correctly - cmake -DKDESU_SYMLINK=OFF && make && make install DESTDIR=y - ls -l y/usr/local/bin/kdesu5 - doesn't exist REPOSITORY R126 KDE CLI Utilities BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10541 AFFECTED FILES kdesu/CMakeLists.txt To: sitter Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
