Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Bugs with latest node (0.6.6) #5

Merged
merged 1 commit into from Dec 30, 2011

Conversation

Projects
None yet
2 participants
Contributor

rickihastings commented Dec 29, 2011

       .replace(new RegExp(':\\s*' + et, 'gm'), '{ %>')
       ^

SyntaxError: Invalid flags supplied to RegExp constructor 'g,m'
at new RegExp (unknown source)
at Object.render (/home/../node_modules/parrot/index.js:156:12)

Line 156 and the following 6 lines have been changed from ["g", "m"] to "gm", not sure why this has been changed in node but it now works.

    Script.runInNewContext(data, config.sandbox);
    ^

TypeError: Cannot call method 'runInNewContext' of undefined
at Object.render (/home/../node_modules/parrot/index.js:165:9)

Script = process.binding('evals').Script has been changed to Script = require('vm') where 'runInNewContext' is now located.

Fixed bugs with 'runInNewContext' being relocated, and a bug where re…
…gex expressions like ['m', 'g'] were invalid in new RegExp()

ollym pushed a commit that referenced this pull request Dec 30, 2011

Merge pull request #5 from n0valyfe/master
Bugs with latest node (0.6.6)

@ollym ollym merged commit e7c4a20 into ollym:master Dec 30, 2011

@e2b e2b referenced this pull request Jan 25, 2013

Closed

Several Issues #6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment