Skip to content

Compile using ref assemblies #2483

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

Merged
merged 6 commits into from
Oct 11, 2019
Merged

Compile using ref assemblies #2483

merged 6 commits into from
Oct 11, 2019

Conversation

JunTaoLuo
Copy link

@JunTaoLuo JunTaoLuo requested a review from a team as a code owner October 10, 2019 03:48
Copy link
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

LGTM other than the few comments

@JunTaoLuo JunTaoLuo added this to the 3.0.x milestone Oct 10, 2019
@JunTaoLuo JunTaoLuo added the servicing-consider Shiproom approval is required for the issue label Oct 10, 2019
Copy link

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

I'll approve after the biggest change is categorized

@@ -0,0 +1,7 @@
<Project>
Copy link

Choose a reason for hiding this comment

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

This is fine but only if it is not creating a new (useless) package

Copy link
Author

Choose a reason for hiding this comment

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

It is being produced. Will remove.

Copy link
Author

@JunTaoLuo JunTaoLuo left a comment

Choose a reason for hiding this comment

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

@dougbu I marked out the two changes that are made for compiling against ref. Addressing Will's feedback mostly involved adding comments and removing errors that are irrelevant in Extensions so I'm not marking them out. The rest of the changes are just copied over from AspNetCore.

Copy link

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Looks good.

Please confirm no new Microsoft.Extensions.ProviderAliasUtilities.Sources package is created.

@JunTaoLuo JunTaoLuo merged commit e771c42 into release/3.0 Oct 11, 2019
@JunTaoLuo JunTaoLuo deleted the johluo/compile-with-ref branch October 11, 2019 10:41
JunTaoLuo pushed a commit to dotnet/aspnetcore that referenced this pull request Feb 12, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework\n\nCommit migrated from dotnet/extensions@e771c42
JunTaoLuo pushed a commit to dotnet/aspnetcore that referenced this pull request Feb 15, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework\n\nCommit migrated from dotnet/extensions@e771c42
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Feb 28, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework


Commit migrated from dotnet/extensions@e771c42
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 2, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework


Commit migrated from dotnet/extensions@e771c42
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework


Commit migrated from dotnet/extensions@e771c42
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework


Commit migrated from dotnet/extensions@e771c42
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 27, 2020
* Compile using ref assemblies

* Use updated ResolveReferences.target from AspNetCore

* Generate ref assemblies for .NET framework


Commit migrated from dotnet/extensions@e771c42
@ghost ghost locked as resolved and limited conversation to collaborators May 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
servicing-consider Shiproom approval is required for the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants