Hi Hans,

On Sunday, 15. January 2006 21:48, [EMAIL PROTECTED] wrote:
> I've created a 0.14  (see attached patch ;-)) which contains more
> matching rules and some other small fixes.
>
> However to post patches, it would be very convenient to have *readonly*
> svn access.
> I tried using tortoise SVN but I get a forbidden. Am I doing something
> wrong ?

You should be able to create a local working copy of the repository using
the command "svn co http://svn.mutatus.co.uk/repos/perl-ldap/trunk/";.
Using "svn diff" inside the perl-ldap directory you can generate the patch
of your changes, and using "svn update" you can update your working copy
to the latest changes from others.
(sorry, I'm a command line junkie ;-)

> I removed the "use Net::LDAP::Filter" & "use Net::LDAP::Schema" as it
> does not seem to harm code execution.
I'd like to keep then in to keep Perl happy even if the user did not specify 
these two use clauses in his script.
Having them in does not hurt either ;-)

> The module seems to be strict, however the &$match is not allowed when
> strict is active (thats why my initial version used the eval )

Rest committed with a few changes (to make it a bit more readable for me ;-)))

Since the patch did not apply cleanly I had to apply a few hunks of it 
manually. Please check if the result is what you wanted.

> Regarding the approx, we could use Text::Soundex or String::Approx and
> conditionally load that if present on the system, "just" a regexp would
> be perfect, but I haven't come across one yet ;-)

_cis_approxMatch() should do regexes already.
Have a try with the attached test file.

It might not be perfect (owing to interactions of the regex syntax with the 
LDAP filter syntax) but better than none.

CU
Peter

-- 
Peter Marschall
eMail: [EMAIL PROTECTED]

Attachment: fm.pl
Description: Perl program

Reply via email to