ruricolist / cloture

Clojure in Common Lisp
377 stars 7 forks source link

change cl-arrows dependency to arrows #9

Closed travv0 closed 3 years ago

travv0 commented 3 years ago

cl-arrows has been removed from QuickLisp due to licensing issues, but I believe arrows is a good alternative that i think has very similar semantics. I wasn't able to figure out how to run the tests (I got a read error when compiling them), but I'd be surprised if this change affected anything's behavior.

travv0 commented 3 years ago

Actually, just ran into this read error when messing around in the REPL so I plan on looking at that closer soon as well. I don't believe it relates to the arrows package, though.

svantevonerichsen6906 commented 3 years ago

Arrows is a drop-in replacement for cl-arrows, yes. There are no funny reader things in there, so I don't think there should be such issues.

Tests should just work, e. g. through (ql:quickload "arrows/test"), then (asdf:test-system "arrows").

travv0 commented 3 years ago

Sorry, I meant the cloture tests, not the arrows tests.

travv0 commented 3 years ago

Ignore what I said about the read error, not able to reproduce on another machine.