-
Notifications
You must be signed in to change notification settings - Fork 339
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 NO packages were installed #503
Comments
I've had this happen on and off too. On my system I have anaconda installed, and I have noticed that rez-pip completes an install of Qt.py into anaconda instead of a rez package. I wonder if this is an issue with anaconda, windows, or both. |
Running the command twice, results in rez telling me that the requirement is satisfied:
Which isn't really helpful in any scenario. The fact that I have a package on my system already shouldn't affect my ability to create a rez pacakge IMO. Just because it's on my system doesn't mean it's on everyone's system. That's rez's job. |
I've also had this happen on windows - I tried a few of my deps without success, then went searching on the google-group and tried installing the package that was being used as an example when rez-pip was launched (flask). That did mostly work, enough that I was able to use what worked as a base to create other packages. It's a bit of a chore, but it works well enough, and similar to the amount of effort I was putting in before rez.
# package.py
# -*- coding: utf-8
name = 'mock
version = '2.0.0'
requires = [
'six',
'funcsigs',
'pbr-1.3+',
'setuptools-17.1+',
]
def commands():
env.PYTHONPATH.append('{this.root}/python')
timestamp = 152391867
format_version = 2 |
I am currently bumping up against this problem attempting to 'rez-pip install celery'. I have downloaded a recent version of rez and attempted on both macOSX and Linux, but afraid I am receiving the same problem. Is there a viable workaround? Thanks! |
The current implementation of rez-pip does not really work on windows (unless you have a very specific setup and take extra special care). The reason is that the way it uses to retargets the install to a custom directory (--install-options=-install-lib etc) enforces use of source packages. That means everything that is not a pure python package needs a proper, working and matching compiler setup. And even then i doubt it would work out of the box simply because this is really non-trivial on windows. I have created an internal tool (sadly also called rez-pip currently heh) that takes a different approach that also works on windows (as long as binary packages are available). It creates an empty virtualenv, installs using standard pip methods, then uses distlib to get a list of installed distributions and files (ignoring the standard installed packages like setuptools) and converts them to rez-packages. It still has some rough edges (partly also in rez, see for example) and it also has some conceptual problems on windows that i am unsure how to work around. The biggest is, that setuptools/distlib uses .exe wrappers to create executables for scripts (e.g. bumpversion.exe). These are sadly hardwired to the python executable they were created with, which creates a hard dependency between the package and the python version rez-pip was called with. This could of course be wired up to be Rez' python, but in our setup we split Rez from python and hence switching to a newly released rez with a different python version associated could break these executables, removing the python version it was deployed with definitely breaks them. Any input on that issue would be really welcome. Cheers, |
Thank you for the feedback Thorsten. We currently do not have to support Windows, but at this time, only MacOSX and Linux. Are you suggesting your solution could work in these environments as well? |
I have no idea, but i doubt it to be honest. This has currently been developed and tested only on windows. |
Hi,
when installing Qt.py package with rez-pip, it fails.
The text was updated successfully, but these errors were encountered: