Skip to content

Renaming#1

Open
seyhak wants to merge 2 commits into
masterfrom
renaming
Open

Renaming#1
seyhak wants to merge 2 commits into
masterfrom
renaming

Conversation

@seyhak

@seyhak seyhak commented Jul 9, 2020

Copy link
Copy Markdown

No description provided.

@spinus

spinus commented Jul 10, 2020

Copy link
Copy Markdown
Member

@seyhak can you paste issue you have with steps to reproduce when installing.
@dougbyrne strange one, he has missing wheel package error on install (this error does not block installation, but breaks at the end, package can be imported). But this package is not wheel package so the message is kinda crap.

Solution Seyhak found is just to instlal wheel, do we want to add this to reqs or just on the machines or try to look deeper?

@seyhak

seyhak commented Jul 13, 2020

Copy link
Copy Markdown
Author
Collecting python-twitter from https://github.com/shareablee/python-twitter/archive/renaming.zip#egg=python-twitter (from -r requirements.txt (line 92))
  Using cached https://github.com/shareablee/python-twitter/archive/renaming.zip
    Complete output from command python setup.py egg_info:
    WARNING: The wheel package is not available.
    ERROR: 'pip wheel' requires the 'wheel' package. To fix this, run: pip install wheel

@spinus

spinus commented Jul 16, 2020

Copy link
Copy Markdown
Member

Can you try to install that without egg_name? I think eggname is only needed if you use git, not zip.
I have tried to install that in local virtualenvs and works for me without any issue (and wheel).
What are steps to reproduce?

@seyhak

seyhak commented Jul 16, 2020

Copy link
Copy Markdown
Author

Can you try to install that without egg_name? I think eggname is only needed if you use git, not zip.
I have tried to install that in local virtualenvs and works for me without any issue (and wheel).
What are steps to reproduce?

Open any staging with additional requirement
https://github.com/shareablee/python-twitter/archive/renaming.zip#egg=python-twitter

@seyhak

seyhak commented Jul 16, 2020

Copy link
Copy Markdown
Author

Still the same issue, there is another .zip package in requirements.txt. It has #egg part

@shareablee shareablee deleted a comment from seyhak Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants