move libs/json/include/json.hpp to libs/json/include/nlohmann/json.hpp

[2bbb]

when other libraries has json.hpp, #include "json.hpp" can't load nlohmann::json.

i think moving include/json.hpp to include/nlohmann/json.hpp and replace
#include "json.hpp"
to
#include "nlohmann/json.hpp"
or
#include <nlohmann/json.hpp>

for reference:
#include <json.hpp> works sometime.
but i think this is not complete solution, maybe...

[oxillo]

👍

MSYS2 is already using #include <nlohmann/json.hpp> in ofJson.h

#if !defined(TARGET_MINGW)
	#include <json.hpp>
#else
	#include <nlohmann/json.hpp> // MSYS2 : use of system-installed include
#endif

Also, examples from nlomann site are using #include <nlohmann/json.hpp>. So, I think this is the right move.

[2bbb]

this fix need to use apothecary...?
if so, i don't know apothecary enough, please fix anyone! ☺️

[danoli3]

Can do

[danoli3]

openframeworks/apothecary#429 done

Should work automatically as include location doesn't change

[dimitre]

Nice! should this should be changed also?
in ofJson.h

#if !defined(TARGET_MINGW)
	#include <json.hpp>
#else
	#include <nlohmann/json.hpp> // MSYS2 : use of system-installed include
#endif

[danoli3]

Yes if someone could make the PR for that once the releases deployed to merge

…

On Mon, 19 Aug 2024 at 03:06, Dimitre ***@***.***> wrote: Nice! should this should be changed also? in ofJson.h #if !defined(TARGET_MINGW) #include <json.hpp> #else #include <nlohmann/json.hpp> // MSYS2 : use of system-installed include #endif — Reply to this email directly, view it on GitHub <#8082 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAGK2HGL25MUCXVMA4NAH2DZSDH7RAVCNFSM6AAAAABMWH42ROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGMZDOOBVGQ> . You are receiving this because you were assigned.Message ID: ***@***.***>

[danoli3]

Nice I think this changed exposed a few major automation test problems! XD