-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an install.ps1
script for Windows.
#93
Conversation
The `install.sh` script can still be used with Git bash, but hopefully this native PowerShell script is a bit more friendly for Windows users. Fixes a-scie#91
I'm pretty sure none of us has Windows / PowerShell experience; so this may just be educational reading. It was for me - learned a bit today. |
@@ -180,10 +178,6 @@ while (($# > 0)); do | |||
INSTALL_PREFIX="$2" | |||
shift | |||
;; | |||
--base-name | -b) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kwlzn I got rid of this because I assume you just copy-pastaed from scie-pants. In that case there was a period of trial where the pants bash script was still in use where having a non-standard name was useful. We don't have that transition problem; so this knob seems gratuitous to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep - sgtm.
…g and use simpler function syntax.
Alright, I'm actually happy with this now. All good for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL some powershell!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
The
install.sh
script can still be used with Git bash, but hopefullythis native PowerShell script is a bit more friendly for Windows users.
Fixes #91