search

maxtaco / coffee-script

IcedCoffeeScript
http://maxtaco.github.com/coffee-script
MIT License
727 stars 58 forks source link

Fix #155 (RequireJS bug) #156

Closed paigeruten closed 9 years ago

paigeruten commented 9 years ago

I figured out what the root.CoffeeScript = CoffeeScript line of code that I was talking about in #155 does. It's used to make the 'test:browser' task (at the very bottom of Cakefile) work, right? So I guess the solution is to test whether CoffeeScript is defined before running that line of code, since with AMD systems like RequireJS the browserify code will never set the CoffeeScript global, causing the ReferenceError to occur on that line.

maxtaco commented 9 years ago

Fix live in 1.8.0-e

maxtaco commented 9 years ago

Thanks for the PR