AdamsLair / duality

a 2D Game Development Framework
https://adamslair.github.io/duality
MIT License
1.41k stars 290 forks source link

Feature/#698 csharp80 upgrade #726

Closed Barsonax closed 5 years ago

Barsonax commented 5 years ago

Note that this does not yet enable nullable references. For that the following line needs to be added to the csproj's:

<NullableContextOptions>enable</NullableContextOptions>

The above could be done in a separate PR though since its also a different issue.

EDIT: Its now changed to:

<Nullable>enable</Nullable>
Barsonax commented 5 years ago

Solves #698

Barsonax commented 5 years ago

Seems we are getting a error in the appveyor build:

C:\projects\duality\Source\Core\Duality\Duality.csproj(479,11): error MSB4226: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Microsoft\Portable\v4.5\Microsoft.Portable.CSharp.targets" was not found

I believe this is because the VS2019 does not have the portable targeting pack installed. It should work if we install that. Locally it works atleast.

Barsonax commented 5 years ago

Jup portable is not even included anymore by default: https://stackoverflow.com/questions/54861759/build-of-pcl-lib-fails-with-new-vs2019-agent

So we have 3 options:

ilexp commented 5 years ago

I think we should take this opportunity to attempt option three and switch to .NET Standard first. If that turns out to be too massive, we can explore the other options. Let's keep this PR around for later, but continue discussion on #573.

ilexp commented 5 years ago

Needs conflict resolve after recent master upgrade to .NET 4.7.2 / C# 7.3 via #698

ilexp commented 5 years ago

Found out that we can't use C# 8.0 until we've also upgrade to .NET Core 3.0. Moved all related issues to a separate milestone so we can focus on .NET Standard 2.0 and C# 7.3 right now. Closing this.