Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bring back exiv2json (or remove the remnants from the repo) #2292

Open
jpicht opened this issue Jul 25, 2022 · 3 comments
Open

bring back exiv2json (or remove the remnants from the repo) #2292

jpicht opened this issue Jul 25, 2022 · 3 comments
Labels
request feature request or any other kind of wish

Comments

@jpicht
Copy link

jpicht commented Jul 25, 2022

Is your feature request related to a problem?

The exiv2json sample has been removed by #2183, but is still referenced in the docs, vs2019 project, doxygen, and tests/suite.conf. As I'm using it (indirectly) in some scripts I'd like for it to come back.

It has been part of the exiv2 binary package for my linux distribution for quite a long time, so I never realized it wasn't in fact supported, but just a sample.

Describe the solution you would like

I'd like the merge be reverted and the problem with the aging JSON library be solved by porting to a newer json-library like (for example) nlohmann/json. As I'm familiar with that library, I would volunteer to do the work, if the MIT license is acceptable for you to include that as a dependency.

Describe alternatives you have considered

Providing a compatible tool, probably based on the last version, as a third-party package.

@jpicht jpicht added the request feature request or any other kind of wish label Jul 25, 2022
@MuhammadSYahyaS
Copy link

I found a simple workaround for this using Boost.PropertyTree v1.74.0 or later. Check https://github.com/MuhammadSYahyaS/exiv2json for the code.

@piponazo
Copy link
Collaborator

Hi @jpicht , thanks for your message.

We would be happy to accept any examples that would be beneficial for Exiv2 users. Although I am not an expert in terms of licensing, I think the MIT one would conflict with the GPLv2 license of Exiv2.

If you would propose an addition which does not conflict with the GPLv2, we would be happy to incorporate it to the project.

@jpicht
Copy link
Author

jpicht commented Dec 5, 2022

Hi @piponazo,

I'm not a legal expert by far, but as far as I can tell, the MIT license is one of the most compatible licenses out there. In my opinion: it would not clash with GPLv2, the authors of the wikipedia page for that license seem to agree.

Edit: gnu.org also agrees, though they call the license "expat license".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request feature request or any other kind of wish
Projects
None yet
Development

No branches or pull requests

3 participants