Skip to content

Commit

Permalink
Fix aliBuild deps invocation (alisw#2707)
Browse files Browse the repository at this point in the history
The package name that aliBuild expects is usually different in capitalisation
from the .sh file name, so parse its metadata to get the real package name.
  • Loading branch information
TimoWilken authored Dec 17, 2020
1 parent a25ad7e commit 3a9bcf1
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/recipe-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ jobs:
# Runs bash -eo pipefail
shell: bash
run: |
set -x
aliBuild analytics off
err_fnames=()
Expand Down Expand Up @@ -50,7 +51,8 @@ jobs:
echo "ERROR: recipe $fname has no modulefile"
err_fnames+=("$fname")
fi
aliBuild deps --defaults o2 "${fname%.sh}" --outgraph deps.pdf --no-system --neat -c . 2>&1 |
aliBuild deps --defaults o2 --outgraph /dev/null --no-system --neat -c . \
"$(awk '/^package: /{print $2}' "$fname")" 2>&1 |
if grep -q 'transitive reduction not unique'; then
echo "ERROR: recipe $fname has circular dependency"
err_fnames+=("$fname")
Expand Down

0 comments on commit 3a9bcf1

Please sign in to comment.