search

suds-community / suds

Suds is a lightweight SOAP python client for consuming Web Services. A community fork of the jurko fork.
https://suds.readthedocs.io/
GNU Lesser General Public License v3.0
172 stars 54 forks source link

setuptools.command.test deprecated (is now a stub) #105

Closed vishwin closed 1 month ago

vishwin commented 1 month ago

https://github.com/suds-community/suds/blob/ff8a698af1255e176e4d7a740704d695b41655c8/setup.py#L411

package now fails to build because setuptools.command.test became a stub to serve as a softer but still forceful deprecation notice, see pypa/setuptools#4522 pypa/setuptools#4519 pypa/setuptools#4520

phillbaker commented 1 month ago

Thanks, reading those summaries, my understanding is that v72.0.0 of setuptools is yanked and this package is installable with 72.1.0. The new due date for removing is 15 Dec 2024, see pypa/setuptools#4522 https://github.com/pypa/setuptools/pull/4522. Given that, we should do this, but it's not super urgent.

On Sun, Aug 11, 2024 at 12:58 AM Charlie Li @.***> wrote:

https://github.com/suds-community/suds/blob/ff8a698af1255e176e4d7a740704d695b41655c8/setup.py#L411

package now fails to build because setuptools.command.test became a stub to serve as a softer but still forceful deprecation notice, see pypa/setuptools#4522 https://github.com/pypa/setuptools/pull/4522 pypa/setuptools#4519 https://github.com/pypa/setuptools/issues/4519 pypa/setuptools#4520 https://github.com/pypa/setuptools/issues/4520

— Reply to this email directly, view it on GitHub https://github.com/suds-community/suds/issues/105, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXCKMNKY2J74BCN3TG3ATZQ3VOPAVCNFSM6AAAAABMKNPFB6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TSNBSGMYDMNA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

vishwin commented 1 month ago

The stub refers to 72.1.0. The error here becomes an ImportError on _normalize_path rather than an outright not found on setuptools.command.test, so building still fails.

vishwin commented 1 month ago

Still fails on 72.2.0.

phillbaker commented 1 month ago

Can you share the command you’re running that fails and the error?

On Tue, Aug 13, 2024 at 3:09 PM Charlie Li @.***> wrote:

Still fails on 72.2.0.

— Reply to this email directly, view it on GitHub https://github.com/suds-community/suds/issues/105#issuecomment-2286951040, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXCKI4LK3336TA4HMTIW3ZRJKXVAVCNFSM6AAAAABMKNPFB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWHE2TCMBUGA . You are receiving this because you commented.Message ID: @.***>

vishwin commented 1 month ago

In this case setup.py config, but literally any setup.py invocation will fail in the same manner, simply because the referenced code exists.

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "setup.py", line 509, in <module>
    from setuptools.command.test import (normalize_path as _normalize_path,
ImportError: cannot import name 'normalize_path' from 'setuptools.command.test' (/usr/local/lib/python3.11/site-packages/setuptools/command/test.py)

The entire testing "framework" in setup.py needs removed in order to fix this. pytest et al can and should be invoked outside of setuptools anyway.