-
Notifications
You must be signed in to change notification settings - Fork 583
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
Language server ignores SLINT_ENABLE_EXPERIMENTAL_FEATURES
#5974
Comments
The language server doesn't ignore the SLINT_ENABLE_EXPERIMENTAL_FEATURES. |
In a component: changed current-item => {
root.old-item = current-item;
}
I have this extension installed: https://marketplace.visualstudio.com/items?itemName=Slint.slint. |
Did you log out and back in again? Better yet: Reboot. In my experience just restarting an application is not enough to get fresh env vars on Windows (and most unixes for that matter). |
In my experience, it is. It's about whether the process is a child process of another process. If it's not, it gets an up-to-date set of env vars from the OS. Only some software seems to be proactive: Directory Opus seems to read the set of env vars from the OS before starting a child process.
In the meantime I rebooted, and the error is still there. |
the slint-lsp process should be a child process of vscode. That said, we could make it an actual settings or enable it by default in the nightly extension. |
Were you able to check why the slint-lsp process doesn't see the |
With System Informer, I detected the process of the VS Code window, then found the only child process of the subtree with There, I could see the process's env vars under the groups "Process", "User" and "System". Neither me nor ChatGPT know of a way to find out what group an env var belongs to. And since having Anyways, the BTW: These changed-callback errors supersede other error messages since I started to use changed-callbacks in my project, deferring the other errors to build time through |
I am pretty sure we build support for experimental features into the LSP at all times, so I fully expect the 1.7.2 LSP to support this. You could try the nightly extension, that works for me... but as I said, that should not be necessary. I have no real idea what might be going wrong here right now :-( |
With the nightly extension, it works (I don't get the errors anymore). Does this mean that it's only a matter of time until it also works with the regular extension? My regular extension is at v1.7.2. Note that I may have to edit the file (like add a space) after opening the project before the error appears. |
The |
I'm on Windows 10 and set the environment variable
SLINT_ENABLE_EXPERIMENTAL_FEATURES
globally to1
. But even though I terminated and restarted VS Code, thechanged
keyword still gives me an error (for changed-callbacks). Running$env:SLINT_ENABLE_EXPERIMENTAL_FEATURES
in the VS Code PowerShell console confirms that the value is1
.cargo run
also works without problems.It would also be nice, if there'd be a non-global project-only way of ungating the feature; if possible via the file
.vscode/settings.json
.The text was updated successfully, but these errors were encountered: