Constellation / ibrik

CoffeeScript meets Istanbul - code coverage tool for CoffeeScript
BSD 2-Clause "Simplified" License
79 stars 32 forks source link

compile with CoffeeScriptRedux and don't distribute compiled output #2

Closed michaelficarra closed 11 years ago

michaelficarra commented 11 years ago

CoffeeScriptRedux is a dev dependency, but it looks like the library was being compiled with CoffeeScript 1.4.0. You should add a Makefile with a task for compiling /src/* to /lib with CoffeeScriptRedux. Also, you should add /lib to a .gitignore and require the library to be built before being used.

Constellation commented 11 years ago

I've just created tools/install.mk and got error by executing make -f tools/install.mk. instruction is here,

make -f tools/install.mk clean
make -f tools/install.mk
./node_modules/.bin/coffee -j < "src/command.coffee" > "lib/command.js"
./node_modules/.bin/coffee -j < "src/cover.coffee" > "lib/cover.js"
./node_modules/.bin/coffee -j < "src/hook.coffee" > "lib/hook.js"
./node_modules/.bin/coffee -j < "src/ibrik.coffee" > "lib/ibrik.js"
./node_modules/.bin/coffee -j < "src/instrumenter.coffee" > "lib/instrumenter.js"
Syntax error on line 116, column 25: unexpected 'w' (\u0077)
113 :                         when 'BlockStatement'
114 :                             start: node.body[0].loc.start
115 :                             end: node.body[node.body.length - 1].loc.end
116 :                         when 'VariableDeclarator'
^^^ :~~~~~~~~~~~~~~~~~~~~~~~~^
117 :                             if node?.init?.loc?
118 :                                 start: node.id.loc.start
119 :                                 end: node.init.loc.end
make: *** [lib/instrumenter.js] Error 1

Is it bug?

Constellation commented 11 years ago

ref michaelficarra/CoffeeScriptRedux#122

Constellation commented 11 years ago

Now ibrik is compiled at npm install phase with CoffeeScriptRedux compiler. 48efdc2a6b5831a18b7a91623cdc37eec02ba5a9