Skip to content

fix(docker): install protoc to fix lance-encoding build#2995

Merged
yingjiehe-xyz merged 1 commit intoblock:mainfrom
davidroman1914:droman/fix/add-protoc-to-dockerfile
Jun 21, 2025
Merged

fix(docker): install protoc to fix lance-encoding build#2995
yingjiehe-xyz merged 1 commit intoblock:mainfrom
davidroman1914:droman/fix/add-protoc-to-dockerfile

Conversation

@davidroman1914
Copy link
Contributor

@davidroman1914 davidroman1914 commented Jun 18, 2025

This PR adds protobuf-compiler to the Dockerfile so that the lance-encoding crate can compile successfully during the cargo build step.

Without this change, the Docker build fails with the following error:

Error: Custom { kind: NotFound, error: "Could not find protoc. If protoc is installed, try setting the PROTOC environment variable..." }

This fix installs the required protoc binary using apt-get, allowing Goose to build in Docker without error.

Tested on Apple M4 MacBook Pro with docker-compose up --build.

@michaelneale
Copy link
Collaborator

nice one - yeah that will help, forgot about docker

@yingjiehe-xyz yingjiehe-xyz merged commit 9c8e1b0 into block:main Jun 21, 2025
7 checks passed
lifeizhou-ap added a commit that referenced this pull request Jun 23, 2025
* main:
  fix: Improves reliability of flaky log tests (#3029)
  Add xAI Test Coverage (#3020)
  Reorganizing tutorials (#3028)
  feat(providers): update Google Gemini models to latest available models (#2989)
  fix(docker): install protoc to fix lance-encoding build (#2995)
  fix: updated openrouter known models (#3021)
  Mnovich/temporal foreground tasks (#2895)
  add 'install in goose' asset (#3016)
  Added useDarkMode hook for detecting dark mode setting dynamically (#3019)
  docs: add sagemaker provider (#2980)
  Docs: Add Goose Recipes Cookbook Page  (#2998)
  feat (cli): add an alias --id for --name param for goose session --resume command (#3018)
  fix: disable extension  (#3000)
  Add xAI Provider Support for Grok Models (#2976)
  removing dlls from core bin (#3014)
  docs: Update Leaderboard in New Community Landing Page (#2954)
  Douwe/fix include for recipes (#2914)
btdeviant pushed a commit to btdeviant/goose that referenced this pull request Jun 25, 2025
laanak08 added a commit that referenced this pull request Jun 26, 2025
# By Alice Hau (6) and others
# Via Alice Hau
* ahau/tool-strategy: (27 commits)
  fix merge
  merged
  updated computercontroller and developer descriptions
  add additional strats to test
  fmt
  update to get vector db path from env var
  feat: list Databricks-supported models and enable fuzzy search during model configuration (#3039)
  fix: update index when tool selection strategy changes (#2991)
  docs: Add "Add Recipe button" To Recipe Cookbook (#3038)
  fix: issue when parsing recipe parameters (#3031)
  fix: Improves reliability of flaky log tests (#3029)
  Add xAI Test Coverage (#3020)
  Reorganizing tutorials (#3028)
  feat(providers): update Google Gemini models to latest available models (#2989)
  fix(docker): install protoc to fix lance-encoding build (#2995)
  fix: updated openrouter known models (#3021)
  Mnovich/temporal foreground tasks (#2895)
  add 'install in goose' asset (#3016)
  Added useDarkMode hook for detecting dark mode setting dynamically (#3019)
  docs: add sagemaker provider (#2980)
  ...

# Conflicts:
#	.gitignore
#	crates/goose/src/agents/router_tools.rs
s-soroosh pushed a commit to s-soroosh/goose that referenced this pull request Jul 18, 2025
Signed-off-by: Soroosh <soroosh.sarabadani@gmail.com>
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments