Skip to content

Commit

Permalink
setup: do not build the C backend on PyPy
Browse files Browse the repository at this point in the history
This should have always been the case. But it surprisingly worked up until
e5a3baf when a new header file referenced
CPython symbols not present in PyPy.

Closes #130.
  • Loading branch information
indygreg committed Dec 31, 2020
1 parent 8e4702a commit 21fba52
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
8 changes: 8 additions & 0 deletions docs/news.rst
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,14 @@ Other Actions Not Blocking Release
0.16.0 (not yet released)
=========================

0.15.1 (not yet released)
=========================

Bug Fixes
---------

* ``setup.py`` no longer attempts to build the C backend on PyPy. (#130)

0.15.0 (released 2020-12-29)
============================

Expand Down
5 changes: 5 additions & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
from __future__ import print_function

from distutils.version import LooseVersion
import platform
import os
import sys
from setuptools import setup
Expand Down Expand Up @@ -52,6 +53,10 @@
if os.environ.get("ZSTD_WARNINGS_AS_ERRORS", ""):
WARNINGS_AS_ERRORS = True

# PyPy doesn't support the C backend.
if platform.python_implementation() == "PyPyy":
C_BACKEND = False

if "--legacy" in sys.argv:
SUPPORT_LEGACY = True
sys.argv.remove("--legacy")
Expand Down

0 comments on commit 21fba52

Please sign in to comment.