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

Re: svn commit: rev 6179 - in trunk: . build/ac-macros

From: Justin Erenkrantz <justin_at_erenkrantz.com>
Date: 2003-06-11 07:33:54 CEST

--On Wednesday, June 11, 2003 6:27 AM +0200 Patrick Mayweg <mayweg@qint.de>

> I am sorry that you feal that way. I have posted the modifications twice to
> the mail list. But nobody cared to answer. This patch is needed to configure
> the java-bindings.

The fact that no one 'cared to answer' does not mean that you should commit
without prior review. We all are extremely busy and this is a high-traffic
list. Some posts just slip through. You should keep reposting until someone
replies, but please don't assume that silence equals review and assent.
Please be patient though. (Two posts in less than two weeks isn't a long

That said, I believe that your patch/commit is incorrect as it tries to avoid
the problem rather than resolve the issue raised in 751. It merely expands
the abs_srcdir and abs_builddir at configure-time rather than teaching
svn-config how to deal with installed copies of our dependencies. So, it
doesn't serve to resolve the end problem as discussed in 751, but merely adds
a layer of indirection that doesn't help anyone. It'll only work for your
specific edge-case, but your edge-case isn't at all what svn-config was meant

In your particular case, the correct solution is to integrate your Java
bindings into the Subversion build system. The top-level configure script has
the knowledge of the variables needed to handle the location of the libraries.
Therefore, you should submit patches to integrate those bindings into the main
build system.

You could either add the required logic to build.conf, or I'd personally be
okay with having autoconf create the Makefile in the subversion/bindings/java

I also noticed that you have introduced a dependency of automake into your
bindings. So, you will have to remove those as the core of Subversion itself
does not use automake.

> svn-config does not need to be installed for the patch to work in the
> current form.

The problem is that svn-config wasn't meant to solve the problem of something
in-tree needing the variables. The correct solution in your case is to
generate the Makefiles from the top-level configure.

I have a hunch that it *may* be better to scuttle svn-config and just create a
pkg-config file. I'm not a great fan of pkg-config, but it may be able to
solve the dependency problems in a way that is hard to do otherwise. -- justin

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jun 11 07:34:58 2003

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.