Re: [PATCH] Correct path to Perl modules (issue 2479)

Mattias Engdegård wrote:
> Sorry for the repost, but the Perl modules have been broken for such a
> long time. Please apply, or tell me what is wrong with it.
> I see no way this could be worse than what already is, in any case.

I suspect your patch was warnocked[1] because no is really sure what is
the "right" way to fix this problem and the Perl binding don't currently
have a "champion" amongst the core developers. For example, see this
thread from last October:

        http://svn.haxx.se/dev/archive-2005-10/0536.shtml

There have been several attempts to deal with the difference in the Perl
bindings (which don't use libtool to automatically link in the correct
libraries) as opposed to the other bindings. Your fix suffers from the
same problem as some of the other attempts, in that the bindings will
use the already installed library files, rather than in the build
directory. This can produce some subtle and difficult to catch
problems. The current "best practice" is to build and install the Perl
bindings before testing them (which isn't really a great idea, I must
admit).

I wish I had the time to go through the Perl binding in detail and
rework them to use a regular makefile and libtool (just for
convenience's sake for the rest of the project) so the bindings would
get linked to the right library all the time automatically. It may be
that using appropriately shaped LD_LIBRARY_PATH would do the job instead.

John

1) http://en.wikipedia.org/wiki/Warnock

-- 
John Peacock
Director of Information Research and Technology
Rowman & Littlefield Publishing Group
4501 Forbes Boulevard
Suite H
Lanham, MD  20706
301-459-3366 x.5010
fax 301-429-5748
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org