Skip to content

Conversation

@safern
Copy link
Member

@safern safern commented Feb 10, 2022

This is the long term fix for: #65018

This new pattern of experimental ref only projects where the implementation is inbox, needs to be considered as an assembly in the targeting pack, so we should not bump the assembly version on servicing, with this we make sure that we catch such cases and not bump the assembly version for these experimental ref only nuget packages.

The fix to release/6.0 will be ported as part of: #64556 as it is adding another ref only nuget package and needs to include it to unblock it's tests, however, we still need to have a PR created into release/6.0 to make sure we build System.Runtime.Experimental with ServicingVersion set to 2 and GeneratePackageOnBuild set to true.

@ghost
Copy link

ghost commented Feb 10, 2022

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

Issue Details

This is the long term fix for: #65018

This new pattern of experimental ref only projects where the implementation is inbox, needs to be considered as an assembly in the targeting pack, so we should not bump the assembly version on servicing, with this we make sure that we catch such cases and not bump the assembly version for these experimental ref only nuget packages.

The fix to release/6.0 will be ported as part of: #64556 as it is adding another ref only nuget package and needs to include it to unblock it's tests, however, we still need to have a PR created into release/6.0 to make sure we build System.Runtime.Experimental with ServicingVersion set to 2 and GeneratePackageOnBuild set to true.

Author: safern
Assignees: safern
Labels:

area-Infrastructure-libraries

Milestone: -

@safern safern merged commit a31ed47 into dotnet:main Feb 15, 2022
@safern safern deleted the FixInboxRefOnlyAssemblyVersion branch February 15, 2022 16:45
@tannergooding
Copy link
Member

@safern, does this need to be backported to .NET 6?

@safern
Copy link
Member Author

safern commented Feb 15, 2022

This was included in your change that added the InteropServices internal package. We just need to put up a PR for System.Runtime.Experimental to bump the servicing version and set GeneratePackageOnBuild=true.

@ghost ghost locked as resolved and limited conversation to collaborators Mar 17, 2022
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.

3 participants