Skip to content

New ArduinoJSON beta doesn't compile with ESP8266 #14

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

Closed
markbeee opened this issue Jun 23, 2018 · 7 comments · Fixed by #99
Closed

New ArduinoJSON beta doesn't compile with ESP8266 #14

markbeee opened this issue Jun 23, 2018 · 7 comments · Fixed by #99
Assignees

Comments

@markbeee
Copy link

From version 6.0.0 (beta) Arduino JSON library doesn't compile with webthing-arduino/ ESP8266.

Quote from [https://github.com/bblanchon/ArduinoJson/issues/760]
"Indeed ArduinoJson 6 replaced the concept of JsonBuffer with the concept of JsonDocument.
To fix this error, you need either:
Downgrade ArduinoJson to 5.x (recommended as v6 is still in beta)
Upgrade your program to work with ArduinoJson 6"

@hobinjk
Copy link
Contributor

hobinjk commented Jul 9, 2018

According to the ArduinoJson website version 6 will be in beta for the rest of 2018 so I think the correct option is to require the use of version 5.

@rzr
Copy link
Collaborator

rzr commented Aug 11, 2018

Please see if this can be tweaked in make files I just added.

@novski
Copy link
Contributor

novski commented Jan 18, 2019

Search tag: error: 'StaticJsonBuffer' was not declared in this scope

In my case this was the errormessage. I first searched for this error but then found this issue so i add this line in this issue for later people searching for the errormessage.

@willdurand
Copy link

@hobinjk the work to upgrade from ArduinoJson 5 to 6 does not look huge: https://arduinojson.org/v6/doc/upgrade/. Would it make sense to upgrade?

Supporting both versions would be doable, but would it make sense?

@matiasdelellis
Copy link

Hi, it's already stable, so IMHO is a good time to update this library.. 😃

Thanks for all.

@matiasdelellis
Copy link

@markbeee @rzr @willdurand @novski
Sorry to mention you with @ 😅

@hobinjk is working on this in the pull request #95
.... and needs to be tested. 😬

Thanks and regards,
Matias,

@rzr
Copy link
Collaborator

rzr commented Oct 21, 2019

If you update it please also try other sample apps built with top makefile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants