Relax framework linking test

[madsmtm]

This test was introduced by myself in #118644, but was over-specified in that it assumed the path of the linker was always cc, which causes a test failure for Chromium.

[rustbot]

r? @fmease

rustbot has assigned @fmease.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[bjorn3]

You can't just omit part of the stderr file. Compiletest tests that the stderr of rustc exactly matches the file. You will have to do normalization to replace part of the stderr with a fixed string. As an example of output normalization see

rust/tests/ui/error-codes/E0523.rs

Line 7 in 4c6c629

//@ normalize-stderr-test: "(lib)?crateresolve1-([123])\.[a-z]+" -> "libcrateresolve1-$2.somelib"

[madsmtm]

Yes you can, because of the //@ compare-output-lines-by-subset directive (which in turn is used exactly because this test is so flaky, and because the linker output sometimes contain unrelated warnings).

[bjorn3]

I didn't know that annotation existed.

[fmease]

Thanks!

[fmease]

@bors r+ rollup

[bors]

📌 Commit de21296 has been approved by fmease

It is now in the queue for this repository.