Common code into source/ or toplevel? (Re: Suggested "combined tree" conditions for success)

[Michael Adam]

Jelmer Vernooij wrote:
> On Mon, 2008-09-15 at 09:25 +0200, Stefan (metze) Metzmacher wrote:
> > >>> Personally, I'm just waiting for the combined build system to hit so I
> > >>> can start converting over the utilities I'm working on.
> > >> you can already start, you just need to rebase
> > >> (or git format-patch && git am) when the final branch is ready...
> > > 
> > > I'm waiting for being able to "make" in the root dir, but you're right. It 
> > > should be possible to move the first utils now. I'm not short on todo list 
> > > items, though, so I don't lose much by waiting for Jelmer's updated build 
> > > system patch.
> > 
> > As we now have the combined 'master' branch with source3/ and source4/
> > sub directories, we can think about where to move common code
> > and the build magic for a combined 'make'.
> > 
> > I'd propose to move them to source/, to that all 3 ways of building
> > code in the master branch have the same weight. I assume have the magic
> > of the combined build in the top level directory would give more weight
> > to the most experimental way of building samba.
> > 
> > Comments please!
> I would prefer to have the source code directly in the top-level
> directory, at least the libraries (lib/tdb rather than source/lib/tdb).

This is my preference, too.

> Perhaps we can leave just the merged build configure + Makefile out of
> the top-level source directory for now?

Sounds like a good idea: this will make the still experimental
stuff not the most prominent choice. Subfolder "franky" for the
merged build?

Cheers - Michael

-- 
Michael Adam <ma at sernet.de>
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.SerNet.DE, mailto: Info @ SerNet.DE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 206 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20080916/00f1ce2c/attachment.bin