Skip Menu |

This queue is for tickets about the SVN-Notify-Config CPAN distribution.

Report information
The Basics
Id: 16505
Status: resolved
Priority: 0/
Queue: SVN-Notify-Config

People
Owner: jpeacock [...] cpan.org
Requestors: grousse [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.06
Fixed in: (no value)



Subject: No possibility to configure svnlook binary before it is used
A new SVN::Notify insteance is created at line 127, without any configuration value. It triggers a systematic failure if svnlook is not at default location (/usr/local/bin/svnlook) or SNVLOOK environment variable is not set (which is unpractical in a remote SVN repository). It could be considered a more general bug in SVN::Notify to hardcode a binary location instead of relying on path for finding it, but it makes SVN::Notify::Config unusable with subversion installed by distribution packages under /usr.
David Wheeler added a new method to SVN::Notify so that subclasses can easily search for the binaries need to do their work, rather than relying on a hard-coded path. This limitation will be fixed in SVN::Notify::Config 0.8. John
My testing with newest SVN::Notify and the 0.8 release of SVN::Notify::Config shows that there is no longer a problem with svnlook binaries located in "non-standard" locations. John