tremor-rs / tremor-runtime

Main Tremor Project Rust Codebase
https://www.tremor.rs
Apache License 2.0
857 stars 125 forks source link

tremor-runtime 0.12.4 failed to build against rust 1.71.0 #2340

Closed chenrui333 closed 11 months ago

chenrui333 commented 1 year ago

While upgrading rust to 1.71.0, we found some build failure as shown below:

error: no rules expected the token `:`
   --> tremor-script/src/std_lib/stats.rs:466:19
    |
466 |           let res = literal!({
    |  ___________________^
467 | |             "count": count,
468 | |             "sum": sum,
469 | |             "min": min,
...   |
472 | |             "percentiles": p
473 | |         });
    | |__________^ no rules expected this token in macro call
    |
note: while trying to match `=>`
   --> /private/tmp/tremor-runtime-20230720-90076-vuf8fy/tremor-runtime-0.12.4/tremor-value/src/macros.rs:187:46
    |
187 |     (@object @count [@entries $(($value:expr => $($key:tt)+))*]) => {
    |                                              ^^
    = note: this error originates in the macro `literal_internal` which comes from the expansion of the macro `literal` (in Nightly builds, run with -Z macro-backtrace for more info)

error: no rules expected the token `:`
   --> tremor-script/src/std_lib/stats.rs:698:12
    |
698 |           Ok(literal!({
    |  ____________^
699 | |             "count": histo.len(),
700 | |             "min": histo.min(),
701 | |             "max": histo.max(),
...   |
705 | |             "percentiles": p,
706 | |         }))
    | |__________^ no rules expected this token in macro call
    |
note: while trying to match `=>`
   --> /private/tmp/tremor-runtime-20230720-90076-vuf8fy/tremor-runtime-0.12.4/tremor-value/src/macros.rs:187:46
    |
187 |     (@object @count [@entries $(($value:expr => $($key:tt)+))*]) => {
    |                                              ^^
    = note: this error originates in the macro `literal_internal` which comes from the expansion of the macro `literal` (in Nightly builds, run with -Z macro-backtrace for more info)

   Compiling gcemeta v0.1.4
   Compiling trust-dns-resolver v0.21.2
   Compiling async-native-tls v0.3.3
   Compiling serde_with v1.14.0
   Compiling web-sys v0.3.57
error: could not compile `tremor-script` (lib) due to 2 previous errors

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/5594719599/job/15158499681?pr=136514

Licenser commented 11 months ago

Tremor build against 1.72.1 and will build with1.7.3 soon.