search

edgecase / dieter

Asset pipeline ring middleware
134 stars 22 forks source link

V8 changes: in particular, fully V8 enabled coffeescript asset compiler #25

Closed ivanstojic closed 12 years ago

ivanstojic commented 12 years ago

With these changes, the V8 asset compiler is now fully JS-engine aware, uses the new clj-v8 wrappers, recognizes and reports errors and everything. Note: only the coffeescript asset compiler has been ported over. Some work still left to be done on hamlcoffee and less, along with testing.

ivanstojic commented 12 years ago

Added a commit to close Dieter's issue #10

pbiggar commented 12 years ago

Github won't auto merge this. do you need me to merge it manually?

ivanstojic commented 12 years ago

Sure, merge manually and then I'll re-fork my own repo direct from edgecase/dieter instead of the long way around.

pbiggar commented 12 years ago

I fixed a few merge conflicts, then go:

$ lein test Copying 34 files to /Users/pbiggar/work/dotbuild/dieter/dieter-core/lib Copying 2 files to /Users/pbiggar/work/dotbuild/dieter/dieter-core/lib/dev

Testing dieter.test.asset.coffeescript Invalid memory access of location 0x0 rip=0x7fff8ef974f0

ivanstojic commented 12 years ago

I'm not sure which OS you are on - I currently only have 64bit OS X compiled in the repo up to the latest version. In any case, before lein test, I'd suggest "rm -rf native/; lein deps; lein clean"

ivanstojic commented 12 years ago

Expiration now works as it should.

ivanstojic commented 12 years ago

Superseded by the new pull request...