Andrei Kulakov <andrei....@gmail.com> added the comment:

This warning can be fixed by changing the following line:

https://github.com/python/cpython/blob/9d1c4d69dbc800ac344565119337fcf490cdc800/Lib/importlib/_bootstrap_external.py#L1419

to:

        if not path and str(path) == '':

and running `make regen-importlib; make`

Alternatively the change can be:

        if not path and isinstance(path, (str,bytes)):

I'm not sure which is preferable or if some other fix would be better (I don't 
know much about importlib).

===
Adding Brett as the expert -- Brett, can you take a look?

Adding a full traceback with -bb argument:

Traceback (most recent call last):
  File "/Users/ak/temp2/zipfile_demo.py", line 29, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/ak/opensource/cpython4/Lib/contextlib.py", line 155, in __exit__
    self.gen.throw(typ, value, traceback)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/ak/temp2/zipfile_demo.py", line 12, in _tmp_path
    yield pathlib.Path(tmp_dir)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/ak/temp2/zipfile_demo.py", line 24, in main
    import module
    ^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1140, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1080, in _find_spec
  File "<frozen importlib._bootstrap_external>", line 1493, in find_spec
  File "<frozen importlib._bootstrap_external>", line 1462, in _get_spec
  File "<frozen importlib._bootstrap_external>", line 1418, in 
_path_importer_cache
BytesWarning: Comparison between bytes and string

----------
nosy: +andrei.avk, brett.cannon

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue47026>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to