atlas-engineer / nactivitypub

ActivityPub & ActivityStreams support library for Common Lisp.
BSD 3-Clause "New" or "Revised" License
11 stars 1 forks source link

Fix dependency on njson/core #1

Closed Ambrevar closed 1 year ago

Ambrevar commented 1 year ago

This fails to load with a recent njson because njson/core was removed.

nactivitypub.asd says:

  ;; FIXME: Depending on njson/core makes us lighter on dependencies
  ;; users of the library don't need, but forces us to load the actual
  ;; NJSON back-end, like njson/cl-json, elsewhere. Is that a
  ;; reasonable trade-off?

@aartaka ?

aartaka commented 1 year ago

Yes, simply replace it with njson!

Ambrevar commented 1 year ago

Done in 03505467029134cb8912c63df0840e3c98bd24a4. Is that good enough?

aartaka commented 1 year ago

Done in 0350546. Is that good enough?

Yes, just fine!