-
-
Notifications
You must be signed in to change notification settings - Fork 612
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
No handlers could be found for logger "pip.vcs.git" #292
Comments
FWIW, this issue came up for me about the same time that I added a |
Has this been since in recent releases? Note to contributors: Try to reproduce by causing a failure with a git repo on |
I'm still seeing this message each time I run my upgrade script. |
@ryanhiebert Could you give me your upgrade script, as well as a |
I may have been mis-remembering what I was seeing. I can't confirm the bug myself ATM. |
Alright, closing this for now. If anyone comes up with this again, we'll reopen. |
Hi, I add the issue myself running pip-compile with git urls indeed. Also, I get |
Did anyone figure this out? @cancan101 ?? |
Hello @koleror,
I cannot reproduce it locally. Is this issue still on, @koleror? echo "-e git+ssh://[email protected]/jazzband/pip-tools.git@master#egg=pip-tools" | pip-compile - -vo- > /dev/null
Using indexes:
https://pypi.org/simple
ROUND 1
Current constraints:
pip-tools from git+ssh://****@github.com/jazzband/pip-tools.git@master#egg=pip-tools (from -r /var/folders/4x/rz_w89dx1z3bydfd6qpdjn_h0000gn/T/tmpflyl5zxs (line 1))
Finding the best candidates:
found candidate -e git+ssh://[email protected]/jazzband/pip-tools.git@master#egg=pip-tools (constraint was <any>)
Finding secondary dependencies:
New dependencies found in this round:
adding ['click', '>=6', '[]']
adding ['six', '', '[]']
Removed dependencies in this round:
------------------------------------------------------------
Result of round 1: not stable
ROUND 2
Current constraints:
click>=6 (from pip-tools->-r /var/folders/4x/rz_w89dx1z3bydfd6qpdjn_h0000gn/T/tmpflyl5zxs (line 1))
pip-tools from git+ssh://****@github.com/jazzband/pip-tools.git@master#egg=pip-tools (from -r /var/folders/4x/rz_w89dx1z3bydfd6qpdjn_h0000gn/T/tmpflyl5zxs (line 1))
six (from pip-tools->-r /var/folders/4x/rz_w89dx1z3bydfd6qpdjn_h0000gn/T/tmpflyl5zxs (line 1))
Finding the best candidates:
found candidate click==7.0 (constraint was >=6)
found candidate -e git+ssh://[email protected]/jazzband/pip-tools.git@master#egg=pip-tools (constraint was <any>)
found candidate six==1.12.0 (constraint was <any>)
Finding secondary dependencies:
click==7.0 requires -
six==1.12.0 requires -
------------------------------------------------------------
Result of round 2: stable, done
#
# This file is autogenerated by pip-compile
# To update, run:
#
# pip-compile --output-file=- -
#
-e git+ssh://[email protected]/jazzband/pip-tools.git@master#egg=pip-tools
click==7.0
six==1.12.0 |
I am still seeing this with ( |
Actually I am seeing a slightly different error:
And also this:
|
@antonagestam could you paste |
I've got the
Like the one from https:/palfrey/parochial currently, but with the Twisted version lockdown removed. Incidentally, it works with |
I think I've finally figured this one out. It should be pretty straightforward to fix, and help give a bit more details from pip logger when we hit some odd failure cases. I'll open up a MR soon. |
Fixes jazzband#292 In a normal use case, `pip` would take care of configuring the logging module as intended. But due to our intrusive import of pip internals, that part wasn't properly done.
@vphilippon cool! Thanks for the fix! @antonagestam @palfrey could you please test the #1035 whether it fixes your issue? Coud be installed via: $ pip install -e 'git+https:/vphilippon/pip-tools.git@configure-builtin-logging-module#egg=pip-tools' |
So it's still not fully working, but we've got a more interesting error message :)
Note that doing |
@palfrey Alright! My fix was just to fix the logger part, and get a better message, not fix the actual underlying issue it was failing to report :). So this is a good sign. For this case, with the output, it seems there's something odd with the A good-ish title to identify it: |
So, it turns out it's got nothing to do with Twisted at all, but setuptools. My virtualenv had setuptools 18.2 installed, which exhibited this bug, but upgrading to latest (44.0.0) fixed the problem. |
Looks like logging is not totally set up. I get this when
pip-compile
wit VCS links:The text was updated successfully, but these errors were encountered: