Do not run apt update if skipRosdepInstall==true

ros-tooling · Dec 9, 2024 · e075c12 · e075c12
1 parent 5816ec0
commit e075c12
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/action-ros-ci.ts b/src/action-ros-ci.ts
@@ -650,7 +650,9 @@ done`;
 	// Print HEAD commits of all repos
 	await execShellCommand(["vcs log -l1 src/"], options);
 
-	if (isLinux) {
+	// rosdep does not really work on Windows, so do not use it
+	// See: https://github.com/ros-infrastructure/rosdep/issues/610
+	if (!isWindows && !skipRosdepInstall) {
 		// Always update package index before installing packages
 		const dist: string = await determineDistrib();
 		if (dist === "ubuntu") {
@@ -662,10 +664,7 @@ done`;
 		} else {
 			core.setFailed(`Unsupported distribution ${dist}`);
 		}
-	}
-	// rosdep does not really work on Windows, so do not use it
-	// See: https://github.com/ros-infrastructure/rosdep/issues/610
-	if (!isWindows && !skipRosdepInstall) {
+
 		await installRosdeps(
 			buildPackageSelection,
 			rosdepSkipKeysSelection,