[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: [PATCH] Don't break 'make install-tools' if TOOLSDIR is same as BINDIR.

From: Gavin Baumanis <gavinb_at_thespidernet.com>
Date: Thu, 7 Mar 2013 09:59:25 -0500

Ping.
Just thought I would bring this back - as I haven’t noticed a commit
response.

Gavin.

> -----Original Message-----
> From: Julian Foad [mailto:julianfoad_at_btopenworld.com]
> Sent: Monday, 11 February 2013 08:54
> To: Alexander Thomas
> Cc: dev_at_subversion.apache.org
> Subject: Re: [PATCH] Don't break 'make install-tools' if TOOLSDIR is same
as
> BINDIR.
>
> Alexander Thomas wrote:
> > This originated from one of my requirement where svn* binaries and
> > tools need to be installed in same location.
> > I'm not sure this one qualifies as a valid patch, because for this to
> > happen someone should tweaks Makefile.in and points $bindir and
> > $toolsdir to same location.
>
> I think this is fine and we should apply it.  I'm having trouble testing
it at the
> moment because 'make install-tools' is failing for me at an earlier stage,
but
> the patch looks fine as far as I can see.
>
> - Julian
>
>
> > [[[
> > Don't break 'make install-tools' if TOOLSDIR is same as BINDIR.
> >
> > * Makefile.in
> >   (INSTALL_EXTRA_TOOLS): Check that target and link_name are not the
> >    same before creating symlink to svnmucc in BINDIR.
> >
> > Patch by: Alexander Thomas <alexander{_AT_}collab.net> ]]]
>
> (For quick reference, the patch is basically this, in Makefile.in:)
> + if test "$(DESTDIR)$(bindir)" != "$(DESTDIR)$(toolsdir)"; then \
>     ln -sf $(DESTDIR)$(bindir)/svnmucc$(EXEEXT)
> $(DESTDIR)$(toolsdir)/svnmucc$(EXEEXT); \
> + fi
Received on 2013-03-07 16:00:24 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.