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.

Sign up for GitHub

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

Jump to bottom

Upgrade lib/jsmn #117

Closed
edsiper opened this issue Oct 11, 2016 · 2 comments
Closed

Upgrade lib/jsmn #117

edsiper opened this issue Oct 11, 2016 · 2 comments
Assignees
@edsiper
Labels
enhancement

Comments

@edsiper
Copy link
Member

edsiper commented Oct 11, 2016

The jsmn library used had a bug and Fluent Bit code have a workaround for it. As soon as jsmn becomes fixed we should upgrade:

zserge/jsmn#81
@edsiper edsiper self-assigned this Oct 11, 2016
@edsiper edsiper mentioned this issue Oct 11, 2016
Closed
@pt300
Copy link

pt300 commented Oct 12, 2016

That pull request got accepted. You can safely use current version from repo.

@edsiper
Copy link
Member Author

edsiper commented Oct 12, 2016

thanks

@edsiper edsiper closed this as completed in d1c9384 Nov 1, 2016
edsiper added a commit that referenced this issue Nov 7, 2016
@edsiper
lib: jsmn: upgrade to latest version '1682c32' (fix #117)
ea0d106 
Signed-off-by: Eduardo Silva <eduardo@treasure-data.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edsiper edsiper

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@edsiper @pt300