Improve error message when backticks fail #31

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@meonkeys

This avoids a very obscure error in webserver logs: sh: 1: cd: can't cd to ~.

@meonkeys meonkeys sdk.class: Improve error message if backticks fail
This can lead to a [very obscure error in your webserver logs](http://blog.isnoop.net/2012/04/02/cd-1-cant-cd-to/): `sh: 1: cd: can't cd to ~`.
14ce560
@jeremeamia

Thanks, we'll check this out. Sorry for the late reply.

@joserobleda

wich dir is ~ in that environment? maybe i can create that folder in order to avoid the log error meanwhile they accepts this pull request...

@joserobleda depends on the system confirmation and what user is running the code. Typically it is /home/USER.

When I write the first post I made some test and I discover the rela path of ~ in apache enviroment: the same folder of the execution script. I just create a folder called ~ on that folder and it works. But its just a patch beacause i have to do this in any folder of my projects wich use the sdk ...

I hope this will be resolved soon

Also see pull request #31

@jeremeamia

I've made the changes in my local copy and have added you to the CONTRIBUTORS doc. We do not accept pull requests directly for logistical reasons, but your changes will appear in the next version of the SDK. Thanks again!

@jeremeamia jeremeamia closed this Sep 20, 2012
@meonkeys

@jeremeamia great, thanks.

@joserobleda

oh! great news!

@jeremeamia I dont know if I will be included, but that would be awesome!!! ;)

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