gh-38747: Fix build.yml broken for releases

    
<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes #12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes #12345". -->

The workflow build.yml does not work well with releases:

https://github.com/sagemath/sage/actions/runs/11091397369

We fix it with
```yaml
      - name: Get changed files and packages
        id: changed-files
        uses: tj-actions/changed-files@v45
        with:
          # File extensions for doctests per
sage.doctest.control.skipfile
          files_yaml: |
            configures:
              - 'build/pkgs/*/spkg-configure.m4'
            pkgs:
              - 'build/pkgs/**'
              - '!build/pkgs/_**'
              - '!build/pkgs/configure/**'
              - 'pkgs/**'
            doctests:
              - 'src/**/*.{py,pyx,pxd,pxi,sage,spyx,rst,tex}'
              - '!src/{setup,conftest*}.py'
```
Note the two lines added
```yaml
              - '!build/pkgs/_**'
              - '!build/pkgs/configure/**'
```

Along the way,  we make `.ci/write-dockerfile.sh` produce well formatted
`Dockerfile` to improve readability.

test:
https://github.com/kwankyu/sage/actions/runs/11122936897/job/30905299233

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - #12345: short description why this is a dependency -->
<!-- - #34567: ... -->
    
URL: #38747
Reported by: Kwankyu Lee
Reviewer(s): Matthias Köppe

.ci/write-dockerfile.sh

@@ -35,7 +35,7 @@ STRIP_COMMENTS="sed s/#.*//;"
 SAGE_ROOT=.
 export PATH="$SAGE_ROOT"/build/bin:$PATH
 SYSTEM_PACKAGES=$EXTRA_SYSTEM_PACKAGES
-SYSTEM_CONFIGURE_ARGS="--enable-option-checking "
+SYSTEM_CONFIGURE_ARGS=" --enable-option-checking"
 for SPKG in $(sage-package list --has-file=spkg-configure.m4 $SAGE_PACKAGE_LIST_ARGS) $EXTRA_SAGE_PACKAGES; do
     SYSTEM_PACKAGE=$(sage-get-system-packages $SYSTEM $SPKG)
     if [ -n "${SYSTEM_PACKAGE}" ]; then
@@ -45,13 +45,20 @@ for SPKG in $(sage-package list --has-file=spkg-configure.m4 $SAGE_PACKAGE_LIST_
             # shell-quote package if necessary
             SYSTEM_PACKAGES+=$(printf " %q" "$a")
         done
-        SYSTEM_CONFIGURE_ARGS+="--with-system-${SPKG}=${WITH_SYSTEM_SPKG} "
+        # Check if SPKG is not a dummy package
+        if [[ $SPKG != _* ]]; then
+            SYSTEM_CONFIGURE_ARGS+=" --with-system-${SPKG}=${WITH_SYSTEM_SPKG}"
+        fi
     fi
 done
 echo "# Automatically generated by SAGE_ROOT/.ci/write-dockerfile.sh"
 echo "# the :comments: separate the generated file into sections"
 echo "# to simplify writing scripts that customize this file"
-ADD="ADD $__CHOWN"
+if [ -z "$__CHOWN" ]; then
+  ADD="ADD"
+else
+  ADD="ADD $__CHOWN"
+fi
 RUN=RUN
 cat <<EOF
 ARG BASE_IMAGE=$(eval echo "${FULL_BASE_IMAGE_AND_TAG}")
@@ -86,10 +93,7 @@ case $SYSTEM in
                 # we remove the unminimize binary here after it has done its job.
                 #
                 cat <<EOF
-RUN if command -v unminimize > /dev/null; then  \
-        (yes | unminimize) || echo "(ignored)"; \
-        rm -f "\$(command -v unminimize)";       \
-    fi
+RUN if command -v unminimize > /dev/null; then (yes | unminimize) || echo "(ignored)"; rm -f "\$(command -v unminimize)"; fi
 EOF
                 if [ -n "$DIST_UPGRADE" ]; then
                     cat <<EOF
@@ -254,10 +258,10 @@ case ${DOCKER_BUILDKIT-0} in
         # With buildkit we cannot retrieve failed builds.
         # So we do not allow the main step of a build stage to fail.
         # Instead we record the exit code in the file STATUS.
-        THEN_SAVE_STATUS='; echo $? > STATUS'
+        THEN_SAVE_STATUS=' ; echo $? > STATUS'
         # ... and at the beginning of the next build stage,
         # we check the status and exit with an error status.
-        CHECK_STATUS_THEN='STATUS=$(cat STATUS 2>/dev/null); case "$STATUS" in ""|0) ;; *) exit $STATUS;; esac; '
+        CHECK_STATUS_THEN=' STATUS=$(cat STATUS 2>/dev/null); case "$STATUS" in ""|0) ;; *) exit $STATUS;; esac;'
 esac
 
 if [ -n "$GITHUB_ACTIONS" ]; then
@@ -281,38 +285,37 @@ $ADD pkgs /new/pkgs
 $ADD build /new/build
 $ADD .upstream.d /new/.upstream.d
 ADD .ci /.ci
-RUN if [ -d /sage ]; then                                               \
-        echo "### Incremental build from \$(cat /sage/VERSION.txt)" &&  \
-        printf '/src/*\n!/src/doc/bootstrap\n!/src/bin\n!/src/*.m4\n!/src/*.toml\n!/src/VERSION.txt\n' >> /sage/.gitignore && \
-        printf '/src/*\n!/src/doc/bootstrap\n!/src/bin\n!/src/*.m4\n!/src/*.toml\n!/src/VERSION.txt\n' >> /new/.gitignore && \
-        if ! (cd /new && /.ci/retrofit-worktree.sh worktree-image /sage); then \
-            echo "retrofit-worktree.sh failed, falling back to replacing /sage"; \
-            for a in local logs; do                                     \
-                if [ -d /sage/\$a ]; then mv /sage/\$a /new/; fi;       \
-            done;                                                       \
-            rm -rf /sage;                                               \
-            mv /new /sage;                                              \
-        fi;                                                             \
-    else                                                                \
-        mv /new /sage;                                                  \
+RUN if [ -d /sage ]; then \\
+        echo "### Incremental build from \$(cat /sage/VERSION.txt)" && \\
+        printf '/src/*\n!/src/doc/bootstrap\n!/src/bin\n!/src/*.m4\n!/src/*.toml\n!/src/VERSION.txt\n' >> /sage/.gitignore && \\
+        printf '/src/*\n!/src/doc/bootstrap\n!/src/bin\n!/src/*.m4\n!/src/*.toml\n!/src/VERSION.txt\n' >> /new/.gitignore && \\
+        if ! (cd /new && /.ci/retrofit-worktree.sh worktree-image /sage); then \\
+            echo "retrofit-worktree.sh failed, falling back to replacing /sage"; \\
+            for a in local logs; do \\
+                if [ -d /sage/\$a ]; then mv /sage/\$a /new/; fi; \\
+            done; \\
+            rm -rf /sage; \\
+            mv /new /sage; \\
+        fi; \\
+    else \\
+        mv /new /sage; \\
     fi
 WORKDIR /sage
-
 ARG BOOTSTRAP="${BOOTSTRAP-./bootstrap}"
-$RUN sh -x -c "\${BOOTSTRAP}" $ENDRUN $THEN_SAVE_STATUS
+$RUN sh -x -c "\${BOOTSTRAP}"$ENDRUN$THEN_SAVE_STATUS
 
 FROM bootstrapped AS configured
 #:configuring:
-RUN $CHECK_STATUS_THEN mkdir -p logs/pkgs; rm -f config.log; ln -s logs/pkgs/config.log config.log
+RUN$CHECK_STATUS_THEN mkdir -p logs/pkgs; rm -f config.log; ln -s logs/pkgs/config.log config.log
 ARG CONFIGURE_ARGS="${CONFIGURE_ARGS:---enable-build-as-root}"
 EOF
 if [ ${WITH_SYSTEM_SPKG} = "force" ]; then
     cat <<EOF
-$RUN ./configure $SYSTEM_CONFIGURE_ARGS \${CONFIGURE_ARGS} || (echo "::group::config.log"; cat config.log; echo "::endgroup::"; echo "********** configuring without forcing ***********"; ./configure \${CONFIGURE_ARGS}; echo "::group::config.log"; cat config.log; echo "::endgroup::"; exit 1) $ENDRUN $THEN_SAVE_STATUS
+$RUN ./configure $SYSTEM_CONFIGURE_ARGS \${CONFIGURE_ARGS} || (echo "::group::config.log"; cat config.log; echo "::endgroup::"; echo "********** configuring without forcing ***********"; ./configure \${CONFIGURE_ARGS}; echo "::group::config.log"; cat config.log; echo "::endgroup::"; exit 1)$ENDRUN$THEN_SAVE_STATUS
 EOF
 else
     cat <<EOF
-$RUN ./configure $SYSTEM_CONFIGURE_ARGS \${CONFIGURE_ARGS} || (echo "::group::config.log"; cat config.log; echo "::endgroup::"; exit 1) $ENDRUN $THEN_SAVE_STATUS
+$RUN ./configure $SYSTEM_CONFIGURE_ARGS \${CONFIGURE_ARGS} || (echo "::group::config.log"; cat config.log; echo "::endgroup::"; exit 1)$ENDRUN$THEN_SAVE_STATUS
 EOF
 fi
 cat <<EOF
@@ -325,7 +328,7 @@ ARG USE_MAKEFLAGS="-k V=0"
 ENV SAGE_CHECK=warn
 ENV SAGE_CHECK_PACKAGES="!cython,!r,!python3,!gap,!cysignals,!linbox,!git,!ppl,!cmake,!rpy2,!sage_sws2rst"
 #:toolchain:
-$RUN $CHECK_STATUS_THEN make \${USE_MAKEFLAGS} base-toolchain $ENDRUN $THEN_SAVE_STATUS
+$RUN$CHECK_STATUS_THEN make \${USE_MAKEFLAGS} base-toolchain$ENDRUN$THEN_SAVE_STATUS
 
 FROM with-base-toolchain AS with-targets-pre
 ARG NUMPROC=8
@@ -335,7 +338,7 @@ ENV SAGE_CHECK=warn
 ENV SAGE_CHECK_PACKAGES="!cython,!r,!python3,!gap,!cysignals,!linbox,!git,!ppl,!cmake,!rpy2,!sage_sws2rst"
 #:make:
 ARG TARGETS_PRE="all-sage-local"
-$RUN $CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS_PRE} $ENDRUN $THEN_SAVE_STATUS
+$RUN$CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS_PRE}$ENDRUN$THEN_SAVE_STATUS
 
 FROM with-targets-pre AS with-targets
 ARG NUMPROC=8
@@ -345,19 +348,18 @@ ENV SAGE_CHECK=warn
 ENV SAGE_CHECK_PACKAGES="!cython,!r,!python3,!gap,!cysignals,!linbox,!git,!ppl,!cmake,!rpy2,!sage_sws2rst"
 $ADD .gitignore /new/.gitignore
 $ADD src /new/src
-RUN cd /new && rm -rf .git && \
-    if /.ci/retrofit-worktree.sh worktree-pre /sage; then \
-        cd /sage && touch configure build/make/Makefile; \
-    else \
-        echo "retrofit-worktree.sh failed, falling back to replacing /sage/src"; \
-        rm -rf /sage/src;                                    \
-        mv src /sage/src;                                    \
-        cd /sage && ./bootstrap && ./config.status;          \
-    fi; \
+RUN cd /new && rm -rf .git && \\
+    if /.ci/retrofit-worktree.sh worktree-pre /sage; then \\
+        cd /sage && touch configure build/make/Makefile; \\
+    else \\
+        echo "retrofit-worktree.sh failed, falling back to replacing /sage/src"; \\
+        rm -rf /sage/src; \\
+        mv src /sage/src; \\
+        cd /sage && ./bootstrap && ./config.status; \\
+    fi; \\
     cd /sage && rm -rf .git; rm -rf /new || echo "(error ignored)"
-
 ARG TARGETS="build"
-$RUN $CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS} $ENDRUN $THEN_SAVE_STATUS
+$RUN$CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS}$ENDRUN$THEN_SAVE_STATUS
 
 FROM with-targets AS with-targets-optional
 ARG NUMPROC=8
@@ -366,7 +368,7 @@ ARG USE_MAKEFLAGS="-k V=0"
 ENV SAGE_CHECK=warn
 ENV SAGE_CHECK_PACKAGES="!cython,!r,!python3,!gap,!cysignals,!linbox,!git,!ppl,!cmake,!rpy2,!sage_sws2rst"
 ARG TARGETS_OPTIONAL="ptest"
-$RUN $CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS_OPTIONAL} || echo "(error ignored)" $ENDRUN $THEN_SAVE_STATUS
+$RUN$CHECK_STATUS_THEN make SAGE_SPKG="sage-spkg -y -o" \${USE_MAKEFLAGS} \${TARGETS_OPTIONAL} || echo "(error ignored)"$ENDRUN$THEN_SAVE_STATUS
 
 #:end:
 EOF

.github/workflows/build.yml

@@ -97,9 +97,11 @@ jobs:
           remove-haskell:       true
           remove-codeql:        true
           remove-docker-images: true
+
       - name: Checkout
         id: checkout
         uses: actions/checkout@v4
+
       - name: Get changed files and packages
         id: changed-files
         uses: tj-actions/changed-files@v45
@@ -110,26 +112,47 @@ jobs:
               - 'build/pkgs/*/spkg-configure.m4'
             pkgs:
               - 'build/pkgs/**'
+              - '!build/pkgs/_**'
+              - '!build/pkgs/configure/**'
               - 'pkgs/**'
             doctests:
               - 'src/**/*.{py,pyx,pxd,pxi,sage,spyx,rst,tex}'
               - '!src/{setup,conftest*}.py'
+
       - name: Determine targets to build
         id: build-targets
         run: |
-          uninstall_targets=$(echo $(for a in '' ${{ steps.changed-files.outputs.configures_all_changed_files }}; do echo $a | sed -E 's,build/pkgs/([a-z0-9][_.a-z0-9]*)/spkg-configure[.]m4 *,\1-uninstall,'; done | sort -u))
-          build_targets=$(echo $(for a in '' ${{ steps.changed-files.outputs.pkgs_all_changed_files }}; do SPKG=$(echo $a | sed -E 's,-,_,g;s,(build/)?pkgs/([a-z0-9][-_.a-z0-9]*)/[^ ]* *,\2,;'); if [ -f "build/pkgs/$SPKG/checksums.ini" -o -f "build/pkgs/$SPKG/requirements.txt" -o -f "build/pkgs/$SPKG/spkg-install" ]; then echo "$SPKG-ensure"; fi; done | sort -u))
+          uninstall_targets=$(echo $(
+            for a in '' ${{ steps.changed-files.outputs.configures_all_changed_files }}; do
+              # Extract package name from the file path and append '-uninstall'
+              echo $a | sed -E 's,build/pkgs/([a-z0-9][_.a-z0-9]*)/spkg-configure[.]m4 *,\1-uninstall,'
+            done | sort -u  # Sort and ensure uniqueness
+          ))
+          build_targets=$(echo $(
+            for a in '' ${{ steps.changed-files.outputs.pkgs_all_changed_files }}; do
+              # Extract package name, replace '-' with '_', and strip extra parts from the path
+              SPKG=$(echo $a | sed -E 's,-,_,g;s,(build/)?pkgs/([a-z0-9][-_.a-z0-9]*)/[^ ]* *,\2,;')
+              # Check if key files exist in the package directory
+              if [ -f "build/pkgs/$SPKG/checksums.ini" ] || \
+                 [ -f "build/pkgs/$SPKG/requirements.txt" ] || \
+                 [ -f "build/pkgs/$SPKG/spkg-install" ]; then
+                echo "$SPKG-ensure"  # add the "$SPKG-ensure" target
+              fi
+            done | sort -u  # Sort and ensure uniqueness
+          ))
           if [ -n "$uninstall_targets" ]; then
-              echo "build_targets=$uninstall_targets reconfigure $build_targets ci-build-with-fallback" >> $GITHUB_OUTPUT
+            echo "build_targets=$uninstall_targets reconfigure $build_targets ci-build-with-fallback" >> $GITHUB_OUTPUT
           else
-              echo "build_targets=$build_targets ci-build-with-fallback" >> $GITHUB_OUTPUT
+            echo "build_targets=$build_targets ci-build-with-fallback" >> $GITHUB_OUTPUT
           fi
           cat $GITHUB_OUTPUT
+
       - uses: actions/checkout@v4
         with:
           ref: ${{ github.base_ref }}
           path: worktree-base
         if: github.base_ref && steps.changed-files.outputs.pkgs_all_changed_files
+
       - name: Compute metrics
         run: |
           export PATH=build/bin:$PATH
@@ -140,13 +163,15 @@ jobs:
           else
             sage-package metrics :all:
           fi
+
       - name: Install test prerequisites
         # From docker.yml
         run: |
           sudo DEBIAN_FRONTEND=noninteractive apt-get update
           sudo DEBIAN_FRONTEND=noninteractive apt-get install tox
           sudo apt-get clean
           df -h
+
       - name: Merge CI fixes from sagemath/sage
         # From docker.yml
         # This step needs to happen after the commit sha is put in DOCKER_TAG
@@ -254,16 +279,19 @@ jobs:
           remove-haskell:       true
           remove-codeql:        true
           remove-docker-images: true
+
       - name: Checkout
         id: checkout
         uses: actions/checkout@v4
+
       - name: Install test prerequisites
         # From docker.yml
         run: |
           sudo DEBIAN_FRONTEND=noninteractive apt-get update
           sudo DEBIAN_FRONTEND=noninteractive apt-get install tox
           sudo apt-get clean
           df -h
+
       - name: Merge CI fixes from sagemath/sage
         # From docker.yml
         # This step needs to happen after the commit sha is put in DOCKER_TAG
@@ -352,16 +380,19 @@ jobs:
           remove-haskell:       true
           remove-codeql:        true
           remove-docker-images: true
+
       - name: Checkout
         id: checkout
         uses: actions/checkout@v4
+
       - name: Install test prerequisites
         # From docker.yml
         run: |
           sudo DEBIAN_FRONTEND=noninteractive apt-get update
           sudo DEBIAN_FRONTEND=noninteractive apt-get install tox
           sudo apt-get clean
           df -h
+
       - name: Merge CI fixes from sagemath/sage
         # From docker.yml
         # This step needs to happen after the commit sha is put in DOCKER_TAG
@@ -469,16 +500,19 @@ jobs:
           remove-haskell:       true
           remove-codeql:        true
           remove-docker-images: true
+
       - name: Checkout
         id: checkout
         uses: actions/checkout@v4
+
       - name: Install test prerequisites
         # From docker.yml
         run: |
           sudo DEBIAN_FRONTEND=noninteractive apt-get update
           sudo DEBIAN_FRONTEND=noninteractive apt-get install tox
           sudo apt-get clean
           df -h
+
       - name: Merge CI fixes from sagemath/sage
         # From docker.yml
         # This step needs to happen after the commit sha is put in DOCKER_TAG