D10541: make kdesu available in bindir and thus $PATH by default (opt-out)

Harald Sitter noreply at phabricator.kde.org
Thu Feb 15 13:46:01 UTC 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180215/3ecfc79d/attachment-0001.html>


More information about the Plasma-devel mailing list