Skip to content

install necessary cmake version directly from cmake github releases#11043

Merged
dhalbert merged 1 commit into
adafruit:mainfrom
bablokb:cmake_version
Jun 4, 2026
Merged

install necessary cmake version directly from cmake github releases#11043
dhalbert merged 1 commit into
adafruit:mainfrom
bablokb:cmake_version

Conversation

@bablokb

@bablokb bablokb commented Jun 3, 2026

Copy link
Copy Markdown

ESP-IDF needs a more recent cmake version than the current base system in Github-codespaces provides. Adding the Kitware repo does not help anymore since the cmake version there is also outdated.

This PR therefore installs a suitable cmake version directly from the Github releases of cmake.

@dhalbert dhalbert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This seems fine! I looked in esp-idf and the minimum CMake version looks like 3.19.0 now, but the minimums vary among components. That's the highest version requested now that I see.

If you just did apt install cmake would that do fine? I don't know how a devcontainer is set up so that may not make sense?

@bablokb

bablokb commented Jun 4, 2026

Copy link
Copy Markdown
Author

The devcontainer is basically a docker image running Ubuntu. cmake is already installed, but the highest version is 3.16. Maybe there are some backport repos with newer versions that could be added/activated, but that seems more complicated than this solution. And after all, devcontainers are throw-away environments, so installing to /usr/local should be fine.

@dhalbert

dhalbert commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

@bablokb Got it, thanks. I have already advanced to Ubuntu 26.04, which has cmake 4.something, but that may not be available.

@dhalbert dhalbert merged commit fffed33 into adafruit:main Jun 4, 2026
14 checks passed
@bablokb bablokb deleted the cmake_version branch June 6, 2026 06:57
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.

2 participants