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

Windows ARM64 builds have non-native ripgrep bundled #237449

Closed
tmm1 opened this issue Jan 8, 2025 · 2 comments · Fixed by #238287
Closed

Windows ARM64 builds have non-native ripgrep bundled #237449

tmm1 opened this issue Jan 8, 2025 · 2 comments · Fixed by #238287
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders search Search widget and operation issues windows VS Code on Windows issues
Milestone

Comments

@tmm1
Copy link
Contributor

tmm1 commented Jan 8, 2025

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version: 1.96.2
  • OS Version: Windows ARM64

Steps to Reproduce:

  1. Install vscode on Windows ARM64 machine
  2. Observe that bundled ripgrep rg is not for arm64
& 'C:\Program Files\Git\usr\bin\file.exe' 'C:\Users\tmm1\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules\@vscode\ripgrep\bin\rg.exe'

C:\Users\tmm1\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules\@vscode\ripgrep\bin\rg.exe: PE32 executable (console) Intel 80386, for MS Windows, 4 sections

note the Intel 80386

cc microsoft/vscode-ripgrep#63

@deepak1556 deepak1556 assigned deepak1556 and roblourens and unassigned joaomoreno Jan 8, 2025
@deepak1556 deepak1556 added bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues windows VS Code on Windows issues labels Jan 8, 2025
@deepak1556 deepak1556 added this to the January 2025 milestone Jan 8, 2025
@deepak1556
Copy link
Collaborator

Was this issue identified because of perf issue in search widget, I believe the intel binary would have been translated and search would have worked today.

I think there is scope for improvement towards the root cause, we have CI check for macOS to ensure correct arch binaries are packaged https://github.com/microsoft/vscode/blob/main/build/darwin/verify-macho.ts. We should have something similar for windows and linux pipeline as well.

@tmm1
Copy link
Contributor Author

tmm1 commented Jan 8, 2025

Correct, the intel binary is translated and works. There are probably NEON and other optimizations being left on the table, although I'm not sure those are enabled in the ripgrep-prebuilt binaries.

I found this through manual audit of the build process and resulting binaries.

@vs-code-engineering vs-code-engineering bot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders search Search widget and operation issues windows VS Code on Windows issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants