- 
                Notifications
    You must be signed in to change notification settings 
- Fork 715
Update to .NET 10 SDK and Arcade 10 #10075
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
4e99ab0
              0d6587c
              0a0e0c4
              62f6ab5
              6b49d70
              f5a801b
              c5a685f
              5458b21
              d3faa58
              10d4e48
              cce0b4f
              13cb134
              f2367dd
              560722d
              567a538
              261091f
              4f45d40
              4fe6756
              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 | 
|---|---|---|
|  | @@ -31,6 +31,14 @@ | |
| <AdditionalProperties Include="PublishAot=false" /> | ||
| <AdditionalProperties Include="PublishSingleFile=true" /> | ||
| <AdditionalProperties Include="SelfContained=true" /> | ||
|  | ||
| <!-- Windows pdb aren't really needed any more. See https://github.com/dotnet/arcade/issues/15724 | ||
| As the pdb file is in the same location for all the RIDs it causes arcade's `_DeployPortableSymbolsToSymStore` | ||
| 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. Isn't the right fix here to use different publish locations for different rids? 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. Publish directories are separate, and even the build dirs for the .dll but not for the .pdb! The .pdb files wouldn't be different per-rid so I think them being in the same location makes sense. We could fix this in arcade somehow, but apparently this whole thing (windows pdbs) is expected to be removed anyway. See dotnet/arcade#15724 and dotnet/arcade#15736 . | ||
| target to race. | ||
|  | ||
| `The process cannot access the file 'D:\a\_work\1\s\artifacts\SymStore\Release\Aspire.Cli\net8.0\aspire.pdb' because it is being used by another process` | ||
| --> | ||
| <AdditionalProperties Include="PublishWindowsPdb=false" /> | ||
| </ItemGroup> | ||
|  | ||
| <!-- Remove OutputPath, and TargetFramework so the defaults can be used --> | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,2 +1,2 @@ | ||
| @echo off | ||
| powershell -ExecutionPolicy ByPass -NoProfile -command "& """%~dp0Build.ps1""" -restore -build -test -sign -pack -publish -ci %*" | ||
| powershell -ExecutionPolicy ByPass -NoProfile -command "& """%~dp0Build.ps1""" -restore -build -test -sign -pack -publish -ci %*" | 
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.
Why is this not the current version? Should we rename the property instead?
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.
These names are arbitrary really. We are following
Previous, Current, Nextto follow 3 active versions.9.0beingCurrentworks here allowingPrevious=8.0andCurrent=10.0.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.
I would use the same nomenclature as dotnet/runtime:
$(NetCoreAppCurrent);$(NetCoreAppPrevious);$(NetCoreAppMinimum)Current = 10
Previous = 9
Minimum = 8
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.
But maybe that only makes sense if we are on
10as well. So maybe havingNextbe 10 until we are version 10...