Closed ghost closed 5 years ago
Does this file exist on your machine?
/cc @meteorcloudy
@netroby Did you install Visual Studio 2017 and then removed it? There might be some remaining registry info that confused Bazel.
@laszlocsomor, in this case, the migration tools doesn't need VC++. If there's no VC installation at all, Bazel should ignore the VC configuration, but there might also be a wrong VC installation there which blocks the configuration.
I think we can do more to let user bypass the VC configuration. For example, let users set BAZEL_NO_VC=1
?
@meteorcloudy : I dislike the idea of having one more configuration variable. Can we do this in a smarter way? I think a broken installation is as good as a non-existing one, so Bazel should just ignore it. What do you and @netroby think?
I agree, we can defer the error message to execution time when we really need the compiler tools.
I had vs 2017 community installed, (not uninstall) . I can confirm it exists in my disk.
Do you mean D:\soft\Microsoft Visual Studio\2017\Community\VC\Auxiliary\Build\VCVARSALL.BAT
does exist?
Oh, i see. I installed Visual Studio community, but i had not select cpp development tools(I only installed C#)
But i want to using bazel to build my java project. not cpp. so why you still check the cpp tools?
@netroby : You're right, Bazel should just work in that case too. And Sir @meteorcloudy is working on that already! :)