Bug 1023999 - startkde adding /bin to $PATH
Summary: startkde adding /bin to $PATH
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kde-workspace
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-28 14:38 UTC by T.C. Hollingsworth
Modified: 2013-10-28 20:36 UTC (History)
10 users (show)

Fixed In Version: kde-workspace-4.11.2-3
Clone Of:
Environment:
Last Closed: 2013-10-28 14:48:06 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description T.C. Hollingsworth 2013-10-28 14:38:17 UTC
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!

Comment 1 Rex Dieter 2013-10-28 14:48:06 UTC
yeah, I think we can assume qdbus is in $PATH already.

%changelog
* Mon Oct 28 2013 Rex Dieter <rdieter> 4.11.2-3
- startkde adding /bin to $PATH (#1023999)

Fix will be rolled into 4.11.3 updates coming soon.

Comment 2 Kevin Kofler 2013-10-28 20:32:42 UTC
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!

Comment 3 Kevin Kofler 2013-10-28 20:36:03 UTC
(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.)


Note You need to log in before you can comment on or make changes to this bug.