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

Annotate copied Content native lib items with package id to enable customization #26725

Merged
merged 1 commit into from
Sep 3, 2021

Conversation

kzu
Copy link
Contributor

@kzu kzu commented Jul 19, 2021

Add %(NuGetPackageId) as is standard procedure for nuget package-provided assets,
so that other MSBuild targets can act on items contributed by Grpc.Core as needed, for
example, in Pack scenarios.

Fixes #26724

@nicolasnoble

…stomization

Add `%(NuGetPackageId)` as is standard procedure for nuget package-provided assets, 
so that other MSBuild targets can act on items contributed by Grpc.Core as needed, for 
example, in Pack scenarios.

Fixes grpc#26724
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jul 19, 2021

CLA Signed

The committers are authorized under a signed CLA.

@jtattermusch jtattermusch added lang/C# release notes: yes Indicates if PR needs to be in release notes labels Aug 27, 2021
@jtattermusch jtattermusch self-requested a review August 27, 2021 08:14
@jtattermusch
Copy link
Contributor

jtattermusch commented Aug 27, 2021

Adhoc distribtest run: https://fusion2.corp.google.com/invocations/4dedcadb-31af-4923-95c3-0c8a3332e8b5/targets (will post back results later)

https://fusion2.corp.google.com/invocations/093ac219-c739-419b-a917-0bf58ffd86c9/targets

macos artifact build consistently hits infrastructure problems (the build is timing out), so I'm unable to get a green build. That problem is orthogonal to this PR, so I'm simply going to merge and check the greenness of distribtests on the continous run afterwards.

Copy link
Contributor

@jtattermusch jtattermusch left a comment

Choose a reason for hiding this comment

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

LGTM once the tests pass.

@jtattermusch
Copy link
Contributor

@apolcyn please add another LGTM.

@jtattermusch jtattermusch merged commit 6c095c9 into grpc:master Sep 3, 2021
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Sep 3, 2021
@jtattermusch
Copy link
Contributor

FTR I checked the distribtests results after merging and they were green.

lidizheng pushed a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
…stomization (grpc#26725)

Add `%(NuGetPackageId)` as is standard procedure for nuget package-provided assets, 
so that other MSBuild targets can act on items contributed by Grpc.Core as needed, for 
example, in Pack scenarios.

Fixes grpc#26724
@kzu kzu deleted the patch-1 branch October 18, 2021 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
imported Specifies if the PR has been imported to the internal repository lang/C# release notes: yes Indicates if PR needs to be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Annotate copied Content native lib items with package id to enable customization (i.e. in Pack)
4 participants