code duplication in ccan (was: Re: snprintf on SunOS)

[simo]

On Fri, 2012-06-15 at 00:36 +0200, Jelmer Vernooij wrote: 
> Hi Rusty,
> 
> On Sun, Jun 10, 2012 at 01:02:50PM +0930, Rusty Russell wrote:
> > On Sat, 9 Jun 2012 01:18:29 +0200, Jelmer Vernooij <jelmer at samba.org> wrote:
> > > We can deprecate functionality from samba-util, but we'll still end up
> > > with the same issue - two libraries that do essentially the same
> > > thing.
> 
> > > In addition, ccan has a modular nature that makes sense for embedded
> > > projects but not really for Samba, since we're just building it as a
> > > library anyway. The library can't be public since it doesn't have a
> > > public ABI (modules can be excluded). Exporting each of the ccan
> > > modules as a shared library doesn't really make sense, as they're so
> > > small.
> > Why don't we just roll ccan into samba-util?  Either move ccan/ under
> > util, or leave it where it is and just make samba-util pull in ccan.
> That seems reasonable. We would have to make sure samba-util builds
> without the rest of Samba so it can be a dependency of tdb, but that's
> doable (can we name it libtutil?).

I am really oppoe to make samba-util a dependency for tdb, samba util is
in NO shape to become a real public library. Let's not do the worst
thing we could please.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>