Can't make php-grunt run as non-persistent about grunt-php

[sindresorhus]

I just published a 0.1.1 which now correctly kills PHP when done. Can you try it out?

If that doesn't solve it, please try and download the repo and and run npm test a couple of times. Does it pass?

[carlesandres]

Sindre,

Can you give me a hint on how to update from within yeoman?

[sindresorhus]

It's the same as any other Node.js dependency; npm update grunt-php in your project folder

[carlesandres]

Ok. I had to create a new yeoman project which downloaded the new version of grunt-php because npm update grunt-php wouldn't update it, even if I changed package.json manually. But I am quite new to npm and this is probably off-topic here.

Anyway, in grunt-php 0.1.1 the PHP server is now properly killed when the keepalive option is absent or set to false.

I also installed grunt-php 0.1.1 by cloning the repo, run grunt on it and the test passed.

Thanks!

[sindresorhus]

Thanks for testing :)

[ullmark]

Hi, I've been having this trouble as well or maybe I've misunderstood what you can do with the task.
I've upgraded ti 0.1.1 and want to use it the following way.

In my project I have a "client" (js-app) folder, and a "server" (api in php) folder.

When I run grunt I would like to fire of a "connect" server on the client folder and a "php" server on the server folder.

so I want my default task to look like this

  grunt.registerTask('default', ['jshint', 'mocha', 'connect', 'php', 'watch']);

But I cant get the watch to run, regardless of keepalive

[johannesjo]

Same problem here. grunt-php blocks everything else.

[sindresorhus]

@ullmark @johannesjo works for me:

https://github.com/sindresorhus/grunt-php#use-it-with-grunt-contrib-watch
https://github.com/sindresorhus/grunt-php/blob/master/Gruntfile.js#L38

[mikemurray]

Ran into this issue today. It seems to happen when you use localhost as your hostname in the config. Using 127.0.0.1 or 0.0.0.0 resolves this issue.

See this gist https://gist.github.com/mikemurray/6092566 for reference.

It may be helpful add this to the documentation.

[sindresorhus]

@mikemurray done