Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

proc-macro-test: Pass target to cargo invocation #15385

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

inferiorhumanorgans
Copy link

When cross compiling macos → dragonfly the dist build fails in the proc-maro-test-impl crate with the following error:

ld: unknown option: -z\nclang: error: linker command failed with exit code 1 (use -v to see invocation)

This appears to be a wart stemming from using an Apple host for cross compiling. Passing the target along to cargo allows it to pick up a linker that it understands and DTRT.

When cross compiling macos → dragonfly the dist build fails in the
proc-maro-test-impl crate with the following error:

ld: unknown option: -z\nclang: error: linker command failed with
exit code 1 (use -v to see invocation)

This appears to be a wart stemming from using an Apple host for cross
compiling.  Passing the target along to cargo allows it to pick up
a linker that it understands and DTRT.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 3, 2023
@Veykril
Copy link
Member

Veykril commented Aug 4, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Aug 4, 2023

📌 Commit c5d4f73 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 4, 2023

⌛ Testing commit c5d4f73 with merge 7f3353f...

bors added a commit that referenced this pull request Aug 4, 2023
…eykril

proc-macro-test: Pass target to cargo invocation

When cross compiling macos → dragonfly the dist build fails in the proc-maro-test-impl crate with the following error:

`ld: unknown option: -z\nclang: error: linker command failed with exit code 1 (use -v to see invocation)`

This appears to be a wart stemming from using an Apple host for cross compiling.  Passing the target along to cargo allows it to pick up a linker that it understands and DTRT.
@bors
Copy link
Contributor

bors commented Aug 4, 2023

💥 Test timed out

@Veykril
Copy link
Member

Veykril commented Aug 5, 2023

@bors retry

@bors
Copy link
Contributor

bors commented Aug 5, 2023

⌛ Testing commit c5d4f73 with merge 83ebbbf...

bors added a commit that referenced this pull request Aug 5, 2023
…eykril

proc-macro-test: Pass target to cargo invocation

When cross compiling macos → dragonfly the dist build fails in the proc-maro-test-impl crate with the following error:

`ld: unknown option: -z\nclang: error: linker command failed with exit code 1 (use -v to see invocation)`

This appears to be a wart stemming from using an Apple host for cross compiling.  Passing the target along to cargo allows it to pick up a linker that it understands and DTRT.
@bors
Copy link
Contributor

bors commented Aug 5, 2023

💔 Test failed - checks-actions

@Veykril
Copy link
Member

Veykril commented Aug 15, 2023

Not sure why CI is failing here tbh

@inferiorhumanorgans
Copy link
Author

Not sure why CI is failing here tbh

@Veykril Is there any way to get an actual error message from that build?

@lnicola
Copy link
Member

lnicola commented Aug 23, 2023

@bors retry

@bors
Copy link
Contributor

bors commented Aug 23, 2023

⌛ Testing commit c5d4f73 with merge d3eda13...

bors added a commit that referenced this pull request Aug 23, 2023
…eykril

proc-macro-test: Pass target to cargo invocation

When cross compiling macos → dragonfly the dist build fails in the proc-maro-test-impl crate with the following error:

`ld: unknown option: -z\nclang: error: linker command failed with exit code 1 (use -v to see invocation)`

This appears to be a wart stemming from using an Apple host for cross compiling.  Passing the target along to cargo allows it to pick up a linker that it understands and DTRT.
@bors
Copy link
Contributor

bors commented Aug 23, 2023

💔 Test failed - checks-actions

@lnicola
Copy link
Member

lnicola commented Aug 23, 2023

@inferiorhumanorgans no need to do anything, I'll try to deal with this later.

rust-lang/rust#115011

@lnicola
Copy link
Member

lnicola commented Aug 23, 2023

@bors retry

@bors
Copy link
Contributor

bors commented Aug 23, 2023

⌛ Testing commit c5d4f73 with merge f5b7c60...

@bors
Copy link
Contributor

bors commented Aug 23, 2023

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing f5b7c60 to master...

@bors bors merged commit f5b7c60 into rust-lang:master Aug 23, 2023
@lnicola
Copy link
Member

lnicola commented Aug 23, 2023

changelog internal (first contribution) pass target in proc-macro-test to fix cross-compilation on Darwin hosts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants