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

"Invalid regular expression" error in minified script #255

Closed
mach3 opened this Issue Aug 21, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@mach3

mach3 commented Aug 21, 2013

When importing minified scripts in "dist/", only which contains "head.css3.min.js", it's aborted with the message.

Uncaught SyntaxError: Invalid regular expression: /(url\s_(._?){3})/: Unmatched ')'

Maybe, compressor couldn't parse the escaped "(", and add needless ")" ?

The one which minified with grunt-contrib-uglify (UglifyJS2), it goes well.

@mach3

This comment has been minimized.

Show comment
Hide comment
@mach3

mach3 Aug 21, 2013

I have read the change in 799a52baf719d8342c16b4abc4c215dca82d0a35
It isn't to blame for compressor.

But it works well in Safari without ")", and doesn't work with ")".

mach3 commented Aug 21, 2013

I have read the change in 799a52baf719d8342c16b4abc4c215dca82d0a35
It isn't to blame for compressor.

But it works well in Safari without ")", and doesn't work with ")".

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Aug 23, 2013

Contributor

a patch for this has been submitted, but my home pc is broken, so i can't clone & sync the changes back to the cdn, nuget, and download packages right now :(

Contributor

ghost commented Aug 23, 2013

a patch for this has been submitted, but my home pc is broken, so i can't clone & sync the changes back to the cdn, nuget, and download packages right now :(

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Sep 13, 2013

Contributor

This has been fixed as commented in #258

Changes have been pushed to src/dist and .map files are now available for all js files ..cdn has not been updated yet

Contributor

ghost commented Sep 13, 2013

This has been fixed as commented in #258

Changes have been pushed to src/dist and .map files are now available for all js files ..cdn has not been updated yet

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