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

[release/5.0-rc2] Fix multiple version strings in coreclr shared libraries #41735

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 2, 2020

Backport of #41711 to release/5.0-rc2

/cc @janvorli

Customer Impact

Attempt to get version string for any coreclr shared library would get that string repeated many times (21 times for coreclr). This may break tooling that check the version.

Testing

Local testing, checking that all shared libraries we build now contain exactly one version string.

Risk

Low, it has no impact on code execution.

We were incorrectly adding the version.c source to all coreclr
libraries and not just the shared ones. That caused multiple
occurences of the version string in coreclr shared libraries
composed of sublibraries.
This change fixes it by adding that source file for shared
libraries only.
I've also discovered that libcoreclrtraceptprovider.so was
missing the version string completely, so I've added it there.
@janvorli janvorli self-assigned this Sep 2, 2020
@janvorli janvorli requested a review from jkotas September 2, 2020 13:38
@janvorli janvorli added this to the 5.0.0 milestone Sep 2, 2020
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

Approved. I will take this for consideration

@jeffschwMSFT jeffschwMSFT added the Servicing-consider Issue for next servicing release review label Sep 3, 2020
@jamshedd jamshedd added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Sep 3, 2020
@jamshedd jamshedd modified the milestones: 5.0.0, 5.0.0 rc2 Sep 3, 2020
@jamshedd
Copy link
Member

jamshedd commented Sep 3, 2020

Approved for RC2.

@jeffschwMSFT jeffschwMSFT merged commit 9bf9663 into release/5.0-rc2 Sep 3, 2020
@jkotas jkotas deleted the backport/pr-41711-to-release/5.0-rc2 branch September 4, 2020 04:08
@karelz karelz modified the milestones: 5.0.0 rc2, 5.0.0 Sep 9, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants