Skip to content

Update zoekt sha

Terri Chu requested to merge tchu-update-zoekt-version into main

What does this merge request do and why?

A recent mac OS (and XCode) upgrade caused issues with the Zoekt binary (due to code signing issues)

Stan upstreamed a PR to Zoekt which was merged in https://github.com/sourcegraph/zoekt/pull/835

This MR bumps the zoekt version used by GDK to the SHA with the fix.

How to set up and validate locally

This is only an issue on macos where clang --version returns 16+

➜ clang --version
Apple clang version 16.0.0 (clang-1600.0.26.3)
Target: arm64-apple-darwin23.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
  1. enable zoekt for gdk
  2. run gdk reconfigure on this branch
  3. verify that you can search for code using Zoekt

note: Exact code search should be present on results page

image

Impacted categories

The following categories relate to this merge request:

Merge request checklist

  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Tests added for new functionality. If not, please raise an issue to follow-up.
  • Documentation added/updated, if needed.
  • Announcement added, if change is notable.
  • gdk doctor test added, if needed.
  • Add the ~highlight label if this MR should be included in the CHANGELOG.md.
Edited by Terri Chu

Merge request reports

Loading