-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Install git: Setup.py Parsing #1761
Labels
kind/bug
Something isn't working as expected
Comments
1 task
In my opinion Fix is on the way: #1764 Thanks a lot for reporting! |
shenek
pushed a commit
to shenek/poetry
that referenced
this issue
Dec 31, 2019
…ython-poetry#1761) (python-poetry#1764) * fix (setup_reader): in `setup.py` the `setup()` method can also be called by `setuptools.setup()` * fix (setup_reader): make black happy * fix (setup_reader): skip test for python <3.4
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hello,
I tried to install a git dependency since the master branch include needed fixes for the development.
Then I received the following error:
[RuntimeError] Unable to retrieve the package version for /tmp/pypoetry-git-self-attentive-parserXXXXX
Based on this error I found the following issue describing a similar problem: #673
I tried to add setuptools as
dev dependency
and installed it as suggested.This doesn't helped me and I still recieved the same error message.
Then I took a deeper look into the
setup.py
of this project (https://github.com/nikitakit/self-attentive-parser/blob/master/setup.py#L23)There I have seen that
setup(
was called assetuptools.setup(
instead of the most common way to import the setup command directly.After changing this to
setup
on my fork everything worked fine.Issue
Now my question:
Is this an issue with
ast
parsing and can this be solved in poetry itself or should I ask the project owner to change itssetup.py
?The text was updated successfully, but these errors were encountered: