Opened 19 months ago

Closed 6 days ago

#31211 closed defect (invalid)

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

Reported by: mkoeppe Owned by:
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: porting Keywords:
Cc: gh-kliem, malb Merged in:
Authors: Reviewers: Kwankyu Lee
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #33791 Stopgaps:

Status badges

Description

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.)

Change History (7)

comment:1 Changed 17 months ago by mkoeppe

  • Milestone changed from sage-9.3 to sage-9.4

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.

comment:2 Changed 12 months ago by mkoeppe

  • Milestone changed from sage-9.4 to sage-9.5

comment:3 Changed 7 months ago by mkoeppe

  • Milestone changed from sage-9.5 to sage-9.6

comment:4 Changed 4 months ago by mkoeppe

  • Milestone changed from sage-9.6 to sage-9.7

comment:5 Changed 3 months ago by mkoeppe

  • Dependencies changed from #31064 to #33791
  • Milestone changed from sage-9.7 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

done in #33791

comment:6 Changed 9 days ago by klee

  • Reviewers set to Kwankyu Lee
  • Status changed from needs_review to positive_review

Okay.

comment:7 Changed 6 days ago by mkoeppe

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.