<div dir="ltr"><br><br><div class="gmail_quote">On Tue, Jul 29, 2008 at 9:33 PM, David Lutterkort <span dir="ltr"><<a href="mailto:dlutter@redhat.com">dlutter@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="Wj3C7c">On Tue, 2008-07-29 at 11:52 +0200, Raphael Pinson wrote:<br>
> Improve regexps in dput.aug and php.aug<br>
> Gather regexps before calling IniFile.entry. Call IniFile.entry only once.<br>
> This fixes most of the slowlyness of php.aug<br>
<br>
</div></div>I still have trouble with the patch 'Rewrite dput.aug using inifile.aug'<br>
- the test_dput.aug still fails for me. It seems the problem is the line<br>
<br>
        post_upload_command = /usr/bin/mini-dinstall --batch<br>
<br>
in the ftp-master section - I think the inifile lens does not like<br>
spaces in the value. When I run the tests through augparse, it reports<br>
<br>
        Test run encountered exception:<br>
        lenses/tests/test_dput.aug:59.8-.25:exception: Get did not match entire input<br>
            Error encountered here (1202 characters into string)<br>
            <nd = /path/to/some/script\n#\n|=|><br>
<br>
Can you just resend these two patches and the following as one patch<br>
series:<br>
<br>
      * Fix empty comments in INI file<br>
      * Allow spaces in titles<br>
      * Rewrite dput.aug using inifile.aug<br>
<br>
This is the order in which I applied them.</blockquote><div><br> </div><div><br>I think you're missing the "Allow spaces in entry values" patch.<br><br><br>Raphael <br></div></div><br></div>