Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

detach bash on windows. #431

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

taichi
Copy link

@taichi taichi commented Mar 20, 2014

windows don't have bash commands.
so switch to javascript.

@taichi taichi changed the title use javascript instead of bash. detach bash on windows. Mar 20, 2014
taichi added 2 commits March 20, 2014 20:59
because windows don't have unzip binary.
@bchintx
Copy link
Contributor

bchintx commented Mar 20, 2014

@taichi - can you please clarify why you'd like to make these changes? brackets-shell builds just fine on Windows currently so long as you have the right dev environment pre-requisites, such as GitBash. More information on building on Windows can be found here.

@taichi
Copy link
Author

taichi commented Mar 21, 2014

this change prevents most Windows users from small pitfall.

most Windows users launch the bash terminal dedicated only when using Git.
For example, I select "Git Bash" from the context menu of Windows Explorer for start the bash terminal.

most Windows users are using the cmd terminal when using grunt and npm.
bash.exe is not included in the PATH environment variable of the user who is using bash terminal of git only.
So, if you run npm install in cmd terminal, GitBash has been installed, it's an error to not find the unzip.exe and bash.exe.

@bchintx
Copy link
Contributor

bchintx commented Mar 21, 2014

Ah, gotcha. Thanks for the clarification.

I understand your motivation, but many of our developers, including myself, use Git Bash as our Brackets command-line environment on Windows. Unfortunately, your change will break our environments.

In short, if you're hacking on Brackets on Windows. Please be sure to use the Git Bash terminal to interact with git and run the grunt scripts as already documented in the Building Brackets Shell for Windows Wiki. Thanks!

Alternatively, you could submit an alternate set of commands or scripts that might be more "Command Prompt"-friendly, without breaking the existing scripts which leverage Git Bash.

@bchintx bchintx closed this Mar 21, 2014
@bchintx bchintx reopened this Mar 21, 2014
@taichi
Copy link
Author

taichi commented Mar 24, 2014

this change don't change script behavior.
so my change will not break your environments.

@JeffryBooher
Copy link
Contributor

@taichi these changes do change the tools used to in the shell's build script so the behavior hasn't changed -- no, but the tools used to build the shell have changed. These may cause our automated scripts to fail and it could cause users with limited rights the inability to build.

Adm-zip would need to be installed npm -g adm-zip which isn't part of the grunt process and added as a node build dependency.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants