Skip to content

Conversation

@omajid
Copy link
Member

@omajid omajid commented Jun 26, 2024

This adds some extra line to the messages that cmake prints out:

-- The C compiler identification is Clang 18.1.6
...
-- The linker identification is LLD 18.1.6 (compatible with GNU linkers)
...

This makes it easier to see which linker is being used by the runtime. This is useful in debugging, specially in scenarios where we pick the wrong linker and then produce a broken runtime (like #43349), but have no quick way to tell what linker was actually used.

This adds some extra line to the messages that cmake prints out:

    -- The C compiler identification is Clang 18.1.6
    ...
    -- The linker identification is LLD 18.1.6 (compatible with GNU linkers)
    ...

This makes it easier to see which linker is being used by the runtime.
This is useful in debugging, specially in scenarios where we pick the
wrong linker and then produce a broken runtime (like
dotnet#43349), but have no quick way
to tell what linker was actually used.
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 26, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jun 26, 2024
@lambdageek lambdageek added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jun 26, 2024
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@akoeplinger akoeplinger merged commit 2fc4bba into dotnet:main Jun 27, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants