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

JSON.pm required #1

Open
werg opened this issue Nov 14, 2014 · 1 comment
Open

JSON.pm required #1

werg opened this issue Nov 14, 2014 · 1 comment

Comments

@werg
Copy link

werg commented Nov 14, 2014

Hey,
the current version seemingly requires the(?) JSON lib for Perl.
I know you mentioned you wanted to make a version without that dependency, but irregardless of that i think it would be nice for Perl neophytes (or longlong-time apostates) who are used to the likes of maven, pip, npm to tell them something like "you can just get this from your package manager", in the readme or the help.

@spencertipping
Copy link
Owner

Thanks for reporting this! nfu was indeed hard-requiring JSON, which is quite unfortunate. I changed this to a soft-require with better failure logic, and it also tries to use JSON::PP if it can't find JSON proper. Failing either one of those it should go on its way with no JSON support (so json_encode and json_decode will throw errors).

Can you give the latest version a try without a JSON library installed and see if it works?

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

No branches or pull requests

2 participants