On 12 October 2011 03:06, Matěj Cepl <span dir="ltr">&lt;<a href="mailto:mcepl@redhat.com">mcepl@redhat.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">&gt; &gt; +    # Return back to the original path<br>
&gt; &gt; +    os.chdir(start_dir)<br>
&gt;<br>
</div><div class="im">&gt; Is this actually necessary?  This only resets the working directory for the<br>
&gt; process that is just about to terminate.<br>
<br>
</div>Yes, you are right. I was afraid of changing cwd of calling shell, but that&#39;s<br>
nonsense.<br>
<br>
Otherwise I have just rewritten and extended patch commentary.<br>
<br>
-----------------------------------<br>
The main purpose of this patch is to make piglit independent of the<br>
current working directory, so it is possible to package piglit as a RPM<br>
package (with binaries symlinked to /usr/bin, most of the files in<br>
read-only /usr/lib*/piglit directory, and results somewhere else).<br>
<div class="im"><br>
So it is now possible to run<br>
<br>
$ piglit-run tests/quick-driver.tests /tmp/piglit<br>
<br>
and then with this command<br>
<br>
$ piglit-summary-html --overwrite /tmp/piglit/results /tmp/piglit/main<br>
<br>
generate a report in /tmp/piglit/results/index.html &amp; al.<br>
<br>
</div><div class="im">Signed-off-by: Matěj Cepl &lt;<a href="mailto:mcepl@redhat.com">mcepl@redhat.com</a>&gt;<br></div></blockquote><div><br>I verified with Ian that these two patches address his comments, so I&#39;ve pushed them to master.<br>
</div></div>