-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 WSL config docs #1176
Update WSL config docs #1176
Conversation
Related microsoft/WSL#2578 |
"intelliSenseMode": "clang-x64", | ||
"includePath": [ | ||
"${workspaceRoot}", | ||
"${localappdata}/lxss/rootfs/usr/lib/gcc/x86_64-linux-gnu/5/include", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these look like the Beta paths.
"includePath": [ | ||
"${workspaceRoot}", | ||
"${localappdata}/Packages/CanonicalGroupLimited.UbuntuonWindows_79rhkp1fndgsc/LocalState/rootfs/usr/include/c++/5", | ||
"${localappdata}/Packages/CanonicalGroupLimited.UbuntuonWindows_79rhkp1fndgsc/LocalState/rootfs/usr/include/x86_64-linux-gnu/c++/5", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this works for Ubuntu, but what about OpenSUSE and SLES which are also available in the Windows Store? Should we provide configs for each of them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect that Ubuntu will be the most popular choice, but we should probably talk about the other distros as well. If you don't have time to test them out yourself, can you just add a note that the paths will be different for the other distros?
I already put a note about other distros on line 48.
|
We can generate paths for openSUSE, SUSE Linux Enterprise Server by PS R:\> ($(get-appxpackage).PackageFamilyName | findstr /i 'SUSE Ubuntu') -replace '^', "$`{localappdata`}/Packages/"
${localappdata}/Packages/CanonicalGroupLimited.UbuntuonWindows_79rhkp1fndgsc
${localappdata}/Packages/46932SUSE.openSUSELeap42.2_022rs5jcyhyac
${localappdata}/Packages/46932SUSE.SUSELinuxEnterpriseServer12SP2_022rs5jcyhyac
Even for Fedora and any other subsystems in the future. ( I think you could put this tip in the WSL config doc. One could translate a Linux path PS R:\> ($(get-appxpackage).PackageFamilyName | findstr /i $subsystem) -replace '^', "$`{localappdata`}/Packages/" -replace '$', "/LocalState/rootfs$WSL_PATH"
${localappdata}/Packages/CanonicalGroupLimited.UbuntuonWindows_79rhkp1fndgsc/LocalState/rootfs/usr/include
for now. |
Thanks @icylogic. I included your Powershell command in the doc. |
Docs needed an updated since the current version of WSL from the Fall Creators Update needs a different configuration to the beta version. See #1148.
It does feel kind of wrong having a path going inside a Windows App folder, but it works. I wonder whether WSL should add some environment variables for
rootfs
instead, though thats not really an issue for this repo.