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.

Sign up for GitHub

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

Jump to bottom

TypeError: Cannot assign to read only property 'warnings' of true #298

Closed
rauberdaniel opened this issue Feb 12, 2015 · 10 comments
Closed

TypeError: Cannot assign to read only property 'warnings' of true #298

rauberdaniel opened this issue Feb 12, 2015 · 10 comments
Labels
duplicate

Comments

@rauberdaniel
Copy link

I recently updated to node 0.12.0 and right now I’m getting the following error with grunt-contrib-uglify version 0.7.0:

Running "uglify:min" (uglify) task
[TypeError: Cannot assign to read only property 'warnings' of true]
>> Uglifying source static/js/main.js failed.
Warning: Uglification failed.
Cannot assign to read only property 'warnings' of true.
 Use --force to continue.

Aborted due to warnings.

Happens with any .js file, so it seems to be a bug either with nodejs v0.12.0 or with this uglify task. Any ideas?
@rauberdaniel
Copy link
Author

I guess I found the reason: Having set the option compress to true instead of {} (which is allowed by documentation) causes the issue.

@oleksiipysanko
Copy link

+1! You saved my day, thanks!

@shahata shahata mentioned this issue Feb 17, 2015
Merged
@XhmikosR
Copy link
Member

Duplicate of #287.

@Lyra2108 Lyra2108 mentioned this issue Feb 19, 2015
Merged
evaneus added a commit to corespring/corespring-container that referenced this issue Mar 3, 2015
@evaneus
workaround uglify js issue
0882fd9 
gruntjs/grunt-contrib-uglify#298
@espinielli espinielli mentioned this issue Mar 9, 2015
Closed
bbinet added a commit to bbinet/grafana that referenced this issue Mar 10, 2015
@bbinet
fix uglify TypeError
ae41fa3 
This patch fix the following uglify error:
TypeError: Cannot assign to read only property 'warnings' of true
Or we can upgrade grunt-contrib-uglify to latest v0.8.0 version which
should also fix the issue.

This issue has been reported on grunt-contrib-uglify bug tracker:
gruntjs/grunt-contrib-uglify#298
@bbinet bbinet mentioned this issue Mar 10, 2015
Merged
sh1989 pushed a commit to sh1989/angular-modularisation that referenced this issue Mar 13, 2015
Working around Uglify/node 0.12 issue
645408f 
compress: true breaks Node 0.1.2, as documented
gruntjs/grunt-contrib-uglify#298
@NorikDavtian
Copy link

👍 @rauberdaniel tnxx

@ahmednuaman
Copy link

So I had this problem too, but it was fixed by running git clean -xdf and npm install again.

andrezero pushed a commit to andrezero/grunt-jsglue that referenced this issue Apr 29, 2015
fix(compress): gruntjs/grunt-contrib-uglify#298
226498c
ryanholder added a commit to cityindex/kibana that referenced this issue Jun 5, 2015
@ryanholder
Change compress from true to empty object
4053577 
gruntjs/grunt-contrib-uglify#298
@4aficiona2
Copy link

replacing true with {} worked for me too. Thanks @rauberdaniel

@2013gang
Copy link

+1

@spiderpug spiderpug mentioned this issue Aug 26, 2015
Merged
@willthemoor
Copy link

Thank you @rauberdaniel!

@tomekwszelaki tomekwszelaki mentioned this issue Sep 4, 2015
Merged
@Deanzie
Copy link

Deanzie commented Sep 4, 2015

+1

JianJiao added a commit to JianJiao/portfolio that referenced this issue Sep 14, 2015
@JianJiao
fix: grunt uglify compress true error...
2c13fb3 
* gruntjs/grunt-contrib-uglify#298
* gruntjs/grunt-contrib-uglify#287
@contolini contolini mentioned this issue Jan 13, 2016
Merged
np added a commit to np/freeboard that referenced this issue Jan 21, 2016
@np
Use compress: {} instead of compress: true
6671250 
See gruntjs/grunt-contrib-uglify#298
r4j4h added a commit to r4j4h/crossfilter-ma that referenced this issue Jan 26, 2016
@r4j4h
Use compress: {} instead of compress: true
45b984a 
See gruntjs/grunt-contrib-uglify#298
@Oliv4945 Oliv4945 mentioned this issue Feb 4, 2016
Closed
@tonyjin tonyjin mentioned this issue Mar 1, 2016
Merged
@drasko drasko mentioned this issue Mar 11, 2016
Closed
@critmcdonald critmcdonald mentioned this issue Mar 15, 2016
Closed
@andres-soto andres-soto mentioned this issue Jun 3, 2016
Open
risq added a commit to risq/simplezoom that referenced this issue Jul 22, 2016
@risq
Fix uglifyjs options (gruntjs/grunt-contrib-uglify#298)
6ccfbd3
aschmoe added a commit to aschmoe/fontawesome-iconpicker that referenced this issue Aug 9, 2016
@aschmoe
updating how uglify compress: true is called, gruntjs/grunt-contrib-u…
456d364 
…glify#298
@aschmoe aschmoe mentioned this issue Aug 9, 2016
Merged
kelp404 added a commit to kelp404/typeahead.js that referenced this issue Sep 2, 2016
@kelp404
fixed `TypeError: Cannot assign to read only property 'warnings' of t…
715823f 
…rue`

gruntjs/grunt-contrib-uglify#298
@amccarthyMatchbook
Copy link

+1

farshidtz pushed a commit to linksmart/dgw-dashboard that referenced this issue Sep 5, 2019
@krylovsk
grunt:uglify compressed: '{}' instead of 'true': see gruntjs/grunt-co…
0751f3a 
…ntrib-uglify#298
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
10 participants
@4aficiona2 @willthemoor @ahmednuaman @XhmikosR @NorikDavtian @rauberdaniel @Deanzie @amccarthyMatchbook @oleksiipysanko @2013gang