PPS: Just FYI, before the above snippet was roughed in, "make configure" from "$SAGE_ROOT" with embedded spaces indicates that "$SYSTEM_PACKAGES_FILE" need be properly quoted in "build/bin/sage-get-system-packages" and "src/doc/bootstrap", at least(?)
It couldn't be figured out why tons of sed 's complaints (related to the pathname with spaces) come from: (echo ".. _spkg_$PKG_BASE:" && echo && OUTPUT_RST=1 sage-spkg-info "$PKG_BASE") | sed -e "s|https://trac.sagemath.org/ticket/\([0-9]*\)|:trac:\`\1\`|g" -e "s|https://arxiv.org/abs/cs/\([0-9]*\)|:arxiv:\`cs/\1\`|g" > "$OUTPUT_DIR"/$PKG_BASE.rst Any help is much appreciated. *<src/doc/bootstrap>* for PKG_BASE in $(sage-package list --has-file SPKG.rst | sort); do PKG_SCRIPTS=build/pkgs/$PKG_BASE # Instead of just copying, we may want to call # a version of sage-spkg-info to format extra information. # for sphinx 4.4 we need to replace all direct links by some "extlink" (ticket 33272) (echo ".. _spkg_$PKG_BASE:" && echo && OUTPUT_RST=1 sage-spkg-info "$PKG_BASE") | sed -e "s| https://trac.sagemath.org/ticket/\([0-9]*\)|:trac:\`\1\`|g" -e "s| https://arxiv.org/abs/cs/\([0-9]*\)|:arxiv:\`cs/\1\`|g" > "$OUTPUT_DIR"/$PKG_BASE.rst echo >> "$OUTPUT_INDEX" " $PKG_BASE" done *</src/doc/bootstrap>* On Fri, May 27, 2022 at 8:28 AM ph h <hohoang...@gmail.com> wrote: > P.S: Would this snippet in 'bootstrap' (below SAGE_ROOT=...) help, too: > > # Set SAGE_ROOT to the path to this file and then cd into it > SAGE_ROOT="$(cd "$(dirname "$0")" && pwd -P)" > > # Check that we are not building in a directory containing spaces > if echo "$SAGE_ROOT" | grep " " > /dev/null ; then > echo "The path to the Sage root directory ($SAGE_ROOT) contains a > space. Sage will not build correctly in this case." > exit 1 > fi > > > > > On Fri, May 27, 2022 at 7:33 AM ph h <hohoang...@gmail.com> wrote: > >> Hi, >> >> Maybe the tests with "contains_spaces()" in 'src/bin/sage-env' can be >> replaced with the followings in 'configure.ac' (after line 75, >> SAGE_LOCAL="$prefix"): >> >> *<configure.ac <http://configure.ac>>* >> # Check that we are not building in a directory containing spaces >> AS_IF([echo "$ac_pwd" |grep " " >/dev/null], >> AC_MSG_ERROR([The path to the Sage root directory ($ac_pwd) >> contains a space. Sage will not build correctly in this case]) >> ) >> >> # Check that we are not installing to a directory containing spaces >> AS_IF([echo "$SAGE_LOCAL" |grep " " >/dev/null], >> AC_MSG_ERROR([The path to the Sage installation directory >> ($SAGE_LOCAL) contains a space. Sage will not work correctly in this case]) >> ) >> *</configure.ac <http://configure.ac>>* >> >> The first AS_IF() was moved up from below (around line 420) >> >> Thank you for your help. >> >> Regards, >> >> phiho >> >> >> >> On Tue, May 24, 2022 at 7:32 PM Matthias Koeppe <matthiaskoe...@gmail.com> >> wrote: >> >>> On Tuesday, May 24, 2022 at 3:11:00 PM UTC-7 hohoa...@gmail.com wrote: >>> >>>> Does this apply only to "$SAGE_ROOT"? >>>> Can sub-directories of "$SAGE_ROOT" (like "lo space cal") contain >>>> spaces? >>>> Is it OK for $SAGE_LOCAL to contain spaces? If it is so, why the >>>> differences? >>>> >>> >>> We do not systematically test this, but also SAGE_LOCAL should not >>> contain spaces. >>> >>> Many of our build scripts do not quote correctly and will break when >>> SAGE_LOCAL contains spaces. >>> For example, look at >>> https://github.com/sagemath/sage/blob/develop/build/pkgs/fplll/spkg-install.in >>> : >>> If SAGE_LOCAL is "/lo space cal", then when >>> CONFIGURE_FLAGS="--prefix=$SAGE_LOCAL --libdir=$SAGE_LOCAL/lib" is passed >>> to sdh_configure, it will expand to too many arguments. >>> >>> Fixing this would be a welcome contribution -- but likely also many >>> upstream projects have the same issues in their build scripts. >>> >>> >>> >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "sage-devel" group. >>> To unsubscribe from this group and stop receiving emails from it, send >>> an email to sage-devel+unsubscr...@googlegroups.com. >>> To view this discussion on the web visit >>> https://groups.google.com/d/msgid/sage-devel/bf505cca-95bc-4fe9-8aa7-4cfa736da55bn%40googlegroups.com >>> <https://groups.google.com/d/msgid/sage-devel/bf505cca-95bc-4fe9-8aa7-4cfa736da55bn%40googlegroups.com?utm_medium=email&utm_source=footer> >>> . >>> >> -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/sage-devel/CALZQg1ZkN4kHKhEMZAa2cMLCohHMp5uo5rjr0ymO42uN3ZieNw%40mail.gmail.com.
