-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Upgrade global.json to 5.0-alpha1 SDK #445
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
Changes from all commits
8ef6001
a1aef8e
8656983
15343f6
70636c9
ea85fd3
81d8b94
8678bcd
0b0d86a
4e4ada0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,35 +1,8 @@ | ||
| <Project> | ||
| <PropertyGroup> | ||
| <TargetFramework>netcoreapp5.0</TargetFramework> | ||
| <NETCoreAppMaximumVersion>5.0</NETCoreAppMaximumVersion> | ||
| <RuntimeFrameworkVersion>$(MicrosoftNETCoreAppVersion)</RuntimeFrameworkVersion> | ||
| <!-- This should be enabled once we have updating assembly versions in our targeting pack. --> | ||
| <ShouldVerifyClosure>false</ShouldVerifyClosure> | ||
| </PropertyGroup> | ||
|
|
||
| <ItemGroup> | ||
| <!-- Since we don't have an SDK that knows about 5.0 framework, we should specify it manually. --> | ||
| <KnownFrameworkReference Update="Microsoft.NETCore.App" | ||
| TargetFramework="$(TargetFramework)" | ||
| RuntimeFrameworkName="Microsoft.NETCore.App" | ||
| DefaultRuntimeFrameworkVersion="$(RuntimeFrameworkVersion)" | ||
| LatestRuntimeFrameworkVersion="$(RuntimeFrameworkVersion)" | ||
| TargetingPackName="Microsoft.NETCore.App.Ref" | ||
| TargetingPackVersion="$(RuntimeFrameworkVersion)" | ||
| RuntimePackNamePatterns="Microsoft.NETCore.App.Runtime.**RID**" | ||
| RuntimePackRuntimeIdentifiers="linux-arm;linux-arm64;linux-musl-arm64;linux-musl-x64;linux-x64;osx-x64;rhel.6-x64;tizen.4.0.0-armel;tizen.5.0.0-armel;win-arm;win-arm64;win-x64;win-x86" | ||
| PackagesToReference="Microsoft.NETCore.App/$(RuntimeFrameworkVersion)" | ||
| IsTrimmable="true" | ||
| /> | ||
|
|
||
| <KnownAppHostPack Update="Microsoft.NETCore.App" | ||
| TargetFramework="$(TargetFramework)" | ||
| AppHostPackNamePattern="Microsoft.NETCore.App.Host.**RID**" | ||
| AppHostPackVersion="$(RuntimeFrameworkVersion)" | ||
| AppHostRuntimeIdentifiers="linux-arm;linux-arm64;linux-musl-arm64;linux-musl-x64;linux-x64;osx-x64;rhel.6-x64;tizen.4.0.0-armel;tizen.5.0.0-armel;win-arm;win-arm64;win-x64;win-x86" | ||
| /> | ||
|
|
||
| <KnownFrameworkReference Remove="Microsoft.AspNetCore.App" /> | ||
| <KnownFrameworkReference Remove="Microsoft.WindowsDesktop.App" /> | ||
| <KnownFrameworkReference Remove="Microsoft.WindowsDesktop.App.WPF" /> | ||
| <KnownFrameworkReference Remove="Microsoft.WindowsDesktop.App.WindowsForms" /> | ||
| </ItemGroup> | ||
| </Project> |
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -11,6 +11,17 @@ | |||
| <PackageReference Update="Microsoft.NETCore.App" IsImplicitlyDefined="false" PrivateAssets="None" /> | ||||
| </ItemGroup> | ||||
|
|
||||
| <!-- Mark framework references from previous netcoreapp tfms >= 3.0 as copy-local. --> | ||||
| <Target Name="_FrameworkReferencesToPrivate" | ||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Instead I’d just do this for all items in depproj targets.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you elaborate? Not sure I follow. Do you mean to actually Define an ItemDefintionGroup for reference?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you mean you would do it for all targets, I would rather not as we include netstandard2.0 here as well which just creates noise in the ref folder, ie dotnet-XmlGenerator.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I meant that the purpose of DepProj is to copy. It flows these items to content here: Line 95 in 8f67999
This is effectively doing the same thing you're trying to do (force to copy). If that wasn't working for framework references we should fix it there.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Actually I investigated why this wasn't working with the depproj/binplacing logic. I will submit a follow-up PR when I have some cycles to fix it there. |
||||
| Condition="'$(TargetsNetCoreApp)' == 'true' and !$(TargetFramework.StartsWith('netcoreapp2.')) and '$(TargetFramework)' != '$(NetCoreAppCurrent)'" | ||||
| AfterTargets="_HandlePackageFileConflicts"> | ||||
| <ItemGroup> | ||||
| <_referencePrivate Include="@(Reference)" Private="true" /> | ||||
| <Reference Remove="@(Reference)" /> | ||||
| <Reference Include="@(_referencePrivate)" /> | ||||
| </ItemGroup> | ||||
| </Target> | ||||
|
|
||||
| <ItemGroup> | ||||
| <!-- for all configurations this project provides refs for that configuration --> | ||||
| <BinPlaceConfiguration Include="$(Configuration)" RefPath="$(RefPath)" /> | ||||
|
|
||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/cc @Anipik