From e075c1255adbcb591c260f7903861b5731af9c76 Mon Sep 17 00:00:00 2001 From: Patrick Roncagliolo Date: Mon, 9 Dec 2024 20:42:18 +0100 Subject: [PATCH] Do not run `apt update` if `skipRosdepInstall==true` --- src/action-ros-ci.ts | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/action-ros-ci.ts b/src/action-ros-ci.ts index b84eb98c..bf9682ee 100644 --- 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,