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

Update wizard.sh to fix WSL compatibility #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RNetc0rp
Copy link

Lines 59-64 changed to switch from using systemctl (which is not supported in WSL ububtu) to service. Fix has been tested in a windows 10 machine running the latest ubuntu (as of 9/22/2024) and looks to work.

Lines 59-64 changed to switch from using systemctl (which is not supported in WSL ububtu) to service.  Fix has been tested in a windows 10 machine running the latest ubuntu (as of 9/22/2024) and looks to work.
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review

Compatibility Issue
The PR switches from using systemctl to service for starting Docker, which may cause issues on non-WSL systems.

Incomplete Implementation
The PR adds code to start Docker but doesn't include code to enable Docker on system boot for WSL systems.

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

Successfully merging this pull request may close these issues.

1 participant