-
Notifications
You must be signed in to change notification settings - Fork 82
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
Error when booting extension on Mac Silicon #974
Comments
@fbricon have you the same problem ? |
I second that, I have a the exact same issue as described by @robertmoura. Thank you for looking into this! I highly appreciate your effort! |
@fbricon @datho7561 @rgrunber have you some idea about the problem? |
Tried on a MacBok Air M1 without any problem.
Can you try the latest vscode-xml pre-release? |
can you check in the Mac security settings if the binary was blocked from running? |
For darwin-arm64, the binary should be
How was your vscode-xml installed ? Did you download it from somewhere, or did you install it through the VS Code Marketplace ? |
I got the same issue as @Estreuselito when installing the pre-release.
Hmm, not exactly sure how to check that but in the Privacy & Security section of the System Settings I can't see anything out of the ordinary. Was there something specific that you were looking for?
Just via the VS Code Marketplace. Nothing out of the ordinary that I know of. |
vscode-xml/src/server/binary/binaryServerStarter.ts Lines 216 to 218 in 4549a37
😐 thanks @datho7561 for reminding about this. I'll propose a fix. As a workaround, could anyone with the issue try :
|
@rgrunber Thanks for looking into this issue! When going to that folder I already had that file. On the pre-release version I had both files in that folder so I'm not sure what to do in this case. Did you have any ideas? Pre release version: |
The first screenshot makes sense. Looks like it's of the 0.26.1 release, which did package the x86_64 LemMinX binary for macos-arm64 (relying on Rosetta layer to work). As for the pre-release, I think I found yet another (similar) issue : Lines 27 to 31 in 4549a37
In the event a binary can't be found locally, we download based on that logic. My guess is if the "aarch_64" (macos-arm64) binary failed to be located (because it was wrongly looking for the x76_64 one), the extension tried to download a pre-release version.. of the wrong version (the "latest" folder only applies to pre-releases). I'll need to update this part of the build. @robertmoura , if you're able to do so the following would help :
Specifically, the aarch64 one should say something about arm64 and the x86_64 one should say something about x86_64.
This should just allow the extension to use the wrong file path, which now has the correct binary.
Hopefully that works and confirms all the changes we need to make to fix things. |
I've run that and got the following which matches what you've said.
After running step 2 and restarting VSCode the extension seems to work as intended! I believe you may have hit this one on the head! Thanks heaps @rgrunber for being on top of this one! Really appreciate it 😊 |
The next pre-release (20240318.. in a few hours from now), should have the necessary fix. |
@rgrunber I've got a similar issue but on Windows: ESET antivirus detects the last version as a trojan:
Extension version: VS Code version info:
Windows version:
ESET version:
|
@kasthack The bug reported in this issue was specific to apple on ARM64 and is resolved, could you please open a new issue for this bug? |
Thanks for working on this extension! Really love using it for formatting XML files.
I recently upgraded my Mac and this extension used to work but is failing on start up now. I think it may have something to do with the M1 chip? Any idea what could be causing this issue? Here's what I'm getting in the XML Support output:
The text was updated successfully, but these errors were encountered: