-
Notifications
You must be signed in to change notification settings - Fork 906
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
portalocker issue #832
Comments
seems that there are issues with that version that are yet unresolved. and some comments explicitely call @mdipierro to an action :P wolph/portalocker#2 |
I'd propose to always import the contrib version of portalocker until a new contrib version (inline with the pip version) is provided. |
sadly, it's nothing new. web2py imports should always be used with gluon as the base, but up until now those kind of issues are not surfacing because web2py prepends the gluon path to sys.path. To be fair, this means that even if you have a pip-installed version of portalocker, inside web2py when you do "import portalocker" you're loading the gluon/portalocker.py one. |
It seems that isn't like that for pydal imports. Paolo 2015-03-08 13:36 GMT+01:00 niphlod notifications@github.com:
|
that's because - fortunately - pydal doesn't manipulate sys.path. I think at this point it's more of a pydal issue than a web2py one. |
The pip version of portalocker (0.5.4 on windows) raises an issue on welcome app:
The text was updated successfully, but these errors were encountered: