Open crramirez opened 3 years ago
[user]
section may not be available in wsl.conf
in older Windows 10 versions. A Windows version check may be useful?CreateProcess
return value is not checked.RunProcess
can be a static function here.WslApiLoader::WslLaunch
may be used instead of RunProcess
wrapper.Hello @Biswa96,
Thank you for your comments. My answers:
@Biswa96 I made a code clean up to the PR; please check it. I figure that I addressed all your concerns.
It won't hurt to have [user] not available in older versions; WSL won't use it in older ones. But when the user upgrades Windows, it will be already there configured.
Regards, Carlos
All CLA requirements met.