search

infoscout / weighted-levenshtein

Weighted Levenshtein library
MIT License
105 stars 26 forks source link

Fix sdist install for python >=3.9 #31

Closed JacobHayes closed 1 year ago

JacobHayes commented 2 years ago

This removes the clev.c file from the sdists, which was generated against an older cython version and references APIs removed in python 3.9. By removing the .c file from the dist, a fresh one will be generated for each install for the correct python version (and likely newer cython versions).

I also switched the build requirements over to PEP 517, which silences this warning during python3 setup.py sdist:

SetuptoolsDeprecationWarning: setuptools.installer is deprecated. Requirements should be satisfied by a PEP 517 installer.

Closes #25

JacobHayes commented 1 year ago

@pachewise do you think you'd have time to look at this? Thanks!

JacobHayes commented 1 year ago

Thanks @pachewise and sorry to bother you then! Perhaps @infoscoutcorp or @RevolutionTech could take a look if you're still there? 😅

Otherwise, do either of you know anyone still there who might be able to merge and release this? Thanks!

RevolutionTech commented 1 year ago

I'm afraid I can't help -- I don't work at InfoScout/Numerator anymore either.