m2r2 icon indicating copy to clipboard operation
m2r2 copied to clipboard

Fix missing setuptools dependency

Open FlotterCodername opened this issue 2 years ago • 1 comments

Fixes https://github.com/CrossNox/m2r2/issues/63

FlotterCodername avatar Nov 06 '23 13:11 FlotterCodername

Alternatively importlib.metadata could be used, its not available in the now EOL python3.7 but it does have a backport.

pkg_resources is discouraged either way by setuptools https://setuptools.pypa.io/en/latest/pkg_resources.html.

The use here is very minimal and can be changed like I did here https://github.com/nicolasbock/ebuildtester/pull/212/commits/e95c34296eb56d55ffca5b38947602e91d73729c

Only minus is python 3.7 requiring a backport, but python3.7 should be dropped from the CI by now already...

parona-source avatar Nov 24 '23 03:11 parona-source

pkg_resources is dropped in setuptools 70, so it should either be pinned to <70 or, better, use importlib.

oscargus avatar Jun 01 '24 13:06 oscargus

pkg_resources is dropped in setuptools 70, so it should either be pinned to <70 or, better, use importlib.

Thanks for the headsup! In this case I'm going to close the PR. I think pinning a package like setuptools is not acceptable. With no response from maintainers in half a year it appears hopeless to get a merge for the correct approach. I've migrated everything to MyST already anyway. The success of MyST is probably also a reason why this package now appears abandoned.

FlotterCodername avatar Jun 04 '24 15:06 FlotterCodername