[PATCH] rearrange code in intl sub directory

[William Hubbs]

On Tue, Aug 10, 2010 at 01:01:36PM -0500, Christopher Brannon wrote:
> > From: William Hubbs <w.d.hubbs at gmail.com>
> > 
> > This moves intl/ to src/common and turns that directory into a library
> > of common routines
> > 
> > This means that we can place other routines in this library also which
> > are shared between the server and the modules.
> > 
> > Also, this creates an include/ directory at the same level as the src/
> 
> Hello William,
> Here are my comments on the most recent version of your patch.
> As it stands, this patch causes "make dist" to fail.
> The problem is with the following line from src/common/Makefile.am:
> +libcommon_la_SOURCES = fdsetconv.c fdsetconv.h
> where fdsetconv.h is actually in the include/ directory.
> Do you even need to mention that file here?
> The patch also needs to be rebased against master.
> There are two small conflicts, one in configure.in, the other in
> src/Makefile.am.
> Aside from those two issues, it looks good.

Hey Chris,

ok, I'll rework the patch and submit it to the list again.

Thanks,

William

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: 
<http://lists.freebsoft.org/pipermail/speechd/attachments/20100810/c6633fbb/attachment-0001.pgp>