regression: overflow evaluating requirement

[Mark-Simulacrum]

• https://crater-reports.s3.amazonaws.com/beta-1.71-4/beta-2023-06-25/reg/html-0.5.2/log.txt

[INFO] [stdout] error[E0275]: overflow evaluating the requirement `html_sys::scripting::noscript::NoScript: Unpin`
[INFO] [stdout]     |
[INFO] [stdout]     = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`test`)
[INFO] [stdout] note: required because it appears within the type `NoScript`
[INFO] [stdout]    --> /opt/rustwide/workdir/src/generated/noscript.rs:8:16
[INFO] [stdout]     |
[INFO] [stdout] 8   |     pub struct NoScript {
[INFO] [stdout]     |                ^^^^^^^^

[hameerabbasi]

This seems to already have been fixed on the main branch and also on cca7ee5 (which is what I get if I run cargo-bisect-rustc with the HEAD of the beta branch).

[Mark-Simulacrum]

Hm, that rollup doesn't have anything obvious in it - can we bisect on main (master) or check the individual PRs in the rollup? Sounds like there may be a missing backport?

[hameerabbasi]

I'm sorry, I might have miscommunicated. Running cargo-bisect-rustc --start 1.70.0 and cargo-bisect-rustc --start 1.70.0 --end cca7ee58110726983951a19d5fb7316d9243925d both say that the issue isn't present on both endpoints.

Sounds like there may be a missing backport?

That's exactly my understanding as well. However, I don't see an obvious way to find out which PR fixed/caused the issue. My first intuition was to check the beta branch, but cargo-bisect-rustc only seems to work on master.

[compiler-errors]

@hameerabbasi make sure you're building all tests too. this continues to fail on master.

[hameerabbasi]

Ah -- my mistake. I'll do cargo test, I was doing cargo build before.

[apiraino]

this now landed on stable. Following the suggestion of rustc to increase the recursion limit fixes the tests. This can be a legit solution, right? maybe worth reporting to the maintainer of the html repository?

[apiraino]

WG-prioritization assigning priority (Zulip discussion).

@rustbot label -I-prioritize +P-medium