jsoncpp-devel Mailing List for json-cpp

Improved patch, also fixes missing assertions header. Without it there
are lots of errors about JSON_FAIL_MESSAGE and others being undefined.
Also, I'm manually specifying -DJSON_IS_AMALGAMATION as part of my
build, but it sounds like I'm not supposed to need to do that?

-Joe G.

On Sat, Jun 11, 2011 at 1:54 PM, Joseph Garvin
<jos...@gm...> wrote:
> amalgamate.py currently doesn't work on Linux because some paths are
> done using backslashes instead of forward slashes. I think the Right
> Way (TM) to do this is to use os.path, but here's a lazy patch that
> just swaps the slashes.
>
> -Joe G.
>

amalgamate.py currently doesn't work on Linux because some paths are
done using backslashes instead of forward slashes. I think the Right
Way (TM) to do this is to use os.path, but here's a lazy patch that
just swaps the slashes.

-Joe G.

Json-C (http://oss.metaparadigm.com/json-c/) also installs file
/usr/include/json/json.c. Json-C package is present at least in
some Linux distributions (it is in Fedora)

It would be nice to avoid conflict with JsonCpp header file
include/json/json.h. One way would be to move all headers to
include/jsoncpp/ from include/json/. Unfortunatelly this
change would break already existing software which uses
JsonCpp.

Perhaps it would be better to put headers in new place and
add forwarding headers in old place. For example:

include/json/json.h could contain

#ifndef __JSONCPP_JSON_JSON_H__
#define __JSONCPP_JSON_JSON_H__
#warning Header file json/json.h is deprecated. Use jsoncpp/json.h instead
#include <jsoncpp/json.h>
#endif

and similarly for other headers. After some time it would then be possible
to remove headers at old place.

Andris