NEW (Extension) @W-15639920@ CLI changes to pass java env variable to…

… enable caching and path for delta runs - part 2
forcedotcom · Sep 5, 2024 · 1ab7f49 · 1ab7f49
1 parent 1f75111
commit 1ab7f49
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/lib/EngineOptionsFactory.ts b/src/lib/EngineOptionsFactory.ts
@@ -29,8 +29,8 @@ abstract class CommonEngineOptionsFactory implements EngineOptionsFactory {
 		if (this.shouldSfgeRun(inputs)) {
 			const sfgeConfig: SfgeConfig = {
 				projectDirs: this.inputProcessor.resolveProjectDirPaths(inputs),
-				cachepath: inputs.cachepath,
-				enablecaching: inputs.enablecaching
+				cachepath: inputs.cachepath as string,
+				enablecaching: inputs.enablecaching as boolean
 			};
 			options.set(CUSTOM_CONFIG.SfgeConfig, JSON.stringify(sfgeConfig));
 		}

diff --git a/src/lib/sfge/SfgeWrapper.ts b/src/lib/sfge/SfgeWrapper.ts
@@ -242,7 +242,7 @@ export class SfgeExecuteWrapper extends AbstractSfgeWrapper {
 		if (this.ruleDisableWarningViolation != null) {
 			flags.push(`-DSFGE_RULE_DISABLE_WARNING_VIOLATION=${this.ruleDisableWarningViolation.toString()}`);
 		}
-		if (this.enablecaching) {
+		if (this.enablecaching != null && this.enablecaching) {
 			flags.push(`-DSFGE_DISABLE_CACHING=false`);
 		}
 		if (this.cachepath != null) {