No need to chmod on Windows

Sam Eagen · Sam Eagen · commit 6a47b68266c6 · 2024-08-28T14:48:25.000-04:00
diff --git a/tasks/install-matlab/v1/src/matlab.ts b/tasks/install-matlab/v1/src/matlab.ts
@@ -144,11 +144,12 @@ export async function setupBatch(platform: string, architecture: string) {
     } catch (err: any) {
         throw new Error("Failed to add MATLAB to system path.");
     }
-    const exitCode = await taskLib.exec("chmod", ["+x", path.join(matlabBatchPath, "matlab-batch" + matlabBatchExt)]);
-    if (exitCode !== 0) {
-        return Promise.reject(Error("Unable to add execute permissions to matlab-batch binary."));
+    if (platform !== "win32") {
+        const exitCode = await taskLib.exec("chmod", ["+x", path.join(matlabBatchPath, "matlab-batch" + matlabBatchExt)]);
+        if (exitCode !== 0) {
+            return Promise.reject(Error("Unable to add execute permissions to matlab-batch binary."));
+        }
     }
-    return;
 }
 
 export async function installSystemDependencies(platform: string, architecture: string, release: string) {

Original file line number	Diff line number	Diff line change
`@@ -144,11 +144,12 @@ export async function setupBatch(platform: string, architecture: string) {`
`144`	`144`	`} catch (err: any) {`
`145`	`145`	`throw new Error("Failed to add MATLAB to system path.");`
`146`	`146`	`}`
`147`		`- const exitCode = await taskLib.exec("chmod", ["+x", path.join(matlabBatchPath, "matlab-batch" + matlabBatchExt)]);`
`148`		`- if (exitCode !== 0) {`
`149`		`- return Promise.reject(Error("Unable to add execute permissions to matlab-batch binary."));`
	`147`	`+ if (platform !== "win32") {`
	`148`	`+ const exitCode = await taskLib.exec("chmod", ["+x", path.join(matlabBatchPath, "matlab-batch" + matlabBatchExt)]);`
	`149`	`+ if (exitCode !== 0) {`
	`150`	`+ return Promise.reject(Error("Unable to add execute permissions to matlab-batch binary."));`
	`151`	`+ }`
`150`	`152`	`}`
`151`		`- return;`
`152`	`153`	`}`
`153`	`154`
`154`	`155`	`export async function installSystemDependencies(platform: string, architecture: string, release: string) {`