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 wsl installation instructions added #125

Merged
merged 3 commits into from
May 19, 2021
Merged

windows wsl installation instructions added #125

merged 3 commits into from
May 19, 2021

Conversation

ougni
Copy link
Contributor

@ougni ougni commented May 15, 2021

Turns out installing on WSL-Windows works just fine as long as the path for the Cha log is specified correctly. Added some instructions for installation using WSL-Ubuntu on Windows.

@chadmccune
Copy link

Does this solve the false offline notification problem?

@ougni
Copy link
Contributor Author

ougni commented May 15, 2021

Was not aware of that problem. But the short answer is NO.
This just proposes you run the chia-dog on WSL-Ubuntu instead of native Windows (with simplers installation process).

@chadmccune
Copy link

I thought it might because I dont think the problem exists on linux as far as i'm aware

WINDOWS.md Outdated Show resolved Hide resolved
@martomi
Copy link
Owner

martomi commented May 15, 2021

My guess is also that it will solve the mentioned above problem if it reuses the tail -F command.

@martomi martomi requested a review from pieterhelsen May 15, 2021 19:06
@chadmccune
Copy link

chadmccune commented May 15, 2021 via email

WINDOWS.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@pieterhelsen pieterhelsen left a comment

Choose a reason for hiding this comment

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

Overall good addition to the documentation. I would add a brief explanation of WSL, so people are not confused when deciding between both options.

WINDOWS.md Outdated Show resolved Hide resolved
@ougni ougni requested a review from pieterhelsen May 18, 2021 01:21
Copy link
Collaborator

@pieterhelsen pieterhelsen left a comment

Choose a reason for hiding this comment

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

Looks good!

@martomi martomi merged commit c997b83 into martomi:dev May 19, 2021
jinnatar pushed a commit to jinnatar/chiadog that referenced this pull request Feb 5, 2023
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.

4 participants