Opened 2 years ago
Last modified 22 months ago
#29850 closed enhancement
Install sageenvconfig as part of sage_conf; make sageenvconfig optional — at Version 23
Reported by:  Matthias Köppe  Owned by:  

Priority:  major  Milestone:  sage9.3 
Component:  build  Keywords:  sd111 
Cc:  John Palmieri, François Bissey, Michael Orlitzky, Dima Pasechnik, Isuru Fernando, Tobias Diez  Merged in:  
Authors:  Matthias Koeppe  Reviewers:  
Report Upstream:  N/A  Work issues:  
Branch:  u/mkoeppe/install_sage_env_config_with_sage_conf (Commits, GitHub, GitLab)  Commit:  709ebca31bd68a64865b6251b264fcd5a2e38aa7 
Dependencies:  #21559  Stopgaps: 
Description (last modified by )
As a followup to #21559 and #30128, we reassign sageenvconfig
from the package sagelib
to the package sage_conf
.
Because from the viewpoint of sagelib
, the package sage_conf
is optional, we change various scripts that source sageenvconfig
so that this file is not required.
Part of Metaticket #21707: Split sageenv into 5 to clean up sage configuration
Change History (23)
comment:1 Changed 2 years ago by
Dependencies:  → #29825 

Description:  modified (diff) 
comment:2 Changed 2 years ago by
Dependencies:  #29825 → #29825, #21559 

comment:3 Changed 2 years ago by
Branch:  → u/mkoeppe/install_sage_env_config_with_sage_conf 

comment:4 followups: 7 9 Changed 2 years ago by
Commit:  → dacea8b8d9a58555003d425957093b11ad0952fd 

Dependencies:  #29825, #21559 → #29825, #21559, #29803 
comment:5 Changed 2 years ago by
Perhaps it's easier to get rid of sageenvconfig altogether (#29852).
comment:6 Changed 2 years ago by
Summary:  Install sageenvconfig with sage_conf → Install sageenvconfig as part of sage_conf 

comment:7 Changed 2 years ago by
comment:8 Changed 2 years ago by
Commit:  dacea8b8d9a58555003d425957093b11ad0952fd → 5f7b1cc40b66a94030e3e53607e2f0350363e40a 

comment:9 Changed 2 years ago by
Replying to mkoeppe:
Branch on this ticket does NOT work because pip/setuptools do not handle the symlink well.
It would probably suffice to just replace the use of pip
by setup.py install
.
comment:10 Changed 2 years ago by
Cc:  John Palmieri François Bissey added 

Description:  modified (diff) 
comment:11 Changed 2 years ago by
Work issues:  → use setup.py install, add spkgsrc 

comment:12 Changed 2 years ago by
Commit:  5f7b1cc40b66a94030e3e53607e2f0350363e40a → 308ca6971af903adf4822e5349ef665cddc1e08a 

comment:13 Changed 2 years ago by
Work issues:  use setup.py install, add spkgsrc → make sageenvconfig optional for sageenv 

comment:14 Changed 2 years ago by
Description:  modified (diff) 

Work issues:  make sageenvconfig optional for sageenv 
comment:15 Changed 2 years ago by
Authors:  → Matthias Koeppe 

Status:  new → needs_review 
comment:16 Changed 2 years ago by
Work issues:  → Move sage_conf.py to SAGE_ROOT/src/, symlink to there 

comment:18 Changed 2 years ago by
Work issues:  Move sage_conf.py to SAGE_ROOT/src/, symlink to there 

comment:19 Changed 2 years ago by
Commit:  308ca6971af903adf4822e5349ef665cddc1e08a → 03301806c3b9d41fdfaf576fd84ac887f4156e1a 

Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
3a0193c  src/bin/sage: Remove handling of 'sage axiom'

6b04075  Merge branch 't/29111/specify_a_subset_of_sage_command_line_options_that_are_supported_by_sagelib___rather_than_sage_the_distribution' into t/21559/changesrcbininstallation

9c7116b  src/bin/sagelistoptional, sagelistexperimental, sageliststandard: Remove deprecated scripts

831cc09  Merge branch 't/29920/remove_deprecated_scripts_sage_list_optional__sage_list_experimental__sage_list_standard' into t/21559/changesrcbininstallation

e3eca85  Merge branch 'public/move_all_code_from_src_setup_py__src_fpickle_setup_py_to_sage_setup' of git://trac.sagemath.org/sage into t/21559/changesrcbininstallation

7d29141  src/setup.py: Do not install removed script sagersyncdist

c5f7f2f  Merge tag '9.2.beta3' into t/21559/changesrcbininstallation

6bc0cd4  Install sageenvconfig with sage_conf

5e6ca95  build/pkgs/sage_conf/spkginstall: Install with setup.py instead of pip

0330180  build/pkgs/sage_conf/spkgsrc: New

comment:20 Changed 2 years ago by
Dependencies:  #29825, #21559, #29803 → #21559 

comment:21 Changed 2 years ago by
Commit:  03301806c3b9d41fdfaf576fd84ac887f4156e1a → 709ebca31bd68a64865b6251b264fcd5a2e38aa7 

Branch pushed to git repo; I updated commit sha1. New commits:
3953671  Merge tag '9.2.beta4' into t/29111/specify_a_subset_of_sage_command_line_options_that_are_supported_by_sagelib___rather_than_sage_the_distribution

19d7f67  Merge branch 't/29111/specify_a_subset_of_sage_command_line_options_that_are_supported_by_sagelib___rather_than_sage_the_distribution' into t/21559/changesrcbininstallation

709ebca  Merge branch 't/21559/changesrcbininstallation' into t/29850/install_sage_env_config_with_sage_conf

comment:22 Changed 2 years ago by
Cc:  Michael Orlitzky added 

comment:23 Changed 2 years ago by
Description:  modified (diff) 

Summary:  Install sageenvconfig as part of sage_conf → Install sageenvconfig as part of sage_conf; make sageenvconfig optional 
Branch on this ticket does NOT work because pip/setuptools do not handle the symlink well.
Last 10 new commits:
build/pkgs/pip/dependencies: Add wheel, simplify $(PYTHON) python3 to $(PYTHON)
build/pkgs/setuptools, build/pkgs/pip: Remove python2 support from dependencies, install scripts
build/pkgs/pkgconfig/spkginstall.in: Remove pyproject.toml so that pip installs using setup.py
build/pkgs/entrypoints: Do not provide setup.py, not needed for entrypoints 0.3
build/pkgs/entrypoints/spkginstall.in: Remove pyproject.toml so that setup.py is used
build/pkgs/terminado/spkginstall.in: Remove pyproject.toml so that setup.py is used
build/pkgs/testpath/spkginstall.in: Remove pyproject.toml so that setup.py is used
build/pkgs/zope_interface: Change to a dummy script package
Merge branch 't/29803/upgrade_setuptools__pip__2020_06_' into t/29850/install_sage_env_config_with_sage_conf
WIP: Install sageenvconfig with sage_conf