lunixbochs/sublimelint — Issue #16: Update README to reflect requirements to run plugin
Author: octoquad
Created: 2011-11-04 14:46:31+00:00
State: open

Hi,

Please could you update your README to inform newcomers to Sublime and your plugin that they need to have either the PHP binary package from php.net, XAMPP or any other LAMP package installed on windows and have the $PATH variable set in order for your plugin to work correctly.

I was a bit confused as I thought this plugin would just work out the box until I looked into the code and saw "process = subprocess.Popen(('php', '-l', '-d display_errors=On'),...". I'm moving away from Netbeans IDE and it does not require php to be installed (or is packaged in netbeans itself), hence the slight confusion. :P

Additionally, I nicer error message should be displayed if the executable could not be found. At the moment the console just spits out:

Traceback (most recent call last):
File ".\sublimelint_plugin.py", line 116, in _view
File ".\sublimelint_plugin.py", line 101, in validate
File ".\sublimelint_plugin.py", line 80, in run
File ".\sublimelint\modules\php.py", line 23, in run
File ".\sublimelint\modules\php.py", line 12, in check
File ".\subprocess.py", line 633, in __init__
File ".\subprocess.py", line 842, in _execute_child
WindowsError: [Error 2] The system cannot find the file specified

Thanks