<div dir="ltr">Please check this patch, previous patch contained extra changes which is not required. <br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Apr 13, 2013 at 2:30 AM, Abhishek Singh <span dir="ltr">&lt;<a href="mailto:abhishekkumarsingh.cse@gmail.com" target="_blank">abhishekkumarsingh.cse@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Required changes done. Please check the patch.<br><br></div></div><div class="HOEnZb"><div class="h5">
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 12, 2013 at 9:28 PM, Jakub Hrozek <span dir="ltr">&lt;<a href="mailto:jhrozek@redhat.com" target="_blank">jhrozek@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Thu, Apr 11, 2013 at 01:47:03PM +0200, Lukas Slebodnik wrote:<br>
&gt; On (11/04/13 11:18), Jakub Hrozek wrote:<br>
&gt; &gt;On Thu, Apr 11, 2013 at 11:04:06AM +0200, Lukas Slebodnik wrote:<br>
&gt; &gt;&gt; I wanted to revie som patches, but test failed with unrelated issue to revieved<br>
&gt; &gt;&gt; patch.<br>
&gt; &gt;&gt;<br>
&gt; &gt;<br>
&gt; &gt;The test_io.c test should also call tests_set_cwd().<br>
&gt; &gt;<br>
&gt; &gt;Is this only an issue with test_io.c ? Did sysdb_tests run OK? I would<br>
&gt; &gt;expect them to have the same problem, can you check?<br>
&gt;<br>
&gt; I fixed also tests_set_cwd(), there was ignored return value if TEST_DIR is empty<br>
&gt; string. It should not happen with default TEST_DIR value.<br>
&gt;<br>
&gt; Attached patch just fix segmentation fault and improve some lines to prevent<br>
&gt; segfault.<br>
&gt;<br>
&gt; Abhishek can rewrite test_io.c to using tests_set_cwd(),<br>
&gt; (use relative paths)<br>
&gt; Also there something what is not very good solution (transformation to relativepath)<br>
&gt;<br>
&gt;     relativepath = strchr(get_filepath(path), &#39;t&#39;);<br>
&gt;                                              ^^^^^<br>
&gt; What if I decide to call ./configure --with-test-dir=&quot;temp_test&quot;?<br>
&gt;<br>
&gt; LS<br>
<br>
</div></div>Lukas, please send a patch that would fix the above with calling basename,<br>
please. We need to fix this in master asap.<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>