[Monetdb-developers] Proposed changes to build setup
sjoerd at acm.org
Wed Jan 17 10:26:48 CET 2007
On 2007-01-17 09:57, Jens Teubner wrote:
> Hi all,
> recently, Jan and I have looked at some aspects of the
> Pathfinder/MonetDB build process. And we had the impression that there
> is some inconsistency in the current setup.
> The makefiles of all MonetDB-related projects currently include
> $MONETDB_CONFDIR/rules.mk. This file basically contains instructions on
> how to use the tools from the buildtools package to compile all the
> projects from CVS sources. To compile a project from distributed
> tarballs, none of these rules should be necessary. Thus, I think a much
> better place to host the rules.mk file would be in the buildtools
> Thus I would suggest to do the following changes and ask for your
> comments whether these changes make sense:
> -- rules.mk goes to the buildtools package. This is where it belongs
> -- The buildtools package gets its `buildtools-config' bash script
> in order to allow remaining packages to actually find rules.mk.
> -- Instead of
> include $(MONETDB_CONFDIR)/*.mk
> the bootstrap script inserts
> if HAVE_BUILDTOOLS
> include $(BUILDTOOLS_CONFDIR)/rules.mk
> to make the inclusion of the rules dependent on the outcome of the
> ./configure run. (Otherwise, we would depend on the buildtools in
> the distribution tarballs, which we don't want.)
> -- The tests invoked by ./configure finally need to take care of
> setting HAVE_BUILDTOOLS appropriately (using buildtools-config).
> If nobody objects, I would do the necessary changes as soon as we have
> forked off the new stable branch.
I'm not entirely sure that what you say is correct. I'm not convinced
that none of the rules are needed if you compile from tarballs. In
other words, check whether that actually works before you commit the change.
If it does work, I'm not against such a change.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 370 bytes
Desc: OpenPGP digital signature
More information about the developers-list