By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 1653

Summary:	KDE4 session launched from TDM has incorrect environment
Product:	TDE	Reporter:	Francois Andriot <albator78>
Component:	tdelibs	Assignee:	Francois Andriot <albator78>
Status:	RESOLVED FIXED
Severity:	normal	CC:	albator78, bugwatch, slavek.banko
Priority:	P5
Version:	R14.0.0 [Trinity]
Hardware:	All
OS:	Linux
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	tdebase : fix TDM variable USER_PATH

Description Francois Andriot 2013-09-07 07:25:45 CDT

Hello, when using TDM, then starting a KDE4 session (strange idea, but ...) the KDE4 session starts with environment variable "PATH=/opt/trinity/bin" prior to "PATH=/usr/bin", so some KDE4 icons and shortcuts (e.g. konsole, konqueror ...) incorrectly run the Trinity version instead of the KDE4 version.

Comment 1 Francois Andriot 2013-09-07 11:35:22 CDT

Looks like it's because of variables defined in "tdmrc".

# The PATH for the Session program.
# Default is "/usr/local/bin:/opt/trinity/bin:/usr/bin:/bin:/opt/trinity/games:/usr/games"
#UserPath=
# The PATH for Setup, Startup and Reset, etc.
# Default is "/usr/local/sbin:/usr/local/bin:/opt/trinity/sbin:/usr/sbin:/opt/trinity/bin:/usr/bin:/sbin:/bin"
#SystemPath=

In source code we find it in "tdebase/tdm/config.def"

I think we should NOT put trinity's specific path in the USER_PATH and SYSTEM_PATH, because these variables are used in ALL desktop environment, not only TDE.

Comment 2 Francois Andriot 2013-09-07 11:42:04 CDT

... but removing the "/opt/trinity" references causes TDM to NOT find the other command it needs (dcopserver, tdmgreeter, etc ...)

We must find a solution so that TDM still works with binaries under /opt/trinity/bin , but that PATH does not contain trinity when starting the user session.

Comment 3 Francois Andriot 2013-09-07 11:54:28 CDT

It looks like keeping the SYSTEM_PATH as is, but just removing the "/opt/trinity" references in USER_PATH, does the trick.
Please, try it.

Comment 4 Francois Andriot 2013-09-08 10:32:16 CDT

Created attachment 1516 [details]
tdebase : fix TDM variable USER_PATH

The attached patch allows starting functional KDE4 session from TDM.

Comment 5 Slávek Banko 2013-09-08 14:06:49 CDT

On my test machine I have next to TDE also LXDE.
With your patch is in LXDE session PATH without paths to TDE.

Good work!
Pushed to GIT in hash b0d67615.