From c68b399d8928e62bb4b78c92b59aa970f3cdf0b8 Mon Sep 17 00:00:00 2001 From: Krzesimir Nowak Date: Mon, 17 Oct 2022 17:25:06 +0200 Subject: [PATCH] github: Fix package check Packages in the list are not necessarily packages only, which are represented as directories (like sys-apps/systemd), but also, in case of eclasses, plain files. The check was checking for the path to be a directory and emitted the warning if it was not, which resulted in eclasses being kept not updated. Just check if the path exists. --- .../.github/workflows/update-packages-from-list.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk_container/src/third_party/portage-stable/.github/workflows/update-packages-from-list.yml b/sdk_container/src/third_party/portage-stable/.github/workflows/update-packages-from-list.yml index 043cd59bf8..b1ac7a2caa 100644 --- a/sdk_container/src/third_party/portage-stable/.github/workflows/update-packages-from-list.yml +++ b/sdk_container/src/third_party/portage-stable/.github/workflows/update-packages-from-list.yml @@ -33,7 +33,7 @@ jobs: old_head=$(git -C portage-stable rev-parse HEAD) cd portage-stable while read -r package; do - if [[ ! -d "${package}" ]]; then + if [[ ! -e "${package}" ]]; then # If this happens, it means that the package was moved to overlay # or dropped, the list ought to be updated. echo "::warning title=${package}::Nonexistent package"