sorry I've misread the patch. Feel free to apply your patch :)<div><br></div><div>Aleix<br><br><div class="gmail_quote">On Thu, Dec 9, 2010 at 10:29 PM, Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote"><div><div></div><div class="h5">On Thu, Dec 9, 2010 at 9:56 PM, Morten Volden <span dir="ltr"><<a href="mailto:mvolden2@gmail.com" target="_blank">mvolden2@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5">
Hi All<br><br>I currently use DvcsJob from kdevplatform when invoking commands in the perforce plugin (<a href="http://gitorious.org/perforce-plugin" target="_blank">http://gitorious.org/perforce-plugin</a>). The perforce client executable will read it's current working directory from the environment variable PWD. This variable is apperently is not what is set when calling QProcess::setWorkingDirectory in the DvcsJob constructor.<br>


<br>In DVcsJob::start function there is a call to QProcess::setEnvironment which sets the environment to that of the parent process (I am told that this is redundant since Kjob already sets it). My patch simply removes this line so that a client using DvcsJob can now set the environment of the job. <br>


<br>Regards<br><font color="#888888"><br>Morten<br>
</font><br></div></div>--<div class="im"><br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org" target="_blank">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
<br></div></blockquote></div><div><br></div><div>The working directroy should be specified by the constructor's first parameter, why do you need to pass such static  ::systemSettings() information?</div><div><br></div>

</blockquote></div><br></div>