Zulu-Inuoe/jzon

Can't load it into CLISP

Closed this issue · 6 comments

catap commented 
:info:build          #<STANDARD-CLASS FUNDAMENTAL-CHARACTER-INPUT-STREAM>) are
:info:build          inconsistent
:info:build WARNING: (class-precedence-list #<STANDARD-CLASS FLEXI-IO-STREAM>) and
:info:build          (class-precedence-list #<STANDARD-CLASS FLEXI-OUTPUT-STREAM>) are
:info:build          inconsistent
:info:build WARNING: in CHECK in lines 53..59 : variable READER is assigned but not read
:info:build WARNING: in #:|69 71 (DEFMETHOD FORMAT-DOCUMENTATION (# TYPE VAR ...)
:info:build          ...)-18-1-1| in lines 69..71 : variable TYPE is not used.
:info:build          Misspelled or missing IGNORE declaration?
:info:build WARNING: in #:|69 71 (DEFMETHOD FORMAT-DOCUMENTATION (# TYPE VAR ...)
:info:build          ...)-18-1-1| in lines 69..71 : variable VAR is not used.
:info:build          Misspelled or missing IGNORE declaration?
:info:build WARNING: in WITH-FLOAT-TRAPS-MASKED in lines 178..262 : FIND was called with 1
:info:build          arguments, but it requires at least 2 arguments.
:info:build WARNING: in WITH-FLOAT-TRAPS-MASKED in lines 178..262 : variable TRAPS is not
:info:build          used.
:info:build          Misspelled or missing IGNORE declaration?
:info:build *** - DEFPACKAGE COM.INUOE.JZON/EISEL-LEMIRE: unknown option :LOCAL-NICKNAMES
:info:build Command failed: /opt/local/bin/clisp --quiet --quiet -x '(require "asdf")' -x '(setf asdf:*central-registry* (list* (quote *default-pathname-defaults*) #p"/opt/local/var/macports/build/_Users_catap_src_macports-ports_lisp_cl-com.inuoe.jzon/cl-com.inuoe.jzon/work/build/system/" #p"/opt/local/share/common-lisp/system/" asdf:*central-registry*))' -x '(asdf:operate (quote asdf:build-op) (quote com.inuoe.jzon))' 2>&1
:info:build Exit code: 1

I'm using clisp which is build from https://gitlab.com/gnu-clisp/clisp/-/commit/66924971790e4cbee3d58f36e530caa0ad568e5f

SBCL, ECL, ABCL and CCL works fine

Zulu-Inuoe commented

Hi, please see https://gitlab.com/gnu-clisp/clisp/-/merge_requests/3
Looks like there's been a PR open to add PLN's to clisp for a while that has not been merged

Zulu-Inuoe commented

Looking at my usage of pln's, I think I'd be OK with forgoing them, though I do wish clisp would accept that PR already since they are much nicer to use compared to alternatives

  • 👍1
catap commented

Yeah, but clips aren’t well supported these days :(

and let me be frankly it usually has it’s own point of view 🤓

hraban commented

100% agreed, CLISP not supporting PLN is a constant pain in the neck.

Zulu-Inuoe commented

See 01da2ed and 6652ed9

I'm still not sold on them as it makes writing the unit tests especially onerous. On the main project the changes are minimal.
I'll let this sit and may revert them later. I'm curious to hear why people are using clisp for anyways

hraban commented

Honestly at this point I only use Clisp to compile SBCL. I applaud your commitment to Clisp but I have adopted the philosophy that PLNs are too useful to be held back by Clisp.

  • 👍1