Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected reserved word in strict mode error (for yield) on Android 4.0 #12657

Closed
fivetanley opened this issue Nov 27, 2015 · 9 comments
Closed
Labels

Comments

@fivetanley
Copy link
Member

The line on HTMLBars here: https://github.com/tildeio/htmlbars/blob/72c7350805e4d7d44e2e88f2377915960cd3513a/packages/htmlbars-runtime/lib/hooks.js#L107

causes an error of Unexpected reserved word in strict mode error on Android 4.0. The solution is probably to add 'yield' to https://github.com/stefanpenner/es3-safe-recast.

@fivetanley fivetanley added the Bug label Nov 27, 2015
@fivetanley
Copy link
Member Author

Another example on another project: angular/angular.js#7969

@rwjblue
Copy link
Member

rwjblue commented Nov 27, 2015

Dupe of tildeio/htmlbars#420, which would be fixed by tildeio/htmlbars#421.

@fivetanley
Copy link
Member Author

I was getting Saucelabs set up for Ember Data which made me notice this. Glad it's been fixed! I think we'll need a point release for the changes to apply? I'm also concerned about previous versions of ember.

@rwjblue
Copy link
Member

rwjblue commented Nov 27, 2015 via email

@mmun
Copy link
Member

mmun commented Nov 27, 2015

@stefanpenner
Copy link
Member

@mmun I don't believe old builds always used babel.

@fivetanley
Copy link
Member Author

FWIW this is now in es3-safe-recast thanks to @stefanpenner, so it may be as simple as bumping the version + doing rebuild.

@rwjblue
Copy link
Member

rwjblue commented Nov 27, 2015 via email

@Serabe
Copy link
Member

Serabe commented May 9, 2016

Closing this since 1.13 is no longer LTSish and the fix was in htmlbars 0.14.11 (the version in Ember 2.3).

Please, reopen it if something is still missing.

@Serabe Serabe closed this as completed May 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants