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

Compiler panic while building datetimeformat example in intro.md #1896

Closed
dminor opened this issue May 17, 2022 · 5 comments
Closed

Compiler panic while building datetimeformat example in intro.md #1896

dminor opened this issue May 17, 2022 · 5 comments
Labels
C-datetime Component: datetime, calendars, time zones help wanted Issue needs an assignee R-obsolete Resolution: This issue is no longer relevant T-bug Type: Bad behavior, security, privacy

Comments

@dminor
Copy link
Contributor

dminor commented May 17, 2022

With the version of the datetimeformat example in intro.md from #1895, I get a compiler panic when building locally:

dminor@treebeard:~/src/icu-temp/myapp$ cargo build
   Compiling myapp v0.1.0 (/home/dminor/src/icu-temp/myapp)
thread 'rustc' panicked at 'assertion failed: !new_self_ty.has_escaping_bound_vars()', compiler/rustc_trait_selection/src/traits/error_reporting/mod.rs:1688:9

@dminor dminor added T-bug Type: Bad behavior, security, privacy help wanted Issue needs an assignee C-datetime Component: datetime, calendars, time zones discuss-priority Discuss at the next ICU4X meeting labels May 17, 2022
@robertbastian
Copy link
Member

I believe this is fixed in 1.61: rust-lang/rust#96223

@sffc
Copy link
Member

sffc commented Jun 13, 2022

@dminor Did you verify whether the update to 1.61 resolves the issue?

@sffc sffc added this to the ICU4X 1.0 (Polish) milestone Jun 13, 2022
@sffc sffc removed the discuss-priority Discuss at the next ICU4X meeting label Jun 13, 2022
@sffc
Copy link
Member

sffc commented Jul 28, 2022

@dminor Could you verify if this is fixed?

@dminor
Copy link
Contributor Author

dminor commented Jul 28, 2022

Yes, it's fixed, although I think the example code needs to be updated to reflect recent changes.

@sffc
Copy link
Member

sffc commented Jul 29, 2022

Cool, thanks for confirming. I opened #2281 to track tutorial updates for 1.0 (surprised we didn't already have an issue for that).

@sffc sffc closed this as completed Jul 29, 2022
@sffc sffc added the R-obsolete Resolution: This issue is no longer relevant label Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-datetime Component: datetime, calendars, time zones help wanted Issue needs an assignee R-obsolete Resolution: This issue is no longer relevant T-bug Type: Bad behavior, security, privacy
Projects
None yet
Development

No branches or pull requests

3 participants