bash-lsp / ide-bash

Bash language support for Atom-IDE

Home Page:https://atom.io/packages/ide-bash

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Python: missing windows prerequisite

roblogic opened this issue · comments

Assuming the user has npm, they will also need Python installed somewhere, lest they get a horrible message like this.
image

Please consider adding the following to the README:

npm install --global --production windows-build-tools

Python should now be installed to %USERPROFILE%\.windows-build-tools\python27.

(per https://www.npmjs.com/package/windows-build-tools )

Another problem in the README:
git clone git@github.com:mads-hartmann/ide-bash && cd ide-bash won't work with Windows Powershell. A possibly more platform-agnostic approach is:

cd <somewhere>
git clone https://github.com/mads-hartmann/ide-bash.git
cd ide-bash

(https is more friendly to users on corporate networks)

@papesch I know next to nothing about getting a proper developer environment up and running on Windows. Could I persuade you to open a PR with these additions to the README? ☺️ Also if you have any ideas on how to fix bash-lsp/bash-language-server#41 I'd truly appreciate it

( comment moved to #4 )
PS: pull request? possibly, never tried it :P

@papesch Thanks a lot for the hints! Could I get you to comment this in the other issue - perhaps someone on a windows computer wants to try it out 👍

The issues in bash-lsp/bash-language-server#41 seem more specific to VS Code, but a similar problem to the Atom spawn issue I suppose. Could be mitigated by #1