Disable more packages with various --disable-foo combinations #36239
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
orlitzky
changed the title
orlitzky
force-pushed
the
more-disable-combinations
branch
from
4e84189
to
4b28328
Compare
mkoeppe
added
s: positive review
c: build: configure
and removed
s: needs review
labels
|
merge conflict |
The pytz package is a dependency of three others: babel, rpy2, and tzlocal. And tzlocal is itself a dependency of rpy2. Babel is already disabled with --disable-doc, and rpy2 and tzlocal are disabled with --disable-r. It therefore makes sense to disable pytz when both --disable-doc and --disable-r are given.
The pycparser package is a dependency of two others, cffi and rpy2, which can be disabled with --disable-notebook and --disable-r respectively. As a result we can disable pycparser when both flags are present.
The pytz_deprecation_shim package is a dependency only of tzlocal, which is already disabled with --disable-r.
Passing --disable-notebook to ./configure already disables the argon2_cffi package, so it should be safe to disable its bindings as well.
The soupsieve package is a dependency of beautifulsoup4, which is already disabled with --disable-notebook. It should therefore be safe to disable soupsieve under the same circumstances.
The sphinx_basic_ng package is solely a dependency of furo, which is already disabled with --disable-doc. We should therefore disable sphinx_basic_ng with --disable-doc as well.
The fastjsonschema package is solely a dependency of nbformat, which is already disabled with --disable-notebook. We should therefore disable fastjsonschema with --disable-notebook as well.
orlitzky
force-pushed
the
more-disable-combinations
branch
from
4b28328
to
c59c407
Compare
|
force-pushed the merged commits on top of develop |
|
Documentation preview for this PR (built with commit c59c407; changes) is ready! 🎉 |
5 tasks
vbraun
pushed a commit
to vbraun/sage
that referenced
this pull request
Sep 25, 2023
… wheel packages, drop build deps
<!-- ^^^^^
Please provide a concise, informative and self-explanatory title.
Don't put issue numbers in there, do this in the PR body below.
For example, instead of "Fixes sagemath#1234" use "Introduce new method to
calculate 1+1"
-->
<!-- Describe your changes here in detail -->
<!-- Why is this change required? What problem does it solve? -->
<!-- If this PR resolves an open issue, please link to it here. For
example "Fixes sagemath#12345". -->
- Cherry-picked from sagemath#36129
<!-- If your change requires a documentation PR, please link it
appropriately. -->
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
<!-- If your change requires a documentation PR, please link it
appropriately -->
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
<!-- Feel free to remove irrelevant items. -->
- [x] The title is concise, informative, and self-explanatory.
- [ ] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation accordingly.
### ⌛ Dependencies
<!-- List all open PRs that this PR logically depends on
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->
- Depends on sagemath#36239 (merged here)
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
URL: sagemath#36267
Reported by: Matthias Köppe
Reviewer(s): Dima Pasechnik, github-actions[bot]
vbraun
pushed a commit
to vbraun/sage
that referenced
this pull request
Sep 27, 2023
… wheel packages, drop build deps
<!-- ^^^^^
Please provide a concise, informative and self-explanatory title.
Don't put issue numbers in there, do this in the PR body below.
For example, instead of "Fixes sagemath#1234" use "Introduce new method to
calculate 1+1"
-->
<!-- Describe your changes here in detail -->
<!-- Why is this change required? What problem does it solve? -->
<!-- If this PR resolves an open issue, please link to it here. For
example "Fixes sagemath#12345". -->
- Cherry-picked from sagemath#36129
<!-- If your change requires a documentation PR, please link it
appropriately. -->
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
<!-- If your change requires a documentation PR, please link it
appropriately -->
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
<!-- Feel free to remove irrelevant items. -->
- [x] The title is concise, informative, and self-explanatory.
- [ ] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation accordingly.
### ⌛ Dependencies
<!-- List all open PRs that this PR logically depends on
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->
- Depends on sagemath#36239 (merged here)
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
URL: sagemath#36267
Reported by: Matthias Köppe
Reviewer(s): Dima Pasechnik, github-actions[bot]
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Prune more leaf packages with
--disable-doc,--disable-notebook,--disable-r, and combinations thereof.