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

Append to the host file on Win32 #37

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

dlannoye
Copy link
Contributor

Currently devcert will create a new host file on windows rather then appending to the existing host file.

@dlannoye
Copy link
Contributor Author

@zetlen I saw that you started merging some PRs back. I have a few changes from a private fork that I would like to bring over. This is the first of them

@zetlen zetlen self-requested a review October 29, 2019 15:39
Copy link
Collaborator

@zetlen zetlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @dlannoye. This is a very important change. I know that the >> append redirect works in CMD.EXE as well as POSIX shells, but can you confirm that it works if the user has forced their standard shell to be PowerShell? (If this command bypasses the user's default shell, that's okay too.

@dlannoye
Copy link
Contributor Author

dlannoye commented Nov 1, 2019

I tested running the >> append works under PowerShell too. I have also run the devcert tool from both CMD.exe and PowerShell and can confirm it works as expected.

@zetlen zetlen merged commit 0df830c into davewasmer:master Nov 4, 2019
@zetlen
Copy link
Collaborator

zetlen commented Nov 4, 2019

Thanks @dlannoye !

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

Successfully merging this pull request may close these issues.

2 participants