jzon icon indicating copy to clipboard operation
jzon copied to clipboard

[FR] Add to `quicklisp`

Open NightMachinery opened this issue 3 years ago • 13 comments

NightMachinery avatar Jun 19 '21 12:06 NightMachinery

Hi @NightMachinary . I appreciate the support, unfortunately I'd like to avoid putting this on Quicklisp until it's more complete. At a minimum I wish to have:

  1. A roundtripping reader/writer (output JSON exactly as it was read)
  2. Object coercion mechanism for reading (eg read JSON as a CL class/structure)

Until these are done, I don't want to pollute QL with yet another JSON library.

Zulu-Inuoe avatar Jun 20 '21 13:06 Zulu-Inuoe

+1 for adding to Quicklisp. IMHO, jzon is already the best JSON library for rendering JSON views in web app. So, I'd like to switch from Jonathan in my libraries, but can't because jzon is not in Quicklisp.

fukamachi avatar Nov 26 '21 06:11 fukamachi

@fukamachi Thanks for the support. Unfortunately I haven't been doing much programming in my free time as of late. But if you think the library is useful as-is, then it's fair to add it to quicklisp, and I can add my other desired features over time as I can.

I just don't want to continue to contribute to the 100 JSON library meme on Quicklisp unless I offer some substantial benefit

Zulu-Inuoe avatar Nov 30 '21 00:11 Zulu-Inuoe

@Zulu-Inuoe I think that jzon is a new generation JSON parser taking into account advantages and disadvantages of previous generation of parsers. And it has full coverage of JSON standard verified by third-party test suite. This is already good reasons for releasing v1 on quicklisp.

Valera avatar Nov 30 '21 11:11 Valera

Yes, can you please add this to quicklisp, the pretty print feature is exceptionally useful :)

K1D77A avatar Dec 30 '21 18:12 K1D77A

Throwing my hat onto the pile. I'd like to start using this.

Shinmera avatar Feb 17 '22 15:02 Shinmera

Pretty please?

phoe avatar Apr 07 '22 16:04 phoe

Oh my gosh. Let me see what I can do this weekend and I'll add v1 on Monday no matter what

Zulu-Inuoe avatar Apr 07 '22 19:04 Zulu-Inuoe

haha jk did I say that weekend? I meant this one.

In all seriousness I've been doing some work and thinking over the week and will definitely have something solid here soon. Thanks all

Zulu-Inuoe avatar Apr 15 '22 15:04 Zulu-Inuoe

fbn7krvnq8381

phoe avatar Jul 01 '22 17:07 phoe

fbn7krvnq8381

Unfortunately I am a busy tortoise :(

Zulu-Inuoe avatar Jul 06 '22 13:07 Zulu-Inuoe

Do you need any support with that - as in, what would happen ~~to me~~ if I did things the naïve way and made a quicklisp-projects issue pointing to the current master branch?

phoe avatar Jul 06 '22 15:07 phoe

I honestly don't remember the status of what things are WIP. I believe there's still a couple of issues with stringify, including the floating point printer, the string replacer, and the auto-coercion stuff in general. As before it's ultimately: I still don't know if it offers anything truly above and beyond the other json libs on QL already.

The other issue is that if we make available a beta version as-is, it'll be hard to fix things without breaking users - the auto-coercion stuff especially scares me.

Zulu-Inuoe avatar Jul 06 '22 15:07 Zulu-Inuoe

Alright this is actually finally happening. All my remaining stuff for v1 are resolved and I've decided what to leave out for a v2.

Code-wise, the true final final stuff is whether or not I keep the alist/plist heuristics in write-value which I am awaiting review. The other changes are things I can do without breaking backwards-compat, like updating docs and optimizing my string-on-string parser

Zulu-Inuoe avatar Feb 10 '23 16:02 Zulu-Inuoe

https://github.com/quicklisp/quicklisp-projects/issues/2256

Zulu-Inuoe avatar Feb 14 '23 03:02 Zulu-Inuoe

Great, thank you!

fukamachi avatar Feb 14 '23 12:02 fukamachi

Added and available; huge thanks! I think this issue can now be closed.

phoe avatar Feb 15 '23 18:02 phoe

Thanks for the heads up, @phoe

Zulu-Inuoe avatar Feb 15 '23 19:02 Zulu-Inuoe