Drop switch --force|-f

Unknown engines are then skipped with a message,
rather than leading to an error.

CHANGELOG.md

@@ -10,6 +10,10 @@ this project uses date-based 'snapshot' version identifiers.
 ### Added
 - Switch `-s|--stdengine` to run a set of tests only with the standard engine
   even where this varies between configs (issue \#343)
+
+### Removed
+- Switch `--force|-f`
+
 ## [2024-01-09]
 
 ### Fixed

l3build-arguments.lua

@@ -89,12 +89,6 @@ option_list =
         desc  = "Name of first test to run",
         type  = "string"
       },
-    force =
-      {
-        desc  = "Forces tests to run, even if engine is not set up",
-        short = "f",
-        type  = "boolean"
-      },
     full =
       {
         desc = "Installs all files",
@@ -295,25 +289,3 @@ local function argparse()
 end
 
 options = argparse()
-
--- Sanity check
-function check_engines(config)
-  if options["engine"] and not options["force"] then
-    -- Make a lookup table
-    local t = { }
-    for _, engine in pairs(checkengines) do
-      t[engine] = true
-    end
-    for _, engine in pairs(options["engine"]) do
-      if not t[engine] then
-        print("\n! Error: Engine \"" .. engine .. "\" not set up for testing with configuration \"" .. config .. "\"!")
-        print("\n  Valid values are:")
-        for _, engine in ipairs(checkengines) do
-          print("  - " .. engine)
-        end
-        print("")
-        exit(1)
-      end
-    end
-  end
-end

l3build-check.lua

@@ -598,7 +598,18 @@ function runcheck(name, hide)
   if options["stdengine"] then
     checkengines = {stdengine}
   elseif options["engine"] then
-    checkengines = options["engine"]
+    local t = {}
+    for _,engine in ipairs(checkengines) do
+      t[engine] = true
+    end
+    checkengines = {}
+    for _,engine in ipairs(options["engine"]) do
+      if t[engine] then
+        insert(checkengines,engine)
+      else
+        print("Skipping unknown engine " .. engine)
+      end
+    end
   end
   local failedengines = {}
   -- Used for both .lvt and .pvt tests

l3build.1

@@ -55,8 +55,6 @@ Sets the epoch for tests and typesetting
 Takes the upload announcement from the given file
 .IP --first
 Name of first test to run
-.IP --force|-f
-Force tests to run if engine is not set up
 .IP --full
 Installs all files
 .IP --halt-on-error|-H

l3build.dtx

@@ -379,9 +379,6 @@
 % \item |--epoch| Sets the epoch for typesetting and testing
 % \item |--file| (|-F|) Takes the upload announcement from the given file
 % \item |--first| Name of the first test to run
-% \item |--force| (|-f|) Forces checks to run even if sanity
-%   checks fail, \emph{e.g.}~when |--engine| is not given in
-%   \var{checkengines}
 % \item |--full| Instructs the \texttt{install} target to include the
 %   \texttt{doc} and \texttt{source} trees
 % \item |--halt-on-error| (|-H|) Specifies that checks
@@ -1676,7 +1673,6 @@
 %       \var{epoch}         & String  \\
 %       \var{file}          & string  \\
 %       \var{first}         & Boolean \\
-%       \var{force}         & Boolean \\
 %       \var{full}          & Boolean \\
 %       \var{halt-on-error} & Boolean \\
 %       \var{help}          & Boolean \\

l3build.lua

@@ -208,17 +208,12 @@ if #checkconfigs > 1 then
 end
 if #checkconfigs == 1 and
   (options["target"] == "check" or options["target"] == "save" or options["target"] == "clean") then
-  if checkconfigs[1] == "build" then
-    -- Sanity check for default config
-    check_engines("build.lua")
-  else
+  if not checkconfigs[1] == "build" then
     local configname  = gsub(checkconfigs[1], "%.lua$", "")
     local config = "./" .. configname .. ".lua"
     if fileexists(config) then
       local savedtestfiledir = testfiledir
       dofile(config)
-      -- Sanity check for non-default config
-      check_engines(configname .. ".lua")
       testdir = testdir .. "-" .. configname
       -- Reset testsuppdir if required
       if savedtestfiledir ~= testfiledir and