Skip to content
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

Universal build uses x64 on M1 mac #114986

Closed
jrieken opened this issue Jan 26, 2021 · 10 comments · Fixed by #115280
Closed

Universal build uses x64 on M1 mac #114986

jrieken opened this issue Jan 26, 2021 · 10 comments · Fixed by #115280
Assignees
Labels
info-needed Issue requires more information from poster

Comments

@jrieken
Copy link
Member

jrieken commented Jan 26, 2021

Testing #114974

  • download latest insiders
  • start
  • first start takes a very long time
  • VS Code running x64, not ARM

image

@jrieken
Copy link
Member Author

jrieken commented Jan 26, 2021

This is maybe because it was running both variants on that machine already?

@deepak1556
Copy link
Collaborator

Was this after quitting all running instances of the app ? It will not pick up correct architecture if there is already a running instance for a particular architecture.

@deepak1556 deepak1556 added 🍎 si info-needed Issue requires more information from poster labels Jan 27, 2021
@jrieken
Copy link
Member Author

jrieken commented Jan 27, 2021

Yeah, all instances quit and uninstalled

@deepak1556
Copy link
Collaborator

Can you open the Info.plist file under Visual Studio Insiders.app/Contents and add the following value https://developer.apple.com/documentation/bundleresources/information_property_list/lsarchitecturepriority with arm64 as the first value in the array followed by x86_64

@deepak1556
Copy link
Collaborator

If it still doesn't pick up the correct architecture after the above, can you also add the boolean https://developer.apple.com/documentation/bundleresources/information_property_list/lsrequiresnativeexecution set to true

@jrieken
Copy link
Member Author

jrieken commented Jan 27, 2021

Downloading fresh from the website and the file command shows this

jrieken@joh-13m1 Code % file ~/Downloads/Visual\ Studio\ Code\ -\ Insiders.app/Contents/MacOS/Electron 
/Users/jrieken/Downloads/Visual Studio Code - Insiders.app/Contents/MacOS/Electron: Mach-O 64-bit executable x86_64
jrieken@joh-13m1 Code % 

@deepak1556
Copy link
Collaborator

@jrieken was it downloaded from https://az764295.vo.msecnd.net/insider/7b0cfdd04ad530a9c8d782c618afd79290be3d64/VSCode-darwin-universal.zip ?

I see the file has universal binaries.

@jrieken
Copy link
Member Author

jrieken commented Jan 28, 2021

downloaded from here: https://code.visualstudio.com/insiders/.

@deepak1556
Copy link
Collaborator

Ah the website has not been updated yet to pull in unviersal https://github.com/microsoft/vscode-website/pull/924 , will be updated for the release. For testing you can download from the builds page.

@jrieken
Copy link
Member Author

jrieken commented Jan 28, 2021

🤦 that explains it

@github-actions github-actions bot locked and limited conversation to collaborators Mar 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants