failed to build archive invalid data encountered

[Mark-Simulacrum]

ea45150 (#59953) appears to introduce some sort of bug in the archive generation, which causes script-servo to fail to build on perf: error: failed to build archive: Invalid data was encountered while parsing the file.

@eddyb, do you think it's worth reverting that PR for now?

[Mark-Simulacrum]

Hm, seems like this has since fixed itself. I'm going to close since it was probably spurious.

[eddyb]

Hmm, that error sounds like a toolchain bug/limitation (it really shouldn't care what we throw in there).
I wonder what the before/after #59953 .rmeta size is for script-servo.

[jdm]

I have encountered this locally building Servo's script crate, as has Servo's CI.

[eddyb]

@jdm @Mark-Simulacrum Could someone provide the .rmeta size (e.g. from cargo check) before and after #59953? (which I was wondering about in #65536 (comment))

Presumably for Servo itself it would have to be before and after a relevant rustup?

[jdm]

Before:

-rw-r--r--  1 jdm  staff   86138724 Nov  6 15:05 target/debug/deps/libscript-c57b17b8b8dc05ef.rmeta

After:

-rw-r--r--  1 jdm  staff  110432968 Nov  6 14:55 target/debug/deps/libscript-d913be2dd3c2bc7a.rmeta

[eddyb]

cc @michaelwoerister That's a 28.2% increase, much larger than what we saw for libcore.

@Mark-Simulacrum Would be interesting to have artifact sizes (for benchmarks themselves) on perf although the damage has been done here.

@jdm Okay that's bad but it doesn't increase the number of bits required to hold the number of bytes, maybe the whole rlib size is closer to that threshold?
That's my only guess so far as to why this is failing.

[jdm]

@eddyb The pre-change rlib was ~530mb and post-change was in the range of ~550mb iirc.