Typo cleanup

This commit is contained in:
Bartosz Polaczyk
2022-05-10 21:31:34 +02:00
parent c39b286222
commit 3dfd6e296f
3 changed files with 8 additions and 7 deletions
+1 -1
View File
@@ -315,7 +315,7 @@ _Note that for the `producer` mode, the prebuild build phase and `xccc`, `xcld`,
| `disable_certificate_verification` | A Boolean value that opts-in SSL certificate validation is disabled | `false` | ⬜️ |
| `disable_vfs_overlay` | A feature flag to disable virtual file system overlay support (temporary) | `false` | ⬜️ |
| `custom_rewrite_envs` | A list of extra ENVs that should be used as placeholders in the dependency list. ENV rewrite process is optimistic - does nothing if an ENV is not defined in the pre/postbuild process. | `[]` | ⬜️ |
| `irrelevant_dependencies_paths` | Regexes of files that should not be included in a list of dependencies. Warning! Be caution when adding entries here - excluding relevant dependencies might lead to targets overcaching. The regex can match either partially of fully the filepath, e.g. `\\.modulemap$` will exclude all `.modulemap` files.. | `[]` | ⬜️ |
| `irrelevant_dependencies_paths` | Regexes of files that should not be included in a list of dependencies. Warning! Add entries here with caution - excluding dependencies that are relevant might lead to a target overcaching. The regex can match either partially or fully the filepath, e.g. `\\.modulemap$` will exclude all `.modulemap` files.. | `[]` | ⬜️ |
## Backend cache server
@@ -138,9 +138,9 @@ public struct XCRemoteCacheConfig: Encodable {
/// A list of extra ENVs that should be used as placeholders in the dependency list
/// ENV rewrite process is optimistic - does nothing if an ENV is not defined in the pre/postbuild process
var customRewriteEnvs: [String] = []
/// Regexes of files that should not be included in a list of dependencies. Warning! Be caution when adding
/// entries here - excluding relevant dependencies might lead to targets overcaching
/// The regex can match either partially of fully the filepath, e.g. `\\.modulemap$` will exclude
/// Regexes of files that should not be included in a list of dependencies. Warning! Add entries here
/// with caution - excluding dependencies that are relevant might lead to a target overcaching
/// Note: The regex can match either partially or fully the filepath, e.g. `\\.modulemap$` will exclude
/// all `.modulemap` files
var irrelevantDependenciesPaths: [String] = []
}
@@ -31,7 +31,7 @@ public struct Dependency: Equatable {
// Product of the target itself
case ownProduct
// User-excluded path
case customSkipped
case userExcluded
case unknown
}
@@ -81,7 +81,7 @@ class DependencyProcessorImpl: DependencyProcessor {
return files.map { file -> Dependency in
let filePath = file.resolvingSymlinksInPath().path
if skippedRegexes.contains(where: { filePath.range(of: $0, options: .regularExpression) != nil }) {
return Dependency(url: file, type: .customSkipped)
return Dependency(url: file, type: .userExcluded)
} else if filePath.hasPrefix(xcodePath) {
return Dependency(url: file, type: .xcode)
} else if filePath.hasPrefix(intermediatePath) {
@@ -120,8 +120,9 @@ class DependencyProcessorImpl: DependencyProcessor {
// because in case of a hit, these will be taken from the artifact
// - Customized DERIVED_FILE_DIR may change a directory of
// derived files, which by default is under `*/Interemediates`
// - User-specified (in .rcinfo) files to exclude
let irrelevantDependenciesType: [Dependency.Kind] = [
.xcode, .intermediate, .ownProduct, .derivedFile, .customSkipped,
.xcode, .intermediate, .ownProduct, .derivedFile, .userExcluded,
]
return !irrelevantDependenciesType.contains(dependency.type)
}