| Subject: | SVN::Look wont install on Windows (ActiveState) |
| Date: | Wed, 17 Mar 2010 14:14:46 -0700 (PDT) |
| To: | bug-SVN-Look [...] rt.cpan.org |
| From: | "Joel Peshkin" <joel [...] peshkin.net> |
Regarding: SVN-Look-0.17
Using: ActiveState Perl 5.10.1003
The makefile of SVN::Look makes several non-portable assumptions...
1) It looks for a file named 'svnlook' and will not match 'svnlook.exe'
2) When it tries the svnlook command, it redirects to /dev/null using
shell redirection that doesn't work in a windows (command.com)
environment.
3) The implementation itself assumes the executable does not have the
'.exe' extension.
4) The tests need similar changes made
----
To really round out the portability, it would be best if the constructor
and makefile accepted an optional argument to specify the path to the SVN
commands. This would help on systems where there are multiple versions of
the commands installed.