search

pixie-lang / pixie

A small, fast, native lisp with "magical" powers
GNU General Public License v3.0
2.35k stars 124 forks source link

Examples are broken #452

Closed ghost closed 8 years ago

ghost commented 8 years ago

Hi,

with a fresh checkout and build, I can run almost none of the examples (assuming ./pixi-vm examples/foo.pxi is the way to go here)

mu-kanren.pxi -> runtimeException: :pixie.stdlib/AssertionException Var sequence is undefined mandelbrot.pxi -> RuntimeException: :pixie.stdlib/EOFWhileReadingException Unexpected EOF while reading tty-io-test.pxi -> RuntimeException: :pixie.io/Exception Expected a LineReader, UTF8InputStream, or BufferedInputStream

I'd like to use pixie for shell scripting, so the tty example is of particular importance to me.

thomasmulvaney commented 8 years ago

I have addressed the tty-io-test.pxi issue in #453 Feel free to add a sequence function to stdlib as it is missing pull requests are very welcome :smile:

ghost commented 8 years ago

tty-io-test works now, thanks! Pixie is phenomenal, just wrote a few scripts and it felt great. I'd rather not promise any contributions, but if I find time I'll definitely give it a try.

jmglov commented 8 years ago

Can we close this now?

thomasmulvaney commented 8 years ago

I suspect you'll find #436 fixes the mandelbrot issue.

thomasmulvaney commented 8 years ago

I'd like to close this issue but it seems like some other examples are broken. The mu-kanren example fails because sequence is not defined. The gen-docs example fails because it depends on hiccup. I think we should remove the gen-docs example or at least make it work without hiccup. I don't think any examples should depend on external libraries.

It looks like sequence was removed way back in d22b9c3deb39d5f2dd11c20a93c21c07227d97b9. I've created issue #502 for adding a sequence function back in.