Okay - Thanks :-)<br><br><div class="gmail_quote">2010/12/9 Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
sorry I've misread the patch. Feel free to apply your patch :)<div><br></div><div><font color="#888888">Aleix</font><div><div></div><div class="h5"><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" target="_blank">aleixpol@kde.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="gmail_quote"><div><div></div><div>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: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div><div></div><div>
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><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></div></div>
<br>--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">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></blockquote></div><br><br>