<div dir="ltr">Hi<div><br></div><div style>I overlooked the newly added watchdog test. Please find the updated attached patch</div><div style><br></div><div style>Thanks</div><div style>Muhammad <span class="" style>Usama</span></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 24, 2013 at 7:18 PM, Tatsuo Ishii <span dir="ltr">&lt;<a href="mailto:ishii@postgresql.org" target="_blank">ishii@postgresql.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch does not include changes for test/regression/tests/004.watchdog/test.sh.<br>
<div class="HOEnZb"><div class="h5">--<br>
Tatsuo Ishii<br>
SRA OSS, Inc. Japan<br>
English: <a href="http://www.sraoss.co.jp/index_en.php" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
Japanese: <a href="http://www.sraoss.co.jp" target="_blank">http://www.sraoss.co.jp</a><br>
<br>
&gt; Thanks Tatsuo<br>
&gt;<br>
&gt; Are you getting hunk failed with the patch?  Patch is applying cleanly on<br>
&gt; the latest code base at my end.<br>
&gt;<br>
&gt; Thanks<br>
&gt; Muhammad Usama<br>
&gt;<br>
&gt;<br>
&gt; On Tue, Jul 23, 2013 at 7:00 PM, Tatsuo Ishii &lt;<a href="mailto:ishii@postgresql.org">ishii@postgresql.org</a>&gt; wrote:<br>
&gt;<br>
&gt;&gt; Thanks! However your patch looks a little bit outdated. Can you please<br>
&gt;&gt; post rebased version?<br>
&gt;&gt; --<br>
&gt;&gt; Tatsuo Ishii<br>
&gt;&gt; SRA OSS, Inc. Japan<br>
&gt;&gt; English: <a href="http://www.sraoss.co.jp/index_en.php" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
&gt;&gt; Japanese: <a href="http://www.sraoss.co.jp" target="_blank">http://www.sraoss.co.jp</a><br>
&gt;&gt;<br>
&gt;&gt; &gt; Hi<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; I tried to make some changes in the regression script, mainly because the<br>
&gt;&gt; &gt; script uses lot of hard coded paths and regression starts reporting<br>
&gt;&gt; &gt; failures if the pgpool-II or PostgreSQL are not found at the default<br>
&gt;&gt; path.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Please find the attached patch which enables the regress.sh script to<br>
&gt;&gt; &gt; install the pgpool on fly in temp directory to run the tests on it<br>
&gt;&gt; &gt; (optional), and now the regress script takes some additional arguments to<br>
&gt;&gt; &gt; specify the paths of required components.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; The list of all arguments accepted by regress.sh script as implemented by<br>
&gt;&gt; &gt; the attached patch are<br>
&gt;&gt; &gt; pgpool2/test/regression$ sh regress.sh -?<br>
&gt;&gt; &gt; Usage:<br>
&gt;&gt; &gt;   regress.sh: [Options]... [test_name]<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Options:<br>
&gt;&gt; &gt;   -p   DIRECTORY           Postgres installed directory<br>
&gt;&gt; &gt;   -b   DIRECTORY           pgbench installed directory, if different from<br>
&gt;&gt; &gt; Postgres installed directory<br>
&gt;&gt; &gt;   -i   DIRECTORY           pgpool installed directory, if already<br>
&gt;&gt; installed<br>
&gt;&gt; &gt; pgpool is to be used for tests<br>
&gt;&gt; &gt;   -m   install/noinstall   make install pgpool to temp directory for<br>
&gt;&gt; &gt; executing regression tests [Default: install]<br>
&gt;&gt; &gt;   -j    FILE               Postgres jdbc jar file path<br>
&gt;&gt; &gt;   -?                       print this help and then exit<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Thanks<br>
&gt;&gt; &gt; Regards<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Muhammad Usama<br>
&gt;&gt;<br>
</div></div></blockquote></div><br></div>