Skip to content

Conversation

@jamadeo
Copy link
Collaborator

@jamadeo jamadeo commented Sep 26, 2025

What broke: cryptographic dependency aws-lc-rs checks during compilation that the C compiler is one that is not affected by https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95189. Our linux builds use Ubuntu 20, which ships with gcc 9, which is affected. This PR installs and uses gcc 10, which includes a fix and passes the aws-lc-rs build.

@jamadeo jamadeo changed the title Jackamadeo/linux cc compiler Use gcc 10 in cross builds Sep 26, 2025
@jamadeo jamadeo changed the title Use gcc 10 in cross builds fix: linux builds: Use gcc 10 in cross builds Sep 26, 2025
@jamadeo jamadeo changed the title fix: linux builds: Use gcc 10 in cross builds fix: linux builds: use gcc 10 in Linux builds Sep 26, 2025
@jamadeo jamadeo requested a review from alexhancock September 26, 2025 14:37
target-suffix: unknown-linux-gnu
build-on: ubuntu-latest
use-cross: true
cc: gcc-10
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like latest stable version of gcc is 15. could we update all the way to newest?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

quite possibly, yeah. I went with 10 because ubuntu 20 (what we're building on) ships with 9, and 10 is where the original issue is fixed: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95189

according to some googling though, 10 is the latest in the official repositories for ubuntu 20 (I haven't verified) so it might mean some extra steps to get it working for not much gain

@jamadeo jamadeo marked this pull request as ready for review September 26, 2025 14:53
@jamadeo jamadeo merged commit 945c783 into main Sep 26, 2025
11 checks passed
@jamadeo jamadeo deleted the jackamadeo/linux-cc-compiler branch September 26, 2025 15:12
katzdave added a commit that referenced this pull request Sep 26, 2025
* 'main' of github.com:block/goose:
  Docs: Add link to Plug & Play video for Reddit MCP (#4852)
  remove only-pr-labels (#4842)
  Update video link in README.md to lowercase goose (#4846)
  chore: fix all warnings in test compilation (#4827)
  fix: linux builds: use gcc 10 in Linux builds (#4841)
  fix: keep one goosed client per BrowswerWindow (#4805)
  add automation for marking and closing stale PRs (#4826)
  chore: turn clippy on for test code (#4817)
  fix: pricing integration test (#4837)
  blog post on subagents vs subrecipes (#4829)
  fix chat button alignment and spacing for attachments (#4794)
  fix: remove nested double quotes in windows automation_script tool description (#4824)
  fix: a few things with the mcp snapshot test (#4818)
  Revert "fix(compaction): try to catch more context limit exceeded erors and compact" (#4820)
katzdave added a commit that referenced this pull request Sep 26, 2025
* 'main' of github.com:block/goose:
  Docs: Add link to Plug & Play video for Reddit MCP (#4852)
  remove only-pr-labels (#4842)
  Update video link in README.md to lowercase goose (#4846)
  chore: fix all warnings in test compilation (#4827)
  fix: linux builds: use gcc 10 in Linux builds (#4841)
  fix: keep one goosed client per BrowswerWindow (#4805)
  add automation for marking and closing stale PRs (#4826)
  chore: turn clippy on for test code (#4817)
  fix: pricing integration test (#4837)
katzdave added a commit that referenced this pull request Sep 29, 2025
…ovements

* 'main' of github.com:block/goose:
  Update video link in README.md to lowercase goose (#4846)
  chore: fix all warnings in test compilation (#4827)
  fix: linux builds: use gcc 10 in Linux builds (#4841)
  fix: keep one goosed client per BrowswerWindow (#4805)
  add automation for marking and closing stale PRs (#4826)
  chore: turn clippy on for test code (#4817)
  fix: pricing integration test (#4837)
HikaruEgashira pushed a commit to HikaruEgashira/goose that referenced this pull request Oct 3, 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