Merge branch 'release/4.2.0'

.config/dotnet-tools.json

@@ -3,7 +3,7 @@
   "isRoot": true,
   "tools": {
     "cake.tool": {
-      "version": "4.0.0",
+      "version": "4.1.0",
       "commands": [
         "dotnet-cake"
       ]

README.md

@@ -41,7 +41,7 @@ Cake (C# Make) is a build automation system with a C# DSL to do things like comp
 
 You can read the latest documentation at [https://cakebuild.net/](https://cakebuild.net/).
 
-For a simple example to get started see [Setting up a new project](https://cakebuild.net/docs/getting-started/setting-up-a-new-project).
+For a simple example to get started see [Setting up a new project](https://cakebuild.net/docs/getting-started/setting-up-a-new-scripting-project).
 
 ## Contributing
 

ReleaseNotes.md

@@ -1,3 +1,7 @@
+### New in 4.2.0 (Released 2024/10/23)
+
+* #4374 Argument 'foo' was not set" after update to 4.1 in Cake Frosting.
+
 ### New in 4.1.0 (Released 2024/10/22)
 
 * 4353 Add DotNetListReference alias for dotnet list reference command.

build.cake

@@ -342,7 +342,8 @@ Task("Frosting-Integration-Tests")
 
         DotNetRun(test.Project.FullPath,
             new ProcessArgumentBuilder()
-                .AppendSwitchQuoted("--verbosity", "=", "quiet"),
+                .AppendSwitchQuoted("--verbosity", "=", "quiet")
+                .AppendSwitchQuoted("--name", "=", "world"),
             new DotNetRunSettings
             {
                 Configuration = parameters.Configuration,

src/Cake.Cli/Infrastructure/RemainingArgsParser.cs

@@ -0,0 +1,60 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Cake.Core;
+using Spectre.Console.Cli;
+
+namespace Cake.Cli.Infrastructure
+{
+    /// <summary>
+    /// Spectre.Console <see cref="IRemainingArguments"/> extensions.
+    /// </summary>
+    public static class IRemainingArgumentsExtensions
+    {
+        /// <summary>
+        /// Parses Spectre.Console <see cref="IRemainingArguments"/> to <see cref="CakeArguments"/>.
+        /// </summary>
+        /// <param name="remainingArguments">The remainingArguments.</param>
+        /// <param name="targets">The optional targets, i.e. if specified by command.</param>
+        /// <param name="preProcessArgs">The optional pre-process arguments.</param>
+        /// <returns><see cref="CakeArguments"/>.</returns>
+        public static CakeArguments ToCakeArguments(
+            this IRemainingArguments remainingArguments,
+            string[] targets = null,
+            Action<IDictionary<string, List<string>>> preProcessArgs = null)
+        {
+            var arguments = new Dictionary<string, List<string>>(StringComparer.OrdinalIgnoreCase);
+
+            // Keep the actual remaining arguments in the cake arguments
+            foreach (var group in remainingArguments.Parsed)
+            {
+                string key = group.Key.TrimStart('-');
+                arguments[key] = new List<string>();
+                foreach (var argument in group)
+                {
+                    arguments[key].Add(argument);
+                }
+            }
+
+            // Fixes #3291, We have to add arguments manually which are defined within the DefaultCommandSettings type. Those are not considered "as remaining" because they could be parsed
+            const string targetArgumentName = "target";
+            if (!arguments.ContainsKey(targetArgumentName))
+            {
+                arguments[targetArgumentName] = new List<string>();
+            }
+
+            if (targets != null)
+            {
+                foreach (var target in targets)
+                {
+                    arguments[targetArgumentName].Add(target);
+                }
+            }
+
+            preProcessArgs?.Invoke(arguments);
+
+            var argumentLookUp = arguments.SelectMany(a => a.Value, Tuple.Create).ToLookup(a => a.Item1.Key, a => a.Item2);
+            return new CakeArguments(argumentLookUp);
+        }
+    }
+}

src/Cake.Frosting/Internal/Commands/DefaultCommand.cs

@@ -6,6 +6,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using Cake.Cli;
+using Cake.Cli.Infrastructure;
 using Cake.Core;
 using Cake.Core.Diagnostics;
 using Cake.Core.IO;
@@ -81,31 +82,7 @@ public override int Execute(CommandContext context, DefaultCommandSettings setti
 
         private static CakeArguments CreateCakeArguments(IRemainingArguments remainingArguments, DefaultCommandSettings settings)
         {
-            var arguments = new Dictionary<string, List<string>>(StringComparer.OrdinalIgnoreCase);
-
-            // Keep the actual remaining arguments in the cake arguments
-            foreach (var group in remainingArguments.Parsed)
-            {
-                arguments[group.Key] = new List<string>();
-                foreach (var argument in group)
-                {
-                    arguments[group.Key].Add(argument);
-                }
-            }
-
-            // Fixes #3291, We have to add arguments manually which are defined within the DefaultCommandSettings type. Those are not considered "as remaining" because they could be parsed
-            const string targetArgumentName = "target";
-            if (!arguments.ContainsKey(targetArgumentName))
-            {
-                arguments[targetArgumentName] = new List<string>();
-            }
-            foreach (var target in settings.Targets)
-            {
-                arguments[targetArgumentName].Add(target);
-            }
-
-            var argumentLookUp = arguments.SelectMany(a => a.Value, Tuple.Create).ToLookup(a => a.Item1.Key, a => a.Item2);
-            return new CakeArguments(argumentLookUp);
+            return remainingArguments.ToCakeArguments(settings.Targets);
         }
 
         private void InstallTools(ServiceProvider provider)

src/Cake/Commands/DefaultCommand.cs

@@ -6,6 +6,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using Cake.Cli;
+using Cake.Cli.Infrastructure;
 using Cake.Core;
 using Cake.Core.Diagnostics;
 using Cake.Features.Bootstrapping;
@@ -124,29 +125,17 @@ private int PerformBootstrapping(CommandContext context, DefaultCommandSettings
 
         private static CakeArguments CreateCakeArguments(IRemainingArguments remainingArguments, DefaultCommandSettings settings)
         {
-            var arguments = new Dictionary<string, List<string>>(StringComparer.OrdinalIgnoreCase);
-
-            // Keep the actual remaining arguments in the cake arguments
-            foreach (var group in remainingArguments.Parsed)
-            {
-                string key = group.Key.TrimStart('-');
-                arguments[key] = new List<string>();
-                foreach (var argument in group)
+            return remainingArguments.ToCakeArguments(
+                preProcessArgs: arguments =>
                 {
-                    arguments[key].Add(argument);
-                }
-            }
-
-            // Fixes #4157, We have to add arguments manually which are defined within the DefaultCommandSettings type. Those are not considered "as remaining" because they could be parsed
-            const string recompileArgumentName = Infrastructure.Constants.Cache.InvalidateScriptCache;
-            if (settings.Recompile && !arguments.ContainsKey(recompileArgumentName))
-            {
-                arguments[recompileArgumentName] = new List<string>();
-                arguments[recompileArgumentName].Add(true.ToString());
-            }
-
-            var argumentLookUp = arguments.SelectMany(a => a.Value, Tuple.Create).ToLookup(a => a.Item1.Key, a => a.Item2);
-            return new CakeArguments(argumentLookUp);
+                    // Fixes #4157, We have to add arguments manually which are defined within the DefaultCommandSettings type. Those are not considered "as remaining" because they could be parsed
+                    const string recompileArgumentName = Infrastructure.Constants.Cache.InvalidateScriptCache;
+                    if (settings.Recompile && !arguments.ContainsKey(recompileArgumentName))
+                    {
+                        arguments[recompileArgumentName] = new List<string>();
+                        arguments[recompileArgumentName].Add(true.ToString());
+                    }
+                });
         }
     }
 }

src/SolutionInfo.cs

@@ -10,7 +10,7 @@
 using System.Reflection;
 
 [assembly: AssemblyProduct("Cake")]
-[assembly: AssemblyVersion("4.1.0.0")]
-[assembly: AssemblyFileVersion("4.1.0.0")]
-[assembly: AssemblyInformationalVersion("4.1.0-beta.1+0.Branch.release-4.1.0.Sha.a19ff93e1383ef8bc2fe0a1885a3fc011c60242e")]
+[assembly: AssemblyVersion("4.2.0.0")]
+[assembly: AssemblyFileVersion("4.2.0.0")]
+[assembly: AssemblyInformationalVersion("4.2.0-beta.1+0.Branch.release-4.2.0.Sha.13fe6c38b31cfe8133e4cbfefc8518aedba42996")]
 [assembly: AssemblyCopyright("Copyright (c) .NET Foundation and Contributors")]

tests/integration/Cake.Frosting/build/Tasks/Hello.cs

@@ -1,3 +1,4 @@
+using Cake.Common;
 using Cake.Common.Diagnostics;
 using Cake.Frosting;
 
@@ -6,6 +7,6 @@ public sealed class Hello : FrostingTask<Context>
 {
     public override void Run(Context context)
     {
-        context.Information("Hello World");
+        context.Information("Hello {0}", context.Argument<string>("name"));
     }
 }