Skip to content

Commit 3d8e5d4

Browse files
authored
Rollup merge of rust-lang#57233 - Smibu:rename-and-fix-nolink-test, r=alexcrichton
Rename and fix nolink-with-link-args test There are three problems with the nolink-with-link-args test: * The test fails when using MSVC. It's caused by the `linker-flavor=ld` flag which was added in rust-lang#46291. * In its comment, this test tests that "link_args are indeed passed when nolink is specified", but the `nolink` attribute has been removed [a long time ago](rust-lang#12826). * Pattern has a small typo. At first I was going to completely remove this test, but there is [a closed pull request for that](rust-lang#21090). So: * rename the file as suggested in the closed PR * adjust the comment * fix typo in the pattern * add `ignore-msvc`. r? @alexcrichton
2 parents 6c58b84 + 794b81e commit 3d8e5d4

File tree

2 files changed

+12
-13
lines changed

2 files changed

+12
-13
lines changed
+12
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
// ignore-msvc due to linker-flavor=ld
2+
// error-pattern:aFdEfSeVEEE
3+
// compile-flags: -C linker-flavor=ld
4+
5+
/* Make sure invalid link_args are printed to stderr. */
6+
7+
#![feature(link_args)]
8+
9+
#[link_args = "aFdEfSeVEEE"]
10+
extern {}
11+
12+
fn main() { }

src/test/compile-fail/nolink-with-link-args.rs

-13
This file was deleted.

0 commit comments

Comments
 (0)