Re: [OSSIM] JsonCPP re-amalgamated in OSSIM

[Potts, G. <Gar...@ma...>]

Hello Oscar:

I would allow to enable internal and external supported builds.   Also,  not sure if there would be symbol conflicts if someone is building to jsoncpp for another library and linking to ossim.


Take care

Garrett


From: Oscar Kramer <osc...@ya...>
Reply-To: Oscar Kramer <osc...@ya...>
Date: Wednesday, September 18, 2019 at 10:36 AM
To: "Potts, Garrett" <Gar...@ma...>, David Burken <db...@pr...>
Cc: OSSIM Developers <oss...@li...>
Subject: JsonCPP re-amalgamated in OSSIM

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

All,

I re-added the amalgamated form of jsoncpp to ossim and related repos. That removes the dependency on an external library that wasn't available as a simple "apt install" for ubuntu systems, and likely was a pain for other systems as well. That necessitated building the library from source and adding the infrastructure to find the includes and lib -- an unnecessary complication. We used to use the amalgamated form (i.e., the .h and .cpp included as part of the OSSIM build) until someone complained of licensing. I see that JsonCPP now shares the same MIT license as OSSIM, and specifically states:


Copyright (c) 2007-2010 Baptiste Lepilleur and The JsonCpp Authors

Permission is hereby granted, free of charge, to any person
obtaining a copy of this software and associated documentation
files (the "Software"), to deal in the Software without
restriction, including without limitation the rights to use, copy,
modify, merge, publish, distribute, sublicense, and/or sell copies
of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be
included in all copies or substantial portions of the Software.
So that should no longer be a problem. JsonCPP provides the tools to create this amalgamated form, so this follows the intent of the authors.

Oscar