Skip to content

Cargo.toml: Update to bindgen 0.72.1#104

Open
drinkcat wants to merge 1 commit into
esp-rs:masterfrom
drinkcat:bindgen
Open

Cargo.toml: Update to bindgen 0.72.1#104
drinkcat wants to merge 1 commit into
esp-rs:masterfrom
drinkcat:bindgen

Conversation

@drinkcat

@drinkcat drinkcat commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Submission Checklist 📝

  • I have updated existing examples or added new ones (if applicable).
  • I have used cargo fmt command to ensure that all changed code is formatted correctly.
  • I have used cargo clippy command to ensure that all changed code passes latest Clippy nightly lints.
  • My changes were added to the CHANGELOG.md in the proper section.

Pull Request Details 📖

Description

Needed for a potential MR on esp-idf-sys, probably good practice anyway.

Testing

With a pending branch where I use BindgenCallbacks::modify_macro.

Needed for a potential MR on esp-idf-sys, probably good practice anyway.
@jarovo

jarovo commented May 29, 2026

Copy link
Copy Markdown

With old bindgen (0.71.1) I was getting:

(venv) jhenner@veverka:~/projects/esp-idf-svc$ LIBCLANG_PATH=/usr/lib64/llvm22/lib64/ MCU=esp32c5 cargo +nightly build -Zbuild-std --target riscv32imac-esp-espidf --example wifi_async_dpp
    Blocking waiting for file lock on package cache
    Updating crates.io index
    Blocking waiting for file lock on package cache
    Blocking waiting for file lock on package cache
    Blocking waiting for file lock on package cache
    Blocking waiting for file lock on package cache
   Compiling esp-idf-hal v0.46.2 (/home/jhenner/projects/esp-idf-hal)
   Compiling esp-idf-svc v0.52.1 (/home/jhenner/projects/esp-idf-svc)
error[E0422]: cannot find struct, variant or union type `spi_transaction_t__bindgen_ty_1` in this scope
     --> /home/jhenner/projects/esp-idf-hal/src/spi.rs:2009:27
      |
 2009 |         __bindgen_anon_1: spi_transaction_t__bindgen_ty_1 {
      |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      ```
      
 Bumping it to 0.72.1 seems have fixed the issue.

@ivmarkov

Copy link
Copy Markdown
Collaborator

I'll merge this at some point, but after the current patch release, because bumping bindgen minor version means bumping embuild's minor version too (bindgen is an API) which in turns means bumping the minor versions of the sys/hal/svc crates...

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