Update relevant parts to net8.0

cincuranet · cincuranet · commit 6c96b0f2d44d · 2024-01-18T10:44:18.000+01:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -19,10 +19,10 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
         
-      - name: .NET 7.0
+      - name: .NET 8.0
         uses: actions/setup-dotnet@v3
         with:
-          dotnet-version: 7.0.x
+          dotnet-version: 8.0.x
 
       - name: Build
         run: |
diff --git a/include.ps1 b/include.ps1
@@ -7,5 +7,5 @@ function Check-ExitCode() {
 }
 
 function Get-UsedTargetFramework() {
-	return 'net7.0'
+	return 'net8.0'
 }
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -27,7 +27,7 @@
 		<EmbedUntrackedSources>true</EmbedUntrackedSources>
 	</PropertyGroup>
 	<ItemGroup Condition="'$(SkipSourceLink)'!='true'">
-		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1">
+		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0">
 			<PrivateAssets>all</PrivateAssets>
 			<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
 		</PackageReference>
diff --git a/src/EntityFramework.Firebird.Tests/EntityFramework.Firebird.Tests.csproj b/src/EntityFramework.Firebird.Tests/EntityFramework.Firebird.Tests.csproj
@@ -1,7 +1,6 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<!-- Figure out a way to run it on net4.8 (FirebirdSql.Data.TestsBase uses IAsyncDisposable) -->
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<GenerateAssemblyInfo>false</GenerateAssemblyInfo>
 		<GeneratePackageOnBuild>false</GeneratePackageOnBuild>
 		<SkipSourceLink>true</SkipSourceLink>
@@ -12,7 +11,8 @@
 		<OutputType>Exe</OutputType>
 		<StartupObject>FirebirdSql.Data.TestsBase.Program</StartupObject>
 	</PropertyGroup>
-	<ItemGroup Condition="'$(TargetFramework)'=='net7.0'">
+	<ItemGroup Condition="'$(TargetFramework)'=='net8.0'">
+		<!-- left in repo as an example/documentation for .NET Framework -->
 		<None Remove="app.config" />
 	</ItemGroup>
 	<ItemGroup>
diff --git a/src/EntityFramework.Firebird.Tests/EntityFrameworkTestsBase.cs b/src/EntityFramework.Firebird.Tests/EntityFrameworkTestsBase.cs
@@ -28,6 +28,7 @@ public abstract class EntityFrameworkTestsBase : FbTestsBase
 	static EntityFrameworkTestsBase()
 	{
 #if !NETFRAMEWORK
+		// example/documentation for .NET Framework
 		System.Data.Common.DbProviderFactories.RegisterFactory(FbProviderServices.ProviderInvariantName, FirebirdClientFactory.Instance);
 #endif
 		DbConfiguration.SetConfiguration(new FbTestDbContext.Conf());
diff --git a/src/FirebirdSql.Data.FirebirdClient.Tests/FirebirdSql.Data.FirebirdClient.Tests.csproj b/src/FirebirdSql.Data.FirebirdClient.Tests/FirebirdSql.Data.FirebirdClient.Tests.csproj
@@ -1,6 +1,6 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<GenerateAssemblyInfo>false</GenerateAssemblyInfo>
 		<GeneratePackageOnBuild>false</GeneratePackageOnBuild>
 		<SkipSourceLink>true</SkipSourceLink>
diff --git a/src/FirebirdSql.Data.FirebirdClient/Common/IscException.cs b/src/FirebirdSql.Data.FirebirdClient/Common/IscException.cs
@@ -27,7 +27,6 @@
 
 namespace FirebirdSql.Data.Common;
 
-[Serializable]
 internal sealed class IscException : Exception
 {
 	private string _message;
@@ -117,27 +116,13 @@ public static IscException ForIOException(IOException exception)
 		return ForErrorCodes(new[] { IscCodes.isc_net_write_err, IscCodes.isc_net_read_err }, exception);
 	}
 
-	private IscException(SerializationInfo info, StreamingContext context)
-			: base(info, context)
-	{
-		Errors = (List<IscError>)info.GetValue(nameof(Errors), typeof(List<IscError>));
-		ErrorCode = info.GetInt32(nameof(ErrorCode));
-	}
-
 	public void BuildExceptionData()
 	{
 		BuildErrorCode();
 		BuildSqlState();
 		BuildExceptionMessage();
 	}
 
-	public override void GetObjectData(SerializationInfo info, StreamingContext context)
-	{
-		base.GetObjectData(info, context);
-		info.AddValue(nameof(Errors), Errors);
-		info.AddValue(nameof(ErrorCode), ErrorCode);
-	}
-
 	private void BuildErrorCode()
 	{
 		ErrorCode = Errors.Count != 0 ? Errors[0].ErrorCode : 0;
diff --git a/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbException.cs b/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbException.cs
@@ -24,7 +24,9 @@
 
 namespace FirebirdSql.Data.FirebirdClient;
 
+#if !NET8_0_OR_GREATER
 [Serializable]
+#endif
 public sealed class FbException : DbException
 {
 	#region Fields
@@ -50,22 +52,26 @@ private FbException(string message, Exception innerException)
 		: base(message, innerException)
 	{ }
 
+#if !NET8_0_OR_GREATER
 	private FbException(SerializationInfo info, StreamingContext context)
 		: base(info, context)
 	{
 		_errors = (FbErrorCollection)info.GetValue("errors", typeof(FbErrorCollection));
 	}
+#endif
 
 	#endregion
 
 	#region Methods
 
+#if !NET8_0_OR_GREATER
 	public override void GetObjectData(SerializationInfo info, StreamingContext context)
 	{
 		base.GetObjectData(info, context);
 
 		info.AddValue("errors", _errors);
 	}
+#endif
 
 	#endregion
 
diff --git a/src/FirebirdSql.Data.FirebirdClient/FirebirdSql.Data.FirebirdClient.csproj b/src/FirebirdSql.Data.FirebirdClient/FirebirdSql.Data.FirebirdClient.csproj
@@ -1,6 +1,6 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<TargetFrameworks>net48;netstandard2.0;netstandard2.1;net5.0;net6.0;net7.0</TargetFrameworks>
+		<TargetFrameworks>net48;netstandard2.0;netstandard2.1;net5.0;net6.0;net7.0;net8.0</TargetFrameworks>
 		<AssemblyName>FirebirdSql.Data.FirebirdClient</AssemblyName>
 		<RootNamespace>FirebirdSql.Data</RootNamespace>
 		<SignAssembly>true</SignAssembly>
@@ -53,6 +53,8 @@
 	</ItemGroup>
 	<ItemGroup Condition="'$(TargetFramework)'=='net7.0'">
 	</ItemGroup>
+	<ItemGroup Condition="'$(TargetFramework)'=='net8.0'">
+	</ItemGroup>
 	<ItemGroup>
 	  <PackageReference Include="ConfigureAwaitChecker.Analyzer" Version="5.0.0.1">
 	    <PrivateAssets>all</PrivateAssets>
diff --git a/src/FirebirdSql.EntityFrameworkCore.Firebird.FunctionalTests/FirebirdSql.EntityFrameworkCore.Firebird.FunctionalTests.csproj b/src/FirebirdSql.EntityFrameworkCore.Firebird.FunctionalTests/FirebirdSql.EntityFrameworkCore.Firebird.FunctionalTests.csproj
@@ -1,6 +1,6 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<GenerateAssemblyInfo>false</GenerateAssemblyInfo>
 		<GeneratePackageOnBuild>false</GeneratePackageOnBuild>
 		<SkipSourceLink>true</SkipSourceLink>
diff --git a/src/FirebirdSql.EntityFrameworkCore.Firebird.Tests/FirebirdSql.EntityFrameworkCore.Firebird.Tests.csproj b/src/FirebirdSql.EntityFrameworkCore.Firebird.Tests/FirebirdSql.EntityFrameworkCore.Firebird.Tests.csproj
@@ -1,6 +1,6 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<GenerateAssemblyInfo>false</GenerateAssemblyInfo>
 		<GeneratePackageOnBuild>false</GeneratePackageOnBuild>
 		<SkipSourceLink>true</SkipSourceLink>
diff --git a/src/Perf/Perf.csproj b/src/Perf/Perf.csproj
@@ -1,7 +1,7 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
 		<OutputType>Exe</OutputType>
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<SkipSourceLink>true</SkipSourceLink>
 	</PropertyGroup>
 	<ItemGroup>
diff --git a/src/Scratchpad/Scratchpad.csproj b/src/Scratchpad/Scratchpad.csproj
@@ -1,7 +1,7 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 	<PropertyGroup>
 		<OutputType>Exe</OutputType>
-		<TargetFramework>net7.0</TargetFramework>
+		<TargetFramework>net8.0</TargetFramework>
 		<SkipSourceLink>true</SkipSourceLink>
 		<SignAssembly>true</SignAssembly>
 		<AssemblyOriginatorKeyFile>..\FirebirdSql.Data.TestsBase\FirebirdSql.Data.TestsBase.snk</AssemblyOriginatorKeyFile>

Original file line number	Diff line number	Diff line change
`@@ -7,5 +7,5 @@ function Check-ExitCode() {`
`7`	`7`	`}`
`8`	`8`
`9`	`9`	`function Get-UsedTargetFramework() {`
`10`		`- return 'net7.0'`
	`10`	`+ return 'net8.0'`
`11`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ public abstract class EntityFrameworkTestsBase : FbTestsBase`
`28`	`28`	`static EntityFrameworkTestsBase()`
`29`	`29`	`{`
`30`	`30`	`#if !NETFRAMEWORK`
	`31`	`+ // example/documentation for .NET Framework`
`31`	`32`	`System.Data.Common.DbProviderFactories.RegisterFactory(FbProviderServices.ProviderInvariantName, FirebirdClientFactory.Instance);`
`32`	`33`	`#endif`
`33`	`34`	`DbConfiguration.SetConfiguration(new FbTestDbContext.Conf());`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,9 @@`
`24`	`24`
`25`	`25`	`namespace FirebirdSql.Data.FirebirdClient;`
`26`	`26`
	`27`	`+#if !NET8_0_OR_GREATER`
`27`	`28`	`[Serializable]`
	`29`	`+#endif`
`28`	`30`	`public sealed class FbException : DbException`
`29`	`31`	`{`
`30`	`32`	`#region Fields`
`@@ -50,22 +52,26 @@ private FbException(string message, Exception innerException)`
`50`	`52`	`: base(message, innerException)`
`51`	`53`	`{ }`
`52`	`54`
	`55`	`+#if !NET8_0_OR_GREATER`
`53`	`56`	`private FbException(SerializationInfo info, StreamingContext context)`
`54`	`57`	`: base(info, context)`
`55`	`58`	`{`
`56`	`59`	`_errors = (FbErrorCollection)info.GetValue("errors", typeof(FbErrorCollection));`
`57`	`60`	`}`
	`61`	`+#endif`
`58`	`62`
`59`	`63`	`#endregion`
`60`	`64`
`61`	`65`	`#region Methods`
`62`	`66`
	`67`	`+#if !NET8_0_OR_GREATER`
`63`	`68`	`public override void GetObjectData(SerializationInfo info, StreamingContext context)`
`64`	`69`	`{`
`65`	`70`	`base.GetObjectData(info, context);`
`66`	`71`
`67`	`72`	`info.AddValue("errors", _errors);`
`68`	`73`	`}`
	`74`	`+#endif`
`69`	`75`
`70`	`76`	`#endregion`
`71`	`77`