Skip to content

Commit

Permalink
remove extras require dead code that never worked in the first place
Browse files Browse the repository at this point in the history
Signed-off-by: Stephen L. <lrq3000@gmail.com>
  • Loading branch information
lrq3000 committed Apr 2, 2023
1 parent 602c432 commit 0a62119
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ Download = "https://github.com/tomerfiliba/reedsolomon/releases"
#Changelog = "https://url/changelog"

#[project.optional-dependencies]
#cythonize = [
#cythonize = [ # cython and other build time dependencies can NOT be defined as optional-dependencies nor extras, they are installed only after build! They need to be defined as build-system requirements
# "cython>=3.0.0b2"
#]

Expand Down
3 changes: 0 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,6 @@
}, # see: https://stackoverflow.com/questions/61156290/how-to-set-project-links-in-pypi and https://github.com/pypi/warehouse/blob/main/warehouse/templates/packaging/detail.html
py_modules = ["reedsolo"],
platforms = ["any"],
#extras_require={
#"cythonize": ["cython>=3.0.0b2"], # see: https://hynek.me/articles/python-recursive-optional-dependencies/ and https://copyprogramming.com/howto/optional-dependencies-in-a-pip-requirements-file
#},
long_description = open("README.rst", "r").read(),
long_description_content_type = 'text/x-rst',
license_files = ('LICENSE',), # force include LICENSE file, requires setuptools >= 42.0.0. Note that this field only support one line text, do not input the full license content here. The full LICENSE file is currently forcefully included via MANIFEST.in, but other methods exist, see: https://stackoverflow.com/a/66443941/1121352
Expand Down

0 comments on commit 0a62119

Please sign in to comment.