Skip to content
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

Windows installer and the stack_root env var #2468

Closed
lehtoj opened this issue Aug 10, 2016 · 4 comments
Closed

Windows installer and the stack_root env var #2468

lehtoj opened this issue Aug 10, 2016 · 4 comments

Comments

@lehtoj
Copy link

lehtoj commented Aug 10, 2016

The Install/upgrade docs advises to set stack_root env variable, but does so in the hardest possible way for new users. The SET command only sets the variable for current session, after closing command prompt the variable is gone. If I have understood correctly, this variable is needed for many of the stack commands even after the initial setup.

There is too high a chance that either users don't realize they need to type the SET command again when they open new command prompt window with the intention of using stack, or they don't know they can set the variable permanently in system settings.

This is obviously not a good thing. The installer already adds %userprofile%\AppData\Roaming\local\bin to the Path variable, so the installer should also add the stack_root variable.

Thanks for your hard work in supporting Windows too. :)

@mgsloan
Copy link
Contributor

mgsloan commented Aug 10, 2016

Good idea, feel free to open a PR to the stable branch, with such additions.

Blaisorblade added a commit to Blaisorblade/stack-installer that referenced this issue Aug 16, 2016
Completely untested yet—treat this as pseudocode for now (though I skimmed docs).
@Blaisorblade
Copy link
Collaborator

I've found the right location and sketched a PR; I'll try to compile/test/iterate later today.

@Blaisorblade
Copy link
Collaborator

Under what conditions can a user write to C:\sr? If not, this PR breaks things. #2548 maybe me realize that's probably not the default, especially in enterprise environment.
Reopened to figure out this.

@mgsloan
Copy link
Contributor

mgsloan commented Sep 26, 2016

I'd say it's a better default. For the enterprise environment, STACK_ROOT can be set.

So to me it is fairly cut and dry.

@mgsloan mgsloan closed this as completed Sep 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants