Opened 17 months ago

Last modified 15 months ago

#31064 closed defect

ci-cygwin*.yml: Adjust to new script packages _bootstrap, _prereq — at Version 13

Reported by: mkoeppe Owned by:
Priority: critical Milestone: sage-9.3
Component: porting: Cygwin Keywords:
Cc: slabbe, gh-kliem, arojas, embray, slelievre Merged in:
Authors: Matthias Koeppe Reviewers: https://github.com/mkoeppe/sage/runs/1580222034
Report Upstream: N/A Work issues:
Branch: u/mkoeppe/ci_cygwin__yml__adjust_to_new_script_packages__bootstrap___prereq (Commits, GitHub, GitLab) Commit: b327031d96686ead49e2386c180fedabd09d2a4e
Dependencies: #29124, #30944 Stopgaps:

Status badges

Description (last modified by mkoeppe)

#29124 moved the files build/pkgs/*.txt to new locations but forgot to update the CI scripts for Cygwin, leading to failures like this (​https://github.com/mkoeppe/sage/runs/1561097256): 

sed: can't read ./build/pkgs/cygwin.txt: No such file or directory
sed: can't read ./build/pkgs/cygwin-bootstrap.txt: No such file or directory
Chocolatey v0.10.15
Package name is required. Please pass at least one package name to install.
Error: Process completed with exit code 1.

We fix it in this ticket by going through SAGE_ROOT/tox.ini for the package installation and invoking the new system package scripts from there.

We also make another improvement that was easy to do: The build now uses SAGE_LOCAL=/opt/sage-$COMMIT instead of /sage/local. This will make it easier to download and install several versions of Sage.

Change History (13)

comment:1 Changed 17 months ago by mkoeppe

  • Dependencies changed from #29124 to #29124, #30944

comment:2 Changed 17 months ago by mkoeppe

  • Branch set to u/mkoeppe/ci_cygwin__yml__adjust_to_new_script_packages__bootstrap___prereq

comment:3 Changed 17 months ago by mkoeppe

  • Authors set to Matthias Koeppe
  • Cc gh-kliem added
  • Commit set to b2ae68aea3cb706650731b0488b59ce992234e7b
  • Description modified (diff)
  • Reviewers set to https://github.com/mkoeppe/sage/actions/runs/427378215
  • Status changed from new to needs_review

Last 10 new commits:

​898758dtox.ini: Add local-root
​559dd8etox.ini (local-root): Pass --enable-build-as-root to configure
​a85f41ctox.ini (local): Do not build the toolchain when posargs = config.status or posargs = configure
​1de912atox.ini (local-sudo): Run apt-get update with sudo
​3c7e5c4tox.ini (local-root, local-sudo): Run output of sage-print-system-package-command through eval
​f7ff30cMerge branch 't/29124/script-packages-prereq-toolchain-bootstrap' into t/30944/tox__improve_local_sudo_ubuntu_standard
​e9ca2c1Merge branch 't/29124/script-packages-prereq-toolchain-bootstrap' into t/30944/tox__improve_local_sudo_ubuntu_standard
​be4177bMerge branch 't/30944/tox__improve_local_sudo_ubuntu_standard' into t/31064/ci_cygwin__yml__adjust_to_new_script_packages__bootstrap___prereq
​f74fe3dtox.ini (local-cygwin-choco): New
​b2ae68a.github/workflows/ci-cygwin*.yml: Install cygwin packages via tox

comment:4 Changed 17 months ago by mkoeppe

  • Status changed from needs_review to needs_work
  • Work issues set to SAGE_LOCAL

comment:5 Changed 17 months ago by git

  • Commit changed from b2ae68aea3cb706650731b0488b59ce992234e7b to 11cf8db0b49607b83c66d0d0d49ec52ea0069f39

Branch pushed to git repo; I updated commit sha1. New commits:

​11cf8db.github/workflows/ci-cygwin*.yml: Use SAGE_LOCAL as set by tox

comment:6 Changed 17 months ago by mkoeppe

  • Reviewers changed from https://github.com/mkoeppe/sage/actions/runs/427378215 to https://github.com/mkoeppe/sage/actions/runs/429356144
  • Status changed from needs_work to needs_review
  • Work issues SAGE_LOCAL deleted

comment:7 Changed 17 months ago by mkoeppe

  • Status changed from needs_review to needs_work
  • Work issues set to update extract-sage-local.sh

comment:8 Changed 17 months ago by git

  • Commit changed from 11cf8db0b49607b83c66d0d0d49ec52ea0069f39 to 05d57c0bf907fb91f229325f5cfa0e35b001fea2

Branch pushed to git repo; I updated commit sha1. New commits:

​05d57c0.github/workflows/ci-cygwin-*.yml: Pass PREFIX to tox

comment:9 Changed 17 months ago by git

  • Commit changed from 05d57c0bf907fb91f229325f5cfa0e35b001fea2 to d87e1604ba0c6a27ffc0e048d1bb351fadc54d90

Branch pushed to git repo; I updated commit sha1. New commits:

​d87e160.github/workflows/extract-sage-local.sh: Use PREFIX

comment:10 Changed 17 months ago by mkoeppe

  • Reviewers changed from https://github.com/mkoeppe/sage/actions/runs/429356144 to https://github.com/mkoeppe/sage/actions/runs/431265667

comment:11 Changed 17 months ago by git

  • Commit changed from d87e1604ba0c6a27ffc0e048d1bb351fadc54d90 to b327031d96686ead49e2386c180fedabd09d2a4e

Branch pushed to git repo; I updated commit sha1. New commits:

​0827c85Quoting fixes
​00a9182Just use github.sha
​b327031.github/workflows/extract-sage-local.sh: Fix up path

comment:12 Changed 17 months ago by mkoeppe

  • Reviewers changed from https://github.com/mkoeppe/sage/actions/runs/431265667 to https://github.com/mkoeppe/sage/runs/1580222034
  • Status changed from needs_work to needs_review
  • Work issues update extract-sage-local.sh deleted

comment:13 Changed 17 months ago by mkoeppe

  • Description modified (diff)
Note: See TracTickets for help on using tickets.