Skip to content

Commit

Permalink
Merge branch 'pr/tornaria/37287' into conda-update
Browse files Browse the repository at this point in the history
  • Loading branch information
tobiasdiez authored Feb 24, 2024
2 parents 4afc299 + e280cba commit a9d89e7
Show file tree
Hide file tree
Showing 7 changed files with 12 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .devcontainer/onCreate-conda.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,5 @@ conda init bash
# Build sage
conda run -n sage-dev ./bootstrap
conda run -n sage-dev ./configure --with-python=/opt/conda/envs/sage-dev/bin/python --prefix=/opt/conda/envs/sage-dev
conda run -n sage-dev pip install --no-build-isolation -v -v -e ./pkgs/sage-conf ./pkgs/sage-setup
conda run -n sage-dev pip install --no-build-isolation -v -v -e ./pkgs/sage-conf
conda run -n sage-dev pip install --no-build-isolation -v -v -e ./src
4 changes: 2 additions & 2 deletions .github/workflows/ci-conda.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ jobs:
shell: bash -l {0}
run: |
# Use --no-deps and pip check below to verify that all necessary dependencies are installed via conda.
pip install --no-build-isolation --no-deps -v -v -e ./pkgs/sage-setup
pip install --no-build-isolation --no-deps -v -v -e ./src
pip install --no-build-isolation --no-deps -v -v -e ./pkgs/sage-conf
pip install --no-build-isolation --no-deps --config-settings editable_mode=compat -v -v -e ./src
env:
SAGE_NUM_THREADS: 2

Expand Down
2 changes: 1 addition & 1 deletion .gitpod.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ tasks:
&& conda activate $(pwd)/venv
&& ./bootstrap
&& ./configure --enable-build-as-root --with-python=$CONDA_PREFIX/bin/python --prefix=$CONDA_PREFIX
&& pip install --no-build-isolation -v -v -e ./pkgs/sage-conf ./pkgs/sage-setup
&& pip install --no-build-isolation -v -v -e ./pkgs/sage-conf
&& pip install --no-build-isolation -v -v -e ./src
# Activate conda environment, set up Trac remote
# RestructuredText extension recommends python extension, although we have already installed it
Expand Down
1 change: 1 addition & 0 deletions pkgs/sagemath-standard/sage_setup
7 changes: 6 additions & 1 deletion src/MANIFEST.in
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,15 @@ include VERSION.txt
global-include *.pxi *.pxd *.h *.hpp

prune sage/ext/interpreters # In particular, __init__.py must not be present in the distribution; or sage_setup.autogen.interpreters.rebuild will not generate the code
prune sage_setup
prune sage_docbuild
prune doc

# include sage_setup
recursive-include sage_setup *
prune sage_setup/autogen/flint*
exclude sage_setup/autogen/flint*
exclude sage_setup/autogen/giacpy*

#
# Most C and C++ files are generated by Cython and should not
# be included in the sdist.
Expand Down
2 changes: 1 addition & 1 deletion src/doc/en/installation/conda.rst
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ Here we assume that you are using a git checkout.
- Bootstrap the source tree and install the build prerequisites and the Sage library::

$ ./bootstrap
$ pip install --no-build-isolation -v -v --editable ./pkgs/sage-conf_conda ./pkgs/sage-setup
$ pip install --no-build-isolation -v -v --editable ./pkgs/sage-conf_conda
$ pip install --no-build-isolation --config-settings editable_mode=compat -v -v --editable ./src

- Verify that Sage has been installed::
Expand Down
2 changes: 0 additions & 2 deletions src/pyproject.toml.m4
Original file line number Diff line number Diff line change
@@ -1,15 +1,13 @@
[build-system]
# Minimum requirements for the build system to execute.
requires = [
"sage_setup[autogen]",
# Some version of sage-conf is required.
# Note that PEP517/518 have no notion of optional sage_spkg dependencies:
# https://github.com/pypa/pip/issues/6144
esyscmd(`sage-get-system-packages install-requires-toml \
sage_conf \
setuptools \
wheel \
sage_setup \
cypari \
cysignals \
cython \
Expand Down

0 comments on commit a9d89e7

Please sign in to comment.