Rust build image needs updating

[nathaniel-brough]

There are quite a number of rust projects that are not building because they depend on the serde crate. But the serde crate requires a more up to date version of rustc (as the current version hasn't been updated since febuary). This is the error across many repositories that have failing builds.

add `#![feature(diagnostic_namespace)]` to the crate attributes to enable�[0m
Step #3 - "compile-libfuzzer-address-x86_64": �[0m    �[0m�[0m�[1m�[38;5;12m= �[0m�[0m�[1mnote�[0m�[0m: this compiler was built on 2024-02-11; consider upgrading it if it is out of date�[0m
Step #3 - "compile-libfuzzer-address-x86_64": 
Step #3 - "compile-libfuzzer-address-x86_64": �[0m�[1mFor more information about this error, try `rustc --explain E0658`.�[0m
Step #3 - "compile-libfuzzer-address-x86_64": The following warnings were emitted during compilation:

So far I've fixed two rounds of builds by simple updating rustc in the project specific Dockerfile e.g.

• rhai: fix build #12409
• starlark-rust: fix rustc version issues #12404

[nathaniel-brough]

Also you appear to have a spam problem...

EDIT: There was some obviously spammy/phishy comment's earlier in the thread that I reported and appear to have been removed by the github security team.

[maflcko]

So far I've fixed two rounds of builds by simple updating rustc in the project specific Dockerfile e.g.

I haven't checked, but I think this will break their coverage build.

See also #11626 and #12365

[nathaniel-brough]

I mean all the builds for the projects i linked are already broken. So I'm not fixing one build at the expense of another because neither the normal builds or the coverage builds are working. That being said I can see the conundrum regarding updating the image in the context of the two issues you linked.

[maflcko]

Sure, your reply and your approach makes sense. However, I think this issue is already tracked in #11626 and attempted to be fixed in #12365, so I think this one can be closed?

[nathaniel-brough]

Hmm I think the issues you've linked are related but aren't the same issue. In fact I think the temporary workaround to the problem you've linked is what has caused the issue with building serde. So I'd say they are kind of separate and apposing issues, until both can be fixed simultaneously. Is that roughly what you are getting at i.e. you believe this issue (with building serde) would be fixed by #12365?

[maflcko]

Is that roughly what you are getting at i.e. you believe this issue (with building serde) would be fixed by #12365?

Yes, that should have been fixed, since it is merged now.