lantiga / ki

lisp + mori, sweet.js
ki-lang.org
MIT License
478 stars 28 forks source link

Install instructions #3

Closed berdario closed 10 years ago

berdario commented 10 years ago

I think that the instructions should add

npm install mori

otherwise, npm will resolve mori as a dependency of ki, but it won't be found by the require()

either that, or the example is changed to use

var mori = require('ki/node_modules/mori')

(or you make each ki statement generate a new require of mori, thus removing the burden of ki-users to deal with it, this shouldn't be a problem performance-wise)

lantiga commented 10 years ago

Thank you @berdario, I think I found a good solution for this and #2. I introduced a

ki require

statement that should be used before using ki, which expands into

var mori = require('ki/node_modules/mori')

This should also solve the gensym error you were getting in issue #2

I'll re-release now, feel free to test.

berdario commented 10 years ago

Thanks, works for me

lantiga commented 10 years ago

Great, thanks