Skip to content

Fix Zoekt build command and bump version again

Dylan Griffith requested to merge fix-zoekt-build-commands into main

What does this merge request do and why?

We reverted some updates in !3010 (merged) because it was failing to build. We learnt from https://github.com/sourcegraph/zoekt/pull/535#issuecomment-1442310328 it was because our go build command was not correct. This fixes it now and all the binaries build correctly with the latest version.

How to set up and validate locally

$ rm -rf zoekt
$ gdk reconfigure
...
--------------------------------------------------------------------------------
Building zoekt/bin/zoekt-git-clone version b65e3e6bceef2587a8a9ecaa4515c886dfbc8837
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Building zoekt/bin/zoekt-git-index version b65e3e6bceef2587a8a9ecaa4515c886dfbc8837
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Building zoekt/bin/zoekt-dynamic-indexserver version b65e3e6bceef2587a8a9ecaa4515c886dfbc8837
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Building zoekt/bin/zoekt-webserver version b65e3e6bceef2587a8a9ecaa4515c886dfbc8837
--------------------------------------------------------------------------------
...

$ ls -l zoekt/bin/
total 170696
-rwxr-xr-x@ 1 dylangriffith  staff    11M 24 Feb 14:04 zoekt-dynamic-indexserver*
-rwxr-xr-x@ 1 dylangriffith  staff    17M 24 Feb 14:04 zoekt-git-clone*
-rwxr-xr-x@ 1 dylangriffith  staff    19M 24 Feb 14:04 zoekt-git-index*
-rwxr-xr-x@ 1 dylangriffith  staff    36M 24 Feb 14:04 zoekt-webserver*

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.

Fixes gitlab#393040 (closed)

Edited by Dylan Griffith

Merge request reports