Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script package _develop; improve installation instructions in the manual #33851

Closed
mkoeppe opened this issue May 14, 2022 · 50 comments
Closed

Script package _develop; improve installation instructions in the manual #33851

mkoeppe opened this issue May 14, 2022 · 50 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented May 14, 2022

(from #29536)

This is like _recommended (from #29557), but for development tools.

Define a script package representing "useful stuff for Sage developers to have on a Docker image":

We also use this package to define packages that would appear as "packages that developers should install" in
https://doc.sagemath.org/html/en/installation/source.html#linux-recommended-installation;
the new version from this ticket can be seen in https://c96c7c5813bd70d7b2d8d51135d57cc6acba18ad--sagemath-tobias.netlify.app/installation/source.html#system-specific-requirements

We also simplify the bootstrap-conda script and make it a bit faster.

Depends on #34301

CC: @tobiasdiez @dimpase @seblabbe @nbruin @egourgoulhon @slel

Component: packages: optional

Author: Matthias Koeppe

Branch/Commit: c39312c

Reviewer: Samuel Lelièvre

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

@mkoeppe mkoeppe added this to the sage-9.7 milestone May 14, 2022
@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Dependencies: #33740

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Branch: u/mkoeppe/script_package_develop

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Commit: 58e112a

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Author: Matthias Koeppe

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Last 10 new commits:

10338a7Change ci as well
0cf7334Add sage-setup as install-requires dep
6d2b149Use relative paths to not install src package
1dba3baMaybe thats the correct shebang?
c1b2fbfRevert install_requires change
ad56ba3Go back to two pip installs for now
b42f5f5Readd how to use specific python version
87f360cReadd name arg
01eb374Merge #33740
58e112abootstrap-conda: Use script package _develop instead of hardcoded list of dev tools; remove unused RECOMMENDED

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Changed commit from 58e112a to 21e06c5

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

21e06c5bootstrap-conda: Write an additional comment

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Changed dependencies from #33740 to #33740, #33825

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

4290b11Add pytest-xdist package
5c13643Merge #33825
46bf63dbuild/pkgs/_develop/dependencies: Add pytest, pytest_xdist

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Changed commit from 21e06c5 to 46bf63d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Changed commit from 46bf63d to 5857902

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

a365828build/pkgs/pytest/dependencies: Add missing dep
a2e396bbuild/pkgs/pytest_xdist/dependencies: Add missing dep
2ec0061Merge #33825
5857902build/pkgs/_develop/dependencies: Add pytest, pytest_xdist

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

2368adcbootstrap-conda: Write an additional comment (fixup)
0df31a9bootstrap-conda: Rewrite using fewer redirects

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Changed commit from 5857902 to 0df31a9

@mkoeppe

This comment has been minimized.

@tobiasdiez
Copy link
Contributor

comment:11

What's the point of introducing a file _develop/conda.txt whose content is only used in bootstrap-conda instead of hardcoding the content there? Consistency doesn't really out-weight the additional complexity in my opinion.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

comment:12

Well, there's also _develop/distros/debian.txt and more to be added.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

comment:13

These files are also used when configure generates information messages for the user etc.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 14, 2022

Changed dependencies from #33740, #33825 to #33740, #33825, #33852

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Changed commit from 0df31a9 to 58ab878

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

765dff7Merge #33740
ccbc21abootstrap, bootstrap-conda, src/doc/bootstrap: Use build/bin/sage-package directly
4560895src/doc/Makefile: Remove explicit use of SAGE_ROOT and the top-level sage script
7759f3cGenerate en/reference/repl/options.txt at build time, not bootstrap time
58ab878bootstrap-conda: Fix typo, generate->generating to match messages from bootstrap

@mkoeppe

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 15, 2022

Changed commit from 58ab878 to 12ea8ce

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 15, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

12ea8cebuild/pkgs/_develop/distros: Add packages providing gpgconf

@mkoeppe

This comment has been minimized.

@mkoeppe mkoeppe changed the title Script package _develop Script package _develop; improve installation instructions in the manual Aug 23, 2022
@slel
Copy link
Member

slel commented Aug 24, 2022

Reviewer: Samuel Lelièvre

@slel

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 24, 2022

comment:35

Thanks, Samuel!

@vbraun
Copy link
Member

vbraun commented Aug 27, 2022

comment:36

Merge failure on top of:

d862581a54e Trac #30787: package modular_resolution: Split out from p_group_cohomology

5821fb070e5 Trac #34421: Fix timeout in jupyter_jsmol installation

8df3aa074a4 Trac #34360: curl configure --without-libmetalink no longer works

500b135d73f Trac #34301: Remove claims that Cygwin is supported

fb050b1 Trac #34298: conda: 3d graphics do not show

20d7f18 Trac #34273: opensuse-tumbleweed: python3 build fails because of openssl

696bf78 Trac #34270: .gitpod.yml: Do not hardcode the workspace name as sagetrac-mirror

5b7fc7e Trac #25675: Crosslinks to poset catalog, add documentation of sage.geometry.polyhedron.base* and combinatorial_polyhedron

4285f3c Trac #34295: Followup to #33627: fix documentation that mentions sage-gdb-commands

08b5040 Trac #34294: SimplicialComplex: Add method is_subcomplex

9b5044a Trac #34289: minor tweaks in the doc

75ae420 Trac #34288: some rst fixes in pyx files in coding, graphs and groups

2f94ddf Trac #34293: rubiks: Work around build failure with XCode

945c339 Trac #34291: Downgrade some optional packages to experimental in Sage 9.7

a965858 Trac #34157: Meta ticket: fix docstring markups

e930477 Trac #34147: Implement the quantum Clifford algebra at a root of unity

75aaf28 Trac #33596: PolyhedralComplex.plot(explosion_factor=1)

e9efc9c Trac #33586: Triangulation.polyhedral_complex, boundary_simplicial_complex, boundary_polyhedral_complex

5247961 Trac #34221: Backport PEP420 namespace package support from Cython 3

12be2d9 Updated SageMath version to 9.7.beta8

merge was not clean: conflicts in src/doc/en/installation/source.rst

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 27, 2022

Dependencies: #34262

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 27, 2022

Changed dependencies from #34262 to #34301

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 27, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

393808csrc/doc/en/installation/binary.rst: No more cygwin binaries
fe98ca1README.md, src/doc/en/installation: Move all Cygwin instructions to one section of the installation manual
550b337README.md, src/doc/en/installation/index.rst: Remove mention of Cygwin
e3af6c9src/doc/en/installation/index.rst: Fix markup
c39312cMerge #34301

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 27, 2022

Changed commit from c96c7c5 to c39312c

@vbraun
Copy link
Member

vbraun commented Aug 30, 2022

Changed branch from u/mkoeppe/script_package_develop to c39312c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants