Posted by ergonlogic on October 24, 2012 at 2:17pm
4 followers
| Project: | Drush |
| Component: | Core Commands |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
When the command-line version of PHP isn't sufficient, we get a helpful error message. Unfortunately, it uses the wrong exit code, making it difficult to react to this failure mode:
# drush
Your command line PHP installation is too old. Drush requires at least PHP 5.3.5
# echo $?
0
Comments
#1
Changed die() to drush_log() and exit(1). Perhaps drush_set_error() is preferable for remote calls ?
#2
A better approach.
#3
Could you change the error message so that Drush is capitalized. It should only be lowercase when referring to the executable shell script and not the Drush project in general.
#4
done
#5
Automatically closed -- issue fixed for 2 weeks with no activity.