Hi Lars,<br><br><div style="margin-left:40px">Did you get a chance to review the patch? <br><br>If you see a need for any additions/corrections to be made, then do let me know so that i can work on updating the patch accordingly. <br>

<br></div>Thanks and Regards,<br>Samba<br><br>=======================================================================<br><br><div class="gmail_quote">On Tue, Aug 7, 2012 at 4:42 PM, Lars Ellenberg <span dir="ltr">&lt;<a href="mailto:lars.ellenberg@linbit.com" target="_blank">lars.ellenberg@linbit.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 class="im">On Mon, Aug 06, 2012 at 07:13:16PM +0530, Samba wrote:<br>
&gt; Lars,<br>
&gt; I apologise for pushing this one a bit fast but i feel it is better to get<br>
&gt; the issue fixed in the trunk so that we do not have any PRIVATE changes in<br>
&gt; our repository.<br>
&gt;<br>
&gt; Further, having the patch committed to the trunk will give assurance the<br>
&gt; fix is code reviewed and tested properly.<br>
&gt;<br>
&gt; If you have any suggestions/corrections, then do let me know so that I can<br>
&gt; incorporate those and submit an updated patch.<br>
&gt;<br>
&gt; You may also suggest verifying specific test cases in order to ensure<br>
&gt; better test coverage and avoid regression.<br>
&gt;<br>
&gt; I really appreciate the fact that you and other commiters put extra efforts<br>
&gt; on top on  your normal day-job for making this software available to users<br>
&gt; like us. I do know that it needs passion for the project and commitment for<br>
&gt; open source.<br>
&gt;<br>
&gt; I hope you would find some time amidst your busy schedule to review the<br>
&gt; patch and commit it to the trunk.<br>
<br>
</div>I&#39;m finally on vacation, so I may find time tonight to leisurely commit<br>
a few patches ;-)<br>
<div class="im"><br>
<br>
&gt;<br>
&gt; Thanks and Regards,<br>
&gt; Samba<br>
&gt;<br>
&gt; PS:<br>
&gt; Alternatively, how about inviting review comments from other users in the<br>
&gt; community so that we can offload the efforts to some extent. I can guess<br>
&gt; that many users would be needing a fix for the issue of  &#39;sync failure<br>
&gt; during subdir deletion&#39;  so those folks can take this patch and review if<br>
&gt; possible or at least test and verify that it works for them. In this way we<br>
&gt; can build an community over the project and ensure that every patch is<br>
&gt; actually reviewed and tested well before getting committed to the trunk.<br>
<br>
<br>
</div>Additional review is always a nice bonus.<br>
<br>
So if anyone would like to comment, please step forward...<br>
<span class="HOEnZb"><font color="#888888"><br>
        Lars<br>
</font></span></blockquote></div><br>