Created attachment 1958 Uniformisation of test method xinitrc script does not use the same test method to evaluate executable names. This patch propose a fix to uniformize some of the tests.
No, this patch is completely wrong. 1. It removes the test of DBUS_SESSION_BUS_ADDRESS to ensure a bus isn't already running. 2. It removes the use of 'eval', so the new environment variables don't get propagated into the shell. Did you even test this? There's no way it can work. Really, there's no reason to change how it is now, except maybe to use 'type -p dbus-launch' instead of 'which dbus-launch', esp since 'which' differs in its output on different platforms when it fails to find an executable. (Also: xinitrc gets generated from xinitrc.in; no need to patch them both.)
Sorry i attached the wrong patch file. You are completly right about this the error. If there is no reason to change it, so no need to fix my error. Just ignore this :) Sorry for the noise. I let you mark the bug to the good status (rejected or so) Regards,
Closing this as per user request.