Skip to content

Fix GNU linker detection on FreeBSD - #45664

Merged
wfurt merged 1 commit into
dotnet:masterfrom
am11:feature/compiler/configs
Dec 7, 2020
Merged

Fix GNU linker detection on FreeBSD#45664
wfurt merged 1 commit into
dotnet:masterfrom
am11:feature/compiler/configs

Conversation

@am11

@am11 am11 commented Dec 6, 2020

Copy link
Copy Markdown
Member

In #37753, we unified linker detection by version output string. At the time, only cross compilation of FreeBSD was validated, but on the OS, the output looks like:

$ clang -Wl,--version
LLD 10.0.1 (FreeBSD llvmorg-10.0.1-0-gef32c611aa2-1200012) (compatible with GNU linkers)

This PR fixes CoreCLR build on FreeBSD and build succeeds.

Fixes #45663

cc @janvorli, @Thefrank

@Dotnet-GitSync-Bot

Copy link
Copy Markdown
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost

ghost commented Dec 7, 2020

Copy link
Copy Markdown

Tagging subscribers to this area: @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

In #37753, we unified linker detection by version output string. At the time, only cross compilation of FreeBSD was validated, but on the OS, the output looks like:

$ clang -Wl,--version
LLD 10.0.1 (FreeBSD llvmorg-10.0.1-0-gef32c611aa2-1200012) (compatible with GNU linkers)

This PR fixes CoreCLR build on FreeBSD and build succeeds.

Fixes #45663

cc @janvorli, @Thefrank

Author: am11
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@janvorli janvorli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@wfurt wfurt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wfurt
wfurt merged commit 216f96b into dotnet:master Dec 7, 2020
@emaste

emaste commented Dec 7, 2020

Copy link
Copy Markdown

Thanks!

@am11
am11 deleted the feature/compiler/configs branch December 7, 2020 20:52
@ghost ghost locked as resolved and limited conversation to collaborators Jan 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ld: unknown directive: V1.0 when building on FreeBSD x64

6 participants