The run-bad-header.c test in the rfc822 module is actually testing handling
of one specific sort of bad header - a header field which contains no
colon character.  Rename the test and its internal variables to better
reflect this.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

The check_header() test function will be useful to more than one testcase
so this patch moves it into the common helper.c file.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

Add a simple full-blown example program for the rfc822 module.
'headernames' simply dumps the header field name for every header field in
each message file given on the command line to stdout.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

Approved by all contributors:
https://lists.ozlabs.org/pipermail/ccan/2012-September/000490.html
https://lists.ozlabs.org/pipermail/ccan/2012-September/000488.html
https://lists.ozlabs.org/pipermail/ccan/2012-September/000489.htmlSigned-off-by: Rusty Russell <rusty@rustcorp.com.au>

And enhance the documentation a little.

Times can be invalid: non-normalized, or negative.  

It's a superset of struct timeval.  It's also a bit trickier, since we
need -lrt on Linux to use clock_gettime(), and also we can't just cram the
time into a u64 for multiply and divide.

If we do have it, and we redefine it, compilation breaks.

In particular, HAVE_CLOCK_GETTIME_IN_LIBRT.

We intuit the dependencies of embedded examples by looking for #include.
But these extra dependencies may require extra libraries, so we need to
add those, too.

If a dependent module needs -l options, we don't notice.  Fix that, now
we have one.

We let the get_deps() caller hand us the compiled _info filename, but what
about recursive dependencies?  These we re-generate every time.

So fix this: hand a generator callback to get_deps(), expose the one
which simply compiles it, and add a ccanlint one which looks up the
manifest to see if we have one already.

Before:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
31

After:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
17

This fixes some format string related bugs that show up on 64-bit builds.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

For convenience, check in a copy of the full text of RFC5322 (the latest
revision of RFC822).
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

In particular, don't complain on conditional lines.  That's less thorough,
but removes false negatives we have now.

next_header_parse() had a bug which would mis-parse an extra header if the
body of the message began with a linear whitespace character (space or
tab).  This patch fixes it and adds a suitable testcase.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

It's only used for testing, but it's LGPLv3 (vs LGPLv2).  But unless we
suppress it, ccanlint exits with status 1 and my sanity checks stop me from
uploading.

Fixes this error:
	/hash.h:304:13: error: two or more data types in
	declaration specifiers

Seems it's not available on Solaris.

This is where you see the genius of the originals: you can't implement them
without horrible hacks to get the program name :(

Seems like Solaris doesn't have err.h, as discovered by Samba.

This adds a "waf" buildtype to create-ccan-tree. When passed it
generates a basic wscript for building everything into a libccan.a

It doesn't support running the generator, so you need to provide
config.h
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>

Add a new build type to create an appropriate Makefile.am for the
newly-created ccan tree.
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>

Replace the --exclude-configurator argument with a
single --build-type=TYPE argument, allowing future expansion of build
system types (and allowing source-tree-only creation). We currently
support:

 * 'make' (just the Makefile, equivalent to --exclude-configurator)
 * 'make+config' (current default)
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>

Rather than explicitly excluding the tests, just copy the source files
by default. Add an option (--copy-all) to include all of the non-source
stuff in each module.
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>

Commit a77cc282 causes tools/create-ccan-tree to fail, as it can't build
tools/ccan_depends (via `make tools/ccan_depends`) without a
configuration header present.

This change adds a dependency on config.h, so that the configurator
kicks-in prior to building ccan_depends.o.

Other tools don't need this dependency, as tools/ccan_depends is built
during the Makefile 'include' process.
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

We create and upload an example for the web anyway, if people really need it.

Add a script to create a ccan source tree for inclusion into projects'
distributed code.

create-ccan-tree takes a list of ccan modules and copies them, plus any
dependent modules, into a target directory. Enough build infrastructure
is added to build libccan.a from the ccan modules.

Typical usage for my project that uses talloc and read-write-all:

  lib/ccan.git/tools/create-ccan-tree --exclude-tests \
		lib/ccan/ talloc read-write-all

- this uses a git submodule in $PWDlib/ccan.git to create a source tree
in $PWD/lib/ccan, containing code for talloc and read-write-all.
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>

Solaris CC seems to support typeof, but not __builtin_types_compatible_p.

cast_const() et. al. are supposed to be a constant expression, so you can do things like:
	static char *p = cast_const(char *, (const char *)"hello");

Unfortunately, a cast to intptr_t and arithmetic makes suncc reject it as
a constant expression.  We need the cast, because (1) the expression could be
a void *, so we can't just add to it, and (2) gcc complains with -Wcast-qual
without it.

So instead of adding BUILD_BUG_OR_ZERO, we use a ? :, which keeps everyone happy.

That's a gcc extension.