Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
chenrui333 opened this issue Jul 23, 2023 · 1 comment
Closed

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

chenrui333 opened this issue Jul 23, 2023 · 1 comment

Comments

@chenrui333
Copy link

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
Copy link
Member

Licenser commented Oct 9, 2023

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

@Licenser Licenser closed this as completed Oct 9, 2023
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

No branches or pull requests

2 participants