Skip to content

Set lowerbound of rustversion of to v1.0.19#336

Merged
Veetaha merged 1 commit intoelastio:masterfrom
Fokko:fd-set-lowerbound
Sep 2, 2025
Merged

Set lowerbound of rustversion of to v1.0.19#336
Veetaha merged 1 commit intoelastio:masterfrom
Fokko:fd-set-lowerbound

Conversation

@Fokko
Copy link
Copy Markdown
Contributor

@Fokko Fokko commented Sep 2, 2025

I got the following error in Iceberg-Rust:

   --> /home/runner/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/bon-macros-3.7.0/src/builder/builder_gen/getters.rs:227:46
    |
227 |                 let module = if rustversion::cfg!(since(1.64.0)) {
    |                                              ^^^ could not find `cfg` in `rustversion`

Which depends on Avro-rs, and Avro-rs depends on Bon.

It looks like rustversion::cfg! has been added in 1.0.19:

I got the following error in Iceberg-Rust:

```
   --> /home/runner/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/bon-macros-3.7.0/src/builder/builder_gen/getters.rs:227:46
    |
227 |                 let module = if rustversion::cfg!(since(1.64.0)) {
    |                                              ^^^ could not find `cfg` in `rustversion`

```

Which depends on Avro-rs, and Avro-rs depends on Bon.

It looks like `rustversion::cfg!` has been added in 1.0.19:

- https://crates.io/crates/rustversion/1.0.19
- https://crates.io/crates/rustversion/1.0.18
Copy link
Copy Markdown
Collaborator

@Veetaha Veetaha left a comment

Choose a reason for hiding this comment

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

Thank you for the fix! I double checked with cargo +nightly update -Z minimal-versions and this seems to be the only offender. Will release this as 3.7.2.

UPD: release is done.

@Veetaha Veetaha merged commit 388f695 into elastio:master Sep 2, 2025
36 checks passed
@github-actions github-actions Bot mentioned this pull request Sep 2, 2025
@Fokko Fokko deleted the fd-set-lowerbound branch September 2, 2025 13:26
@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Sep 2, 2025

Wow, thanks for the quick follow-up, much appreciated 👍

Fokko added a commit to Fokko/avro-rs that referenced this pull request Sep 2, 2025
Bon had a faulty lower-bound, see:

elastio/bon#336
martin-g pushed a commit to apache/avro-rs that referenced this pull request Sep 2, 2025
Bon had a faulty lower-bound, see:

elastio/bon#336
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