Red Hat Bugzilla – Bug 1023999
startkde adding /bin to $PATH
Last modified: 2013-10-28 16:36:03 EDT
Looks like this snippet in startkde is adding /bin to $PATH: bindir=`echo "$0" | sed -n 's,^\(/.*\)/[^/][^/]*$,\1,p'` if [ -n "$bindir" ]; then qbindir=`$bindir/kde4-config --qt-binaries` qdbus=$qbindir/qdbus $bindir|$bindir:*|*:$bindir|*:$bindir:*) ;; *) PATH=$bindir:$PATH; export PATH;; I added an `echo $bindir > /tmp/bindir` and restarted KDE to make sure. Might be where this is coming from: https://lists.fedoraproject.org/pipermail/devel/2013-October/190776.html Please patch startkde not to do this, it confuses some applications (and users) and is rather undesirable in the post-UsrMove universe. Thanks!
yeah, I think we can assume qdbus is in $PATH already. %changelog * Mon Oct 28 2013 Rex Dieter <rdieter@fedoraproject.org> 4.11.2-3 - startkde adding /bin to $PATH (#1023999) Fix will be rolled into 4.11.3 updates coming soon.
Why is this crap snippet AGAIN in our startkde? We were always patching that out (in past upstream incarnations, it would not add /bin, but the path containing the unsuffixed assistant etc., breaking the Qt 3 .desktop files), the hunk patching it out had already gone lost at least once and I had readded it. Please do not randomly drop hunks of our patches when rebasing patches, they are all there for a reason!
(I'm starting to think that instead of that startkde patch that needs constant rebasing, we should just ship a cleaned startkde we control as a Source* and use that to replace the upstream junk-loaded script wholesale.)