Re: r23113 changes

On 1/21/07, Nik Clayton <nik@ngo.org.uk> wrote:
> OK. Can I make sure there's a consensus on this before I expend much more
> effort please. I've bought this up a couple of times before. I first
> mentioned the idea on 24th Nov, then posted patches that implemented it on
> 30th Nov and 12th Dec, and again mentioned the external dependency
> requirement on 17th Jan, before I committed.
>
> Each time, the only comment was that we needed to document the requirement
> clearly.

Yeah, I've been pretty busy with school and work lately and haven't
had time to monitor the list as well as I'd like to.

Perl is Perl and writing good Perl does generally mean using CPAN
modules. And I see you have added the dependency to swig/INSTALL.
Perhaps you should mention a specific version number that you have
tested as working, and make sure (at least manually, if not in the
test suite) that both the pure-Perl and XS implementation of
Param::Validate work?

(And we should make sure that when a version with this in it is
released, we don't forget to include the new dep in CHANGES, and
perhaps even proactively notify some of the packagers who provide the
bindings.)

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org