sagemath/sage

ci-cygwin*.yml: Fix packaging up of SAGE_LOCAL/var/tmp/sage/build/*

mkoeppe opened this issue · 5 comments

These files are intended to be put in the logs artifact, but the code is being run with the wrong shell, so it fails silently:

https://github.com/kliem/sage/runs/1626910566

0s
Run mkdir -p "artifacts/$LOGS_ARTIFACT_NAME"; for a in "/opt/sage-255cf51a4f41b054d32a39562102877b0b18969f"/var/tmp/sage/build/*; do if [ -d "$a" ]; then tar -c --remove-files -f "artifacts/$LOGS_ARTIFACT_NAME/$(basename "$a").tar" "$a"; fi; done; cp -r logs/*  "artifacts/$LOGS_ARTIFACT_NAME"
  mkdir -p "artifacts/$LOGS_ARTIFACT_NAME"; for a in "/opt/sage-255cf51a4f41b054d32a39562102877b0b18969f"/var/tmp/sage/build/*; do if [ -d "$a" ]; then tar -c --remove-files -f "artifacts/$LOGS_ARTIFACT_NAME/$(basename "$a").tar" "$a"; fi; done; cp -r logs/*  "artifacts/$LOGS_ARTIFACT_NAME"
  shell: C:\Program Files\Git\bin\bash.EXE --noprofile --norc -e -o pipefail {0}

(the correct shell to use for running tar is the cygwin shell.)

Depends on #33791

CC: @kliem @malb

Component: porting

Reviewer: Kwankyu Lee

Issue created by migration from https://trac.sagemath.org/ticket/31211

comment:1

Sage development has entered the release candidate phase for 9.3. Setting a new milestone for this ticket based on a cursory review of ticket status, priority, and last modification date.

Changed dependencies from #31064 to #33791

comment:5

done in #33791

comment:6

Okay.

Reviewer: Kwankyu Lee