Simplest pyproject.toml containing build-system only by ianthomas23 · Pull Request #24102 · matplotlib/matplotlib · GitHub
Skip to content

Simplest pyproject.toml containing build-system only - #24102

Merged
tacaswell merged 7 commits into
matplotlib:mainfrom
ianthomas23:simple_pyproject
Oct 12, 2022
Merged

Simplest pyproject.toml containing build-system only#24102
tacaswell merged 7 commits into
matplotlib:mainfrom
ianthomas23:simple_pyproject

Conversation

@ianthomas23

@ianthomas23 ianthomas23 commented Oct 6, 2022

Copy link
Copy Markdown
Member

This is the start of a transition to a pyproject.toml build system (issue #23815) and is a alternative to (or precursor to) PR #23829.

This is the only part of the pyproject.toml transition that is required for the pybind11 progress to continue. So after this is merged work can continue on both pyproject.toml and pybind11 independently.

@ianthomas23

ianthomas23 commented Oct 7, 2022 •
edited
Loading

Copy link
Copy Markdown
Member Author

@ianthomas23

ianthomas23 commented Oct 11, 2022

Copy link
Copy Markdown
Member Author

This is now working as expected, and is the smallest set of changes needed to use pyproject.toml's [build-system]. All of the GitHub Action test runs except for the minimum versions one now use build isolation, i.e. build in a temporary isolated virtual environment to create a wheel and that wheel is then installed in the current working environment. The minimum versions test run does not use build isolation so that it builds against the already installed minimum versions of dependencies.

ianthomas23
ianthomas23 commented Oct 11, 2022
View reviewed changes
Comment thread setup.py
super().run()
update_matplotlibrc(
Path(self.build_lib, "matplotlib/mpl-data/matplotlibrc"))
if not self.editable_mode:

@ianthomas23 ianthomas23 Oct 11, 2022

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It surprised me to discover that when you build the main branch in editable mode this run function is not called. This branch does call the run function so here I am explicitly avoiding the update_matplotlibrc call when in editable mode to give the same behaviour as main branch.

@timhoffm timhoffm Oct 12, 2022

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Editable (via pip/setuptools) is quite different from a regular build. I'm not an expert there, but have the feeling that if in doubt expectations on editable installs do not hold. Possibly pep660 will change that.

@tacaswell tacaswell added this to the v3.7.0 milestone Oct 12, 2022
@tacaswell
tacaswell merged commit af39f1e into matplotlib:main Oct 12, 2022
@tacaswell

tacaswell commented Oct 12, 2022

Copy link
Copy Markdown
Member

@ianthomas23 ianthomas23 mentioned this pull request Oct 16, 2022
Closed
13 tasks
@ianthomas23 ianthomas23 mentioned this pull request Nov 5, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@timhoffm timhoffm timhoffm left review comments
@oscargus oscargus oscargus approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v3.7.0

Development

Successfully merging this pull request may close these issues.

4 participants

@ianthomas23 @tacaswell @timhoffm @oscargus