Bad setup.cfg causes new setuptools to stop installing kylinpy

[potiuk]

The setup.py / .cfg of yours caused failures when trying to install kylinpy after setuptools 78.0.1 has been released in Airfow CI.

While this release will be likely yanked/replaced with a new version, it is also a signal that you should fix your build configuration because sooner or later it will stop working.

Lack of doing it, will mean that your project is essentially abandoned and we will remove it from Airflow dependencies soon.

See pypa/setuptools#4910

[zhaoyongjie]

@liyang-kylin Do we decide mark this library as legacy, or evolve new contributor to continue working on it?

[arcadecoffee]

@potiuk I'm actually not even able to get this module to install if I rollback setuptools. It's kind of breaking on of my Airflow deployment pipelines at the moment. Any advice on the best way to exclude or a workaround?

[zhaoyongjie]

@potiuk and @arcadecoffee, I am a Superset PMC member and the original creator of Kylinpy. I will directly contact Kyligence Inc. in person this week and keep you updated in this thread. I promise to provide a proper solution for this issue.

[arcadecoffee]

@potiuk and @arcadecoffee, I am a Superset PMC member and the original creator of Kylinpy. I will directly contact Kyligence Inc. in person this week and keep you updated in this thread. I promise to provide a proper solution for this issue.

FWIW, it looks like the new setuptools release 78.0.2 has things working for now.

[potiuk]

FWIW, it looks like the new setuptools release 78.0.2 has things working for now.

Yeah. 78.0.2 fixed things, but regardless, it would be great to fix the setup config for the future.

[potiuk]

BTW. I am going to make a PR fixing it, but someone will have to release the package

[potiuk]

Actualy @zhaoyongjie @arcadecoffee -> it looks like the planned kylinpy 2.8.5 already has the problem fixed. I looked up in main and https://github.com/Kyligence/kylinpy/blob/master/setup.cfg#L5 already has description_file with _ rather than - .

So this is really just a matter of releasing it to PyPI