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

Require toml extra for setuptools_scm #12

Merged
merged 4 commits into from
Jan 23, 2020
Merged

Require toml extra for setuptools_scm #12

merged 4 commits into from
Jan 23, 2020

Conversation

layday
Copy link
Contributor

@layday layday commented Jan 21, 2020

setuptools_scm does not know to invoke itself
if it can't read pyproject.toml. This broke
sdist installs for projects deriving from skeleton:

$ python -m pip install zipp --no-binary zipp
Collecting zipp
  [...]
Successfully installed zipp-0.0.0

Note the version number defaulting to '0.0.0'.
Building locally only works because pep517,
the build tool, depends on toml which it exposes
to the build environment.

setuptools_scm does not know to invoke itself
if it can't read pyproject.toml.  This broke
sdist installs for projects deriving from skeleton:

  $ python -m pip install zipp --no-binary zipp
  Collecting zipp
    [...]
  Successfully installed zipp-0.0.0

Note the version number defaulting to '0.0.0'.
Building locally only works because pep517,
the build tool, depends on toml which it exposes
to the build environment.
A bare

  [tool.setuptools_scm]

does not work in 3.4.0.
@layday
Copy link
Contributor Author

layday commented Jan 21, 2020

Running pip wheel with SETUPTOOLS_SCM_DEBUG=1 and the verbose flag reveals that this produces an error which is ignored:

Traceback (most recent call last):
  File "/private/var/folders/18/r2nky_v137d3fwhtx0644m940000gn/T/pip-build-env-y35hsj2d/overlay/lib/python3.8/site-packages/setuptools_scm/integration.py", line 45, in infer_version
    config = Configuration.from_file()
  File "/private/var/folders/18/r2nky_v137d3fwhtx0644m940000gn/T/pip-build-env-y35hsj2d/overlay/lib/python3.8/site-packages/setuptools_scm/config.py", line 121, in from_file
    defn = __import__("toml").load(strm)
ModuleNotFoundError: No module named 'toml'

@jaraco
Copy link
Owner

jaraco commented Jan 23, 2020

Oh, dear! I had forgotten to include that extra. Thanks for this. (Squashing to limit noise in the skeleton).

@jaraco jaraco merged commit 29c4972 into jaraco:master Jan 23, 2020
This was referenced Jan 23, 2020
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Mar 12, 2020
v3.1.0
======

#47: ``.open`` now raises ``FileNotFoundError`` and
``IsADirectoryError`` when appropriate.

v3.0.0
======

#44: Merge with v1.2.0.

v1.2.0
======

#44: ``zipp.Path.open()`` now supports a compatible signature
as ``pathlib.Path.open()``, accepting text (default) or binary
modes and soliciting keyword parameters passed through to
``io.TextIOWrapper`` (encoding, newline, etc). The stream is
opened in text-mode by default now. ``open`` no
longer accepts ``pwd`` as a positional argument and does not
accept the ``force_zip64`` parameter at all. This change is
a backward-incompatible change for that single function.

v2.2.1
======

#43: Merge with v1.1.1.

v1.1.1
======

#43: Restored performance of implicit dir computation.

v2.2.0
======

#36: Rebuild package with minimum Python version declared both
in package metadata and in the python tag.

v2.1.0
======

#32: Merge with v1.1.0.

v1.1.0
======

#32: For read-only zip files, complexity of ``.exists`` and
``joinpath`` is now constant time instead of ``O(n)``, preventing
quadratic time in common use-cases and rendering large
zip files unusable for Path. Big thanks to Benjy Weinberger
for the bug report and contributed fix (#33).

v2.0.1
======

#30: Corrected version inference (from jaraco/skeleton#12).

v2.0.0
======

Require Python 3.6 or later.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants