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

Add WSL guidance to start guide #17

Merged
merged 3 commits into from
May 14, 2020
Merged

Conversation

Ruzihm
Copy link
Collaborator

@Ruzihm Ruzihm commented May 14, 2020

Fixes #16

@AEFeinstein
Copy link
Owner

Hm, did you check out those files somewhere on your Windows hard drive (/mnt/c/... in WSL) or on WSL's disk?

@Ruzihm
Copy link
Collaborator Author

Ruzihm commented May 14, 2020

oh jeeze yes. should i go fix the line endings?

@Ruzihm Ruzihm closed this May 14, 2020
@AEFeinstein
Copy link
Owner

The bash thing was still useful, but the file permissions definitely felt odd. Windows and Linux still aren't quite best friends, but you can access your WSL drive from Windows explorer.exe at \\wsl$\Ubuntu

@Ruzihm
Copy link
Collaborator Author

Ruzihm commented May 14, 2020

Oh, I see what you're getting at. I'll re-open then.

Basically, because I cloned into /mnt/c/..., crosstool-NG threw a fit about directory permissions. Should I remove that part and instead add a bit about not cloning into a windows directory e.g. /mnt/c/...?

@Ruzihm Ruzihm reopened this May 14, 2020
@AEFeinstein
Copy link
Owner

That would be a wise bit in the guide, yes.

@Ruzihm
Copy link
Collaborator Author

Ruzihm commented May 14, 2020

Actually, with cloning into a non-windows dir, the case insensitivity fix may not be necessary. Let me close this again and update with my findings.

@Ruzihm Ruzihm closed this May 14, 2020
@Ruzihm Ruzihm reopened this May 14, 2020
@AEFeinstein AEFeinstein merged commit 710fea1 into AEFeinstein:emu May 14, 2020
AEFeinstein pushed a commit that referenced this pull request Dec 17, 2020
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