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

README: Add WSL install #171

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

README: Add WSL install #171

wants to merge 1 commit into from

Conversation

Creammilky
Copy link

I found overlayfs somehow are not compatible with WSL when using try.
Solution:
It can be solved by installing unionfs so I add the WSL subtitle in README-Installing

@ezrizhu ezrizhu requested a review from mgree August 2, 2024 21:22
@@ -28,6 +28,10 @@ commands that you don't already trust on your system, (i.e. network calls are al
* `attr` (for `getfattr`)
* `pandoc` and `autoconf` (if working from a GitHub clone)

To install these packages, run the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

This is debian/Ubuntu specific. Best to leave it out?

@@ -89,6 +93,21 @@ cd try
makepkg -sic
```

#### WSL2
It is highly recommended to use [unionfs](https://github.com/rpodgorny/unionfs-fuse) to run `try`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Will try even work without it? Then it's not "highly recommended", but required for use on WSL!

Copy link
Contributor

@mgree mgree left a comment

Choose a reason for hiding this comment

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

We should firm up the text here to work on all platforms (and all versions of WSL).


To install `unionfs` for your WSL, run the following:
```ShellSession
$ sudo apt install unionfs-fuse
Copy link
Contributor

Choose a reason for hiding this comment

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

Debian/Ubuntu specific.

To install `unionfs` for your WSL, run the following:
```ShellSession
$ sudo apt install unionfs-fuse
```
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is unnecessary: we have plenty of description of how to run try.

@ezrizhu
Copy link
Collaborator

ezrizhu commented Aug 8, 2024

and i'll take care of the CI failures.

@ezrizhu
Copy link
Collaborator

ezrizhu commented Aug 18, 2024

You can merge the latest main branch now to resolve the CI failures.

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.

3 participants