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

docpad hangs, doesn't exit, after 'The action completed successfully' #595

Closed
Ashton-W opened this issue Aug 12, 2013 · 8 comments
Closed

Comments

@Ashton-W
Copy link

Docpad is not exiting after emitting the message info: The action completed successfully when running most commands.

info: Generated 66/66 files in 30.916 seconds
    → [2013-08-12 10:02:44.766] [/usr/local/lib/node_modules/docpad/out/lib/docpad.js:3016] [unknown]
debug: Performed the action generate
    → [2013-08-12 10:02:44.773] [/usr/local/lib/node_modules/docpad/out/lib/docpad.js:2826] [forward]
info: The action completed successfully
    → [2013-08-12 10:02:44.777] [/usr/local/lib/node_modules/docpad/out/lib/interfaces/console.js:135] [ConsoleInterface.completeAction]

How to reproduce (OSX)

git clone git://github.com/bevry/docpad-website.git
cd docpad-website
npm install
docpad --debug 7 generate --env static
@balupton
Copy link
Member

Thanks for reporting this. It is likely that we are not shutting down things correctly. Effort has been made on this earlier but we could take it further. I'll look into it.

@Ashton-W
Copy link
Author

Does not affect Twitter Bootstrap skeleton. I think it is safe to assume this only affects docpad-website, possibly others.

git clone git://github.com/docpad/twitter-bootstrap.docpad.git
cd twitter-bootstrap.docpad
npm install
docpad --debug 7 generate --env static

@Ashton-W
Copy link
Author

When it is hanging, activity monitor shows node retaining a handle on docpad-website/node_modules/docpad-plugin-robotskirt/node_modules/robotskirt/build/Release/robotskirt.node.

Update: Removing robotskirt does not solve it.

@Ashton-W
Copy link
Author

Found a temporary fix:

Remove regenerateEvery from docpad.coffee

- regenerateEvery: 1000*60*60*24

@chase
Copy link
Contributor

chase commented Aug 12, 2013

I'm guessing something isn't calling the next function. That's usually the case with this.

@balupton
Copy link
Member

Fix coming soon :)

balupton added a commit that referenced this issue Aug 12, 2013
- v6.49.0 August 9, 2013
	- DocPad will now shutdown and destroy itself more thoroughly
		- Thanks to [Ashton Williams](https://github.com/Ashton-W) for [issue #595](#595)
	- Added the event `docpadDestroy` for plugins that are doing anything long-running so they can destroy themselves thoroughly too
@balupton
Copy link
Member

Released to v6.49.0 :) 👍

@Ashton-W
Copy link
Author

Amazing turn around, thanks.

balupton added a commit that referenced this issue Oct 23, 2013
- v6.49.0 August 9, 2013
	- DocPad will now shutdown and destroy itself more thoroughly
		- Thanks to [Ashton Williams](https://github.com/Ashton-W) for [issue #595](#595)
	- Added the event `docpadDestroy` for plugins that are doing anything long-running so they can destroy themselves thoroughly too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants