sagemath/sage

Fix tox-docker builds broken by #29884

mkoeppe opened this issue · 10 comments

mkoeppe commented

In 9.2.beta3 (https://github.com/sagemath/sage/runs/837071227):

Step 13/18 : RUN ./bootstrap
 ---> Running in 371ddf6853f5
rm -rf config configure build/make/Makefile-auto.in
rm -f src/doc/en/installation/*.txt
rm -rf src/doc/en/reference/spkg/*.rst
rm -f src/doc/en/reference/repl/*.txt
src/doc/bootstrap:48: installing src/doc/en/installation/arch.txt and src/doc/en/installation/arch-optional.txt
src/doc/bootstrap:48: installing src/doc/en/installation/debian.txt and src/doc/en/installation/debian-optional.txt
src/doc/bootstrap:48: installing src/doc/en/installation/fedora.txt and src/doc/en/installation/fedora-optional.txt
src/doc/bootstrap:48: installing src/doc/en/installation/cygwin.txt and src/doc/en/installation/cygwin-optional.txt
src/doc/bootstrap:48: installing src/doc/en/installation/homebrew.txt and src/doc/en/installation/homebrew-optional.txt
src/doc/bootstrap:55: installing src/doc/en/reference/spkg/*.rst
src/doc/bootstrap:83: installing src/doc/en/reference/repl/options.txt
src/doc/bootstrap: line 84: src/doc/en/reference/repl/options.txt: No such file or directory
The command '/bin/sh -c ./bootstrap' returned a non-zero code: 1

Depends on #29987

CC: @dimpase @kliem @jhpalmieri @sheerluck

Component: porting

Author: Matthias Koeppe

Branch/Commit: b7bf43b

Reviewer: Jonathan Kliem

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

mkoeppe commented

Dependencies: #29987

mkoeppe commented

Author: Matthias Koeppe

mkoeppe commented

Commit: b7bf43b

mkoeppe commented

New commits:

539c182build/make/install: Do not depend on src/bin/sage-version.sh
761092cMerge branch 't/29987/build_make_install__do_not_depend_on_src_bin_sage_version_sh' into t/30064/fix_tox_docker_builds_broken_by__29884
f2efa6asrc/doc/bootstrap: Create the directory src/doc/en/reference/repl if it does not exist
b7bf43bbuild/bin/write-dockerfile.sh: ADD src/bin for bootstrapping, needed by src/doc/bootstrap after #29884
kliem commented

Reviewer: Jonathan Kliem

kliem commented
comment:4

LGTM.

Strangly, centos 7 works again. So maybe we can mark #30008 as invalid then.

mkoeppe commented
comment:5

Thanks!