Permalink
Browse files

Make sure that PCDM runs ~/.xprofile NOT as a forked process. That sc…

…ript might need to set env variables, and cannot do that for the entire session if it is a fork of the main process.
  • Loading branch information...
1 parent c43f7c2 commit 18c6ea0a5c07fba0f2819e2113c9b024e609646b Ken Moore committed Jan 27, 2014
Showing with 1 addition and 1 deletion.
  1. +1 −1 src-qt4/PCDM/src/pcdm-xprocess.cpp
@@ -142,7 +142,7 @@ bool XProcess::startXSession(){
// - Setup to run the user's <home-dir>/.xprofile startup script
if(QFile::exists(xhome+"/.xprofile")){
qDebug() << "Run user ~/.xprofile";
- this->start("sh "+xhome+"/.xprofile &");//make sure to start it in parallel
+ this->start("sh "+xhome+"/.xprofile");//Cannot start in parallel if it sets env variables
if(!this->waitForFinished(30000) ){
//If it still has not finished this after 30 seconds, kill it
this->terminate();

0 comments on commit 18c6ea0

Please sign in to comment.