-
Notifications
You must be signed in to change notification settings - Fork 449
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
useCMakeServer option no longer respected #1059
Comments
We deprecated the useCMakeServer setting in release 1.3.0 and we are going to remove it completely in 1.4. |
@andreeis Sorry for the delay in getting back to you.
Is that documented anywhere? I couldn't find it but maybe I missed it.
I had Looking at the change that introduced File API (0b55e5b), it shows that "conversion" is only done when |
Yes, it was an oversight. We'd be interested to understand why you prefer legacy mode over server and file api modes though. |
@bobbrow Not exactly my choice. I'm using a SDK and the included CMake doesn't support file API and server mode gives an error (I couldn't find out exactly why). |
Is the SDK available online, or is it proprietary? |
Unfortunately it's proprietary. I'm not sure it is CMake Tools fault that server mode doesn't work, but I couldn't find a log that said what command it is trying to run - is it written somewhere? |
You can enable trace logging with the |
CMake Tools 1.4.0 Beta is available on GitHub. Please try it out and let us know if you encounter any issues. |
@bobbrow Sorry for the big delay, the "new world" started and I completely forgot about this. In the meanwhile, I've been able to make server mode work (it was SDK's fault), but the @andreeis @bobbrow The fix inserted by #1218 was to remove the setting, which is good, but doesn't really fix this issue because the warning (and setting |
Brief Issue Summary
The
useCMakeServer
option is no longer respected, although it is still a valid option.Expected:
Either the option is removed (and, if currently set to false, it is transitioned to "legacy" in the new
cmakeCommunicationMode
option) or the option is respected.Platform and Versions
The text was updated successfully, but these errors were encountered: