[SWBCore] Enhance specs tools quality (swiftlang#59)

MojtabaHs · web-flow · commit ed8a8f990b06 · 2025-02-03T17:06:31.000-08:00
* docs: fix `command` spelling

* docs: fix `Artificially` spelling

* docs: fix `environment` spelling

* docs: fix `accommodate` spelling

* docs: fix `corresponding` spelling

* docs: fix `suppress` spelling
diff --git a/Sources/SWBCore/Specs/Tools/CCompiler.swift b/Sources/SWBCore/Specs/Tools/CCompiler.swift
@@ -550,7 +550,7 @@ public class ClangCompilerSpec : CompilerSpec, SpecIdentifierType, GCCCompatible
             /// The compilation inputs implied by these flags.
             let inputs: [Path]
 
-            /// Maps response files in `flags` to the correspoding recorded attachment in the build description.
+            /// Maps response files in `flags` to the corresponding recorded attachment in the build description.
             let responseFileMapping: [Path: Path]
 
         }
@@ -936,7 +936,7 @@ public class ClangCompilerSpec : CompilerSpec, SpecIdentifierType, GCCCompatible
                     }
                 }
 
-                // If an open-source Swift toolchain is in use, supress warnings about libclang mismatch.
+                // If an open-source Swift toolchain is in use, suppress warnings about libclang mismatch.
                 if !cbc.producer.toolchains.contains(where: { toolchain in
                     toolchain.identifier.hasPrefix("org.swift.")
                 }) {
diff --git a/Sources/SWBCore/Specs/Tools/CodeSign.swift b/Sources/SWBCore/Specs/Tools/CodeSign.swift
@@ -223,7 +223,7 @@ public final class CodesignToolSpec : CommandLineToolSpec, SpecIdentifierType {
             if isReSignTask {
                 // We have to infer the binary path, which is something of a hack.  It also assumes the bundle we're signing *has* a binary.
                 //
-                // Note that the difference between use of `productToSign` and `outputPath` here is intentional, and is designed to accomodate the handling of macOS `Versions/A/<name>`.
+                // Note that the difference between use of `productToSign` and `outputPath` here is intentional, and is designed to accommodate the handling of macOS `Versions/A/<name>`.
                 let bundleName = productToSign.basenameWithoutSuffix
 
                 let format: BundleFormat
@@ -319,7 +319,7 @@ public final class CodesignToolSpec : CommandLineToolSpec, SpecIdentifierType {
     }
 
 
-    /// Computes the enviornment for invoking the code signing tool.
+    /// Computes the environment for invoking the code signing tool.
     ///
     /// - Parameters:
     ///   - cbc: The command build context.
diff --git a/Sources/SWBCore/Specs/Tools/SetAttributes.swift b/Sources/SWBCore/Specs/Tools/SetAttributes.swift
@@ -72,7 +72,7 @@ public final class SetAttributesSpec: CommandLineToolSpec, SpecImplementationTyp
             let args = [cbc.scope.evaluate(BuiltinMacros.CHMOD).str, flags, mode, input.absolutePath.str]
             let outputs: [any PlannedNode] = [delegate.createNode(input.absolutePath), delegate.createVirtualNode("SetMode \(input.absolutePath.str)")]
 
-            // Artifically enforce an ordering between chown and chmod, if both are used.
+            // Artificially enforce an ordering between chown and chmod, if both are used.
             var inputs: [any PlannedNode] = [delegate.createNode(input.absolutePath)]
             if let chownOutput = chownOutput {
                 inputs.append(chownOutput)
diff --git a/Sources/SWBCore/Specs/Tools/TAPISymbolExtractor.swift b/Sources/SWBCore/Specs/Tools/TAPISymbolExtractor.swift
@@ -54,7 +54,7 @@ final public class TAPISymbolExtractor: GenericCompilerSpec, GCCCompatibleCompil
         return await hasPlusPlusHeaders(cbc)
     }
 
-    // Which -x option should this task pass along to clang for the extract-api commnand?
+    // Which -x option should this task pass along to clang for the extract-api command?
     static private func clangHeaderOption(cbc: CommandBuildContext) async -> String {
         return await shouldBuildInCXXMode(cbc: cbc) ? "objective-c++-header" : "objective-c-header"
     }

Original file line number	Diff line number	Diff line change
`@@ -550,7 +550,7 @@ public class ClangCompilerSpec : CompilerSpec, SpecIdentifierType, GCCCompatible`
`550`	`550`	`/// The compilation inputs implied by these flags.`
`551`	`551`	`let inputs: [Path]`
`552`	`552`
`553`		- /// Maps response files in `flags` to the correspoding recorded attachment in the build description.
	`553`	+ /// Maps response files in `flags` to the corresponding recorded attachment in the build description.
`554`	`554`	`let responseFileMapping: [Path: Path]`
`555`	`555`
`556`	`556`	`}`
`@@ -936,7 +936,7 @@ public class ClangCompilerSpec : CompilerSpec, SpecIdentifierType, GCCCompatible`
`936`	`936`	`}`
`937`	`937`	`}`
`938`	`938`
`939`		`- // If an open-source Swift toolchain is in use, supress warnings about libclang mismatch.`
	`939`	`+ // If an open-source Swift toolchain is in use, suppress warnings about libclang mismatch.`
`940`	`940`	`if !cbc.producer.toolchains.contains(where: { toolchain in`
`941`	`941`	`toolchain.identifier.hasPrefix("org.swift.")`
`942`	`942`	`}) {`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ final public class TAPISymbolExtractor: GenericCompilerSpec, GCCCompatibleCompil`
`54`	`54`	`return await hasPlusPlusHeaders(cbc)`
`55`	`55`	`}`
`56`	`56`
`57`		`- // Which -x option should this task pass along to clang for the extract-api commnand?`
	`57`	`+ // Which -x option should this task pass along to clang for the extract-api command?`
`58`	`58`	`static private func clangHeaderOption(cbc: CommandBuildContext) async -> String {`
`59`	`59`	`return await shouldBuildInCXXMode(cbc: cbc) ? "objective-c++-header" : "objective-c-header"`
`60`	`60`	`}`