Sure, check it in.  Be sure to include it in the MANIFEST, ChangeLog, etc.<div><br><div><div class="gmail_quote">On Tue, May 5, 2009 at 7:55 PM, Gavin Carr <span dir="ltr">&lt;<a href="mailto:gavin@openfusion.com.au">gavin@openfusion.com.au</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 Tue, May 05, 2009 at 11:49:20AM -0400, William Wolf wrote:<br>
&gt; I&#39;m trying to use your script to verify the inventory database, but I am<br>
&gt; running into errors.  I am getting the following when I try to run<br>
&gt; ./brackup-verify-inventory -q --delete TARGET<br>
&gt;<br>
&gt;<br>
&gt; Can&#39;t locate object method &quot;count&quot; via package &quot;Brackup::Dict::SQLite&quot; at<br>
&gt; ./brackup-verify-inventory line 111.<br>
&gt;<br>
&gt;<br>
&gt; Did you have to modify lib/Brackup/Dict/SQLite.pm to add a count function<br>
&gt; there?  It also looks like it will be missing an each function, but I may<br>
&gt; be missing something here.  Thanks.<br>
<br>
</div>Eerk, sorry, I&#39;d forgotten I&#39;d patched Dict/SQLite. Patch attached.<br>
<br>
Mainline only has get/set interfaces. Patch adds count, each, and delete.<br>
Brad, are you interested in this?<br>
<br>
Cheers,<br>
<font color="#888888">Gavin<br>
<br>
</font></blockquote></div><br></div></div>