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

An error occured: Cannot find module 'uglify-js' #137

Closed
wants to merge 1 commit into from
Closed

An error occured: Cannot find module 'uglify-js' #137

wants to merge 1 commit into from

Conversation

pismute
Copy link
Contributor

@pismute pismute commented Feb 18, 2012

Docpad requires 'uglify-js' module. I got a message "An error occured: Cannot find module 'uglify-js'".

Docpad requires 'uglify-js'
@balupton
Copy link
Member

Very strange, I'd say the fix for this would actually be finding out whatever requires it and kill it - as we should keep the core to as minimal dependencies as possible (more dependencies, more risk of failure on different environments).

The culprit is perhaps the buildr plugin - which for now can be deleted.

Would you be able to look into this further, by tracking down the violator? If so, that'll be awesome!

@pismute
Copy link
Contributor Author

pismute commented Feb 19, 2012

Thank you.

Ok, I will check it.

@pismute
Copy link
Contributor Author

pismute commented Feb 20, 2012

It's Very strange. I updated all of modules. then I got same error message from socket.io-client module in autoupdate Plugin.

but I clone again. there are no error.:)

Thank you.

@pismute pismute closed this Feb 20, 2012
@balupton
Copy link
Member

Figured out what this was. config.enabledPlugins if set by your website, would not be merged in with docpad's enabled plugins, instead it would overwrite it, thus enabling the disabled-by-default plugins.

This will be fixed in the upcoming version of docpad.

@balupton balupton reopened this Feb 29, 2012
@balupton
Copy link
Member

Fixed in the just released v3.3.0 :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants