Refactoring: splitting the Product class, moving namespaces, fixing accessibility.

Author: gfraiteur

global.json

@@ -1,6 +1,11 @@
 {
+  "//": "This file is generated by PostSharp.Engineering. Do not edit or add to git.",
   "sdk": {
-    "version": "9.0.100",
-    "rollForward": "major"
+    "version": "8.0",
+    "rollForward": "patch",
+    "allowPrerelease": "false"
+  },
+  "msbuild-sdks": {
+    "PostSharp.Engineering.Sdk": "2023.2.197.1191-local-GaelFraiteur-debug"
   }
 }

src/PostSharp.Engineering.BuildTools/AppExtensions.cs

@@ -3,15 +3,22 @@
 using JetBrains.Annotations;
 using PostSharp.Engineering.BuildTools.BillOfMaterials;
 using PostSharp.Engineering.BuildTools.Build;
+using PostSharp.Engineering.BuildTools.Build.Bumping;
 using PostSharp.Engineering.BuildTools.Build.Model;
+using PostSharp.Engineering.BuildTools.Build.MSBuild;
+using PostSharp.Engineering.BuildTools.Build.Publishing;
+using PostSharp.Engineering.BuildTools.Build.Swapping;
+using PostSharp.Engineering.BuildTools.Build.Testing;
 using PostSharp.Engineering.BuildTools.CodeStyle;
 using PostSharp.Engineering.BuildTools.ContinuousIntegration;
-using PostSharp.Engineering.BuildTools.Csproj;
 using PostSharp.Engineering.BuildTools.Dependencies;
 using PostSharp.Engineering.BuildTools.DotNetTools;
-using PostSharp.Engineering.BuildTools.Git;
-using PostSharp.Engineering.BuildTools.NuGet;
-using PostSharp.Engineering.BuildTools.XmlDoc;
+using PostSharp.Engineering.BuildTools.Tools;
+using PostSharp.Engineering.BuildTools.Tools.Csproj;
+using PostSharp.Engineering.BuildTools.Tools.Git;
+using PostSharp.Engineering.BuildTools.Tools.NuGet;
+using PostSharp.Engineering.BuildTools.Tools.TeamCity;
+using PostSharp.Engineering.BuildTools.Tools.XmlDoc;
 using Spectre.Console.Cli;
 using System;
 using System.Linq;
@@ -48,9 +55,9 @@ internal static void AddCommands( this CommandApp app, Product product )
                         .WithData( data )
                         .WithDescription( "Builds all packages in the product (implies 'prepare')" );
 
-                    root.AddCommand<GenerateCiScriptsCommand>( "generate-scripts" )
+                    root.AddCommand<GenerateScriptsCommand>( "generate-scripts" )
                         .WithData( data )
-                        .WithDescription( "Generates the continuous integration scripts" );
+                        .WithDescription( "Generates the CI and Docker scripts" );
 
                     root.AddCommand<ListSolutionsCommand>( "list-solutions" )
                         .WithData( data )

src/PostSharp.Engineering.BuildTools/BaseCommand.cs

@@ -2,13 +2,15 @@
 
 using PostSharp.Engineering.BuildTools.Build;
 using PostSharp.Engineering.BuildTools.Build.Model;
+using PostSharp.Engineering.BuildTools.Build.MSBuild;
 using PostSharp.Engineering.BuildTools.Utilities;
 using Spectre.Console;
 using Spectre.Console.Cli;
 using System;
 using System.Diagnostics;
 using System.IO;
 using System.Linq;
+using System.Runtime.InteropServices;
 using System.Text.Json;
 
 namespace PostSharp.Engineering.BuildTools
@@ -91,7 +93,10 @@ public sealed override int Execute( CommandContext context, T settings )
                             .Color( Color.Purple ) );
 
                     buildContext.Console.Out.WriteLine();
-                    buildContext.Console.WriteMessage( $"Using PostSharp.Engineering v{myVersion}. TeamCity: {buildContext.IsContinuousIntegrationBuild}. Docker: {buildContext.IsRunningUnderContainer}." );
+
+                    buildContext.Console.WriteMessage(
+                        $"Using PostSharp.Engineering v{myVersion}. TeamCity: {buildContext.IsContinuousIntegrationBuild}. Docker: {buildContext.IsRunningUnderContainer}. Runtime: {RuntimeInformation.FrameworkDescription} {RuntimeInformation.RuntimeIdentifier}." );
+
                     buildContext.Console.Out.WriteLine();
                 }
 

src/PostSharp.Engineering.BuildTools/BillOfMaterials/DependencyWalker.cs

@@ -2,6 +2,7 @@
 
 using Microsoft.Extensions.FileSystemGlobbing;
 using PostSharp.Engineering.BuildTools.Build;
+using PostSharp.Engineering.BuildTools.Build.Files;
 using PostSharp.Engineering.BuildTools.Build.Model;
 using System.Net;
 using System.Text.RegularExpressions;
@@ -41,7 +42,7 @@ internal static class DependencyWalker
 
     private static IReadOnlyList<PackageDependencyInfo> GetPackageDependencies( BuildContext context )
     {
-        var defaultConfiguration = context.Product.ReadDefaultConfiguration( context ) ?? BuildConfiguration.Debug;
+        var defaultConfiguration = ConfigurationNeutralVersionFile.ReadDefaultConfiguration( context ) ?? BuildConfiguration.Debug;
 
         var list = new List<PackageDependencyInfo>();
 

src/PostSharp.Engineering.BuildTools/BillOfMaterials/GenerateThirdPartyNoticesCommand.cs

@@ -2,9 +2,7 @@
 
 using JetBrains.Annotations;
 using PostSharp.Engineering.BuildTools.Build;
-using PostSharp.Engineering.BuildTools.Build.Model;
 using System;
-using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 

src/PostSharp.Engineering.BuildTools/Build/BaseBuildSettings.cs

@@ -1,6 +1,8 @@
 // Copyright (c) SharpCrafters s.r.o. See the LICENSE.md file in the root directory of this repository root for details.
 
 using JetBrains.Annotations;
+using PostSharp.Engineering.BuildTools.Build.Files;
+using PostSharp.Engineering.BuildTools.Build.Publishing;
 using Spectre.Console.Cli;
 using System;
 using System.ComponentModel;
@@ -55,7 +57,7 @@ public override void Initialize( BuildContext context )
             return;
         }
 
-        var defaultConfiguration = context.Product.ReadDefaultConfiguration( context );
+        var defaultConfiguration = ConfigurationNeutralVersionFile.ReadDefaultConfiguration( context );
 
         if ( defaultConfiguration == null )
         {