Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rez-pip error when using python3.7 #906

Open
bpabel opened this issue Jul 17, 2020 · 0 comments
Open

rez-pip error when using python3.7 #906

bpabel opened this issue Jul 17, 2020 · 0 comments
Labels
bug rez-pip ingesting py pkgs into rez (pip, wheels, etc)

Comments

@bpabel
Copy link
Contributor

bpabel commented Jul 17, 2020

System

  • Windows 10
  • rez 2.61.1
  • python 3.7

Error

$ rez-pip -i -r --python-version 2.7 ampq
... [trim some site information]
Collecting amqp
  Downloading amqp-2.6.0-py2.py3-none-any.whl (47 kB)
     |UUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUU| 47 kB 280 kB/s
Collecting vine<5.0.0a1,>=1.1.3
  Downloading vine-1.3.0-py2.py3-none-any.whl (14 kB)
Installing collected packages: vine, amqp
Successfully installed amqp-2.6.0 vine-1.3.0
WARNING: You are using pip version 20.1; however, version 20.1.1 is available.
You should consider upgrading via the 'c:\python27\python.exe -m pip install --upgrade pip' command.
21:57:18 INFO     Installed [amqp-2.6.0] i:\tools\rez\packages\release\amqp\2.6.0\package.py (ff5a17a870e473adea6d65972631222d54a381e6)
Traceback (most recent call last):
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\serialise.py", line 72, in open_file_for_write
    f.write(content)
  File "C:\Program Files\rez\2.61.1\Python\Lib\contextlib.py", line 119, in __exit__
    next(self.gen)
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\vendor\atomicwrites\__init__.py", line 154, in _open
    self.commit(f)
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\vendor\atomicwrites\__init__.py", line 185, in commit
    replace_atomic(f.name, self._path)
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\vendor\atomicwrites\__init__.py", line 92, in replace_atomic
    return _replace_atomic(src, dst)
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\vendor\atomicwrites\__init__.py", line 75, in _replace_atomic
    _windows_default_flags | _MOVEFILE_REPLACE_EXISTING
  File "c:\progra~1\rez\261~1.1\lib\site-packages\rez\vendor\atomicwrites\__init__.py", line 70, in _handle_errors
    raise WinError()
OSError: [WinError 17] The system cannot move the file to a different disk drive.

Description

Looks like serialise.py is set up to catch a WindowsError, but the vendored atomicwrites raises an OSError in python 3.7

WindowsError merged into OSError in python 3.3

@JeanChristopheMorinPerso JeanChristopheMorinPerso added bug rez-pip ingesting py pkgs into rez (pip, wheels, etc) labels Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug rez-pip ingesting py pkgs into rez (pip, wheels, etc)
Projects
None yet
Development

No branches or pull requests

2 participants