Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Modify installation steps on Windows if Docker is already installed #140

Closed
mikelik opened this issue Apr 3, 2023 · 0 comments · Fixed by #206
Closed

Modify installation steps on Windows if Docker is already installed #140

mikelik opened this issue Apr 3, 2023 · 0 comments · Fixed by #206
Assignees
Labels
documentation Improvements or additions to documentation 👍 lgtm
Milestone

Comments

@mikelik
Copy link
Contributor

mikelik commented Apr 3, 2023

From @lparisc:
If we're going to continue to support Dune for windows using chocolatey (the other option is to support it using the WSL linux version), then it's important to warn the user during the install to skip the docker installation if Docker is already installed by using the option that prompts the user to install (y/n) every dependency. Otherwise, it will corrupt the current Docker installation.

As part of the fix:

  1. Verify that installing Docker by Chocolatey if it is already installed corrupts Docker
  2. Modify installation steps in README.md

Note: this may not be testable in a virtual machine. It may only be testable on native Windows.

@enf-ci-bot enf-ci-bot moved this to Todo in Team Backlog Apr 3, 2023
@stephenpdeos stephenpdeos added the documentation Improvements or additions to documentation label Apr 19, 2023
@stephenpdeos stephenpdeos added this to the DUNE 1.2.0-rc1 milestone Apr 19, 2023
@mikelik mikelik self-assigned this May 30, 2023
@mikelik mikelik moved this from Todo to In Progress in Team Backlog May 31, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Team Backlog Jun 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation 👍 lgtm
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants