Skip to content
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

git-prompt blocks the use of virtualenv's prompt decoration #2

Open
offbyone opened this issue Dec 7, 2010 · 5 comments
Open

git-prompt blocks the use of virtualenv's prompt decoration #2

offbyone opened this issue Dec 7, 2010 · 5 comments

Comments

@offbyone
Copy link

offbyone commented Dec 7, 2010

Ian Bicking's virtualenv project amends the prompt when you select a virtualenv. git-prompt erases or blocks this content:

old::

me@myhost $ . venv/bin/activate
(venv) me@myhost $ deactivate
me@myhost $

git-prompt::

(D) [email protected] ~> . venv/bin/activate
(D) [email protected] ~> deactivate
(D) [email protected] ~> 
@niklas
Copy link
Contributor

niklas commented Jan 3, 2011

What would be the expected behaviour?

PS: If this disturbs you in any way, you can disable virtualenv's prompt by setting VIRTUAL_ENV_DISABLE_PROMPT. (http://virtualenv.openplans.org/#activate-script)

@offbyone
Copy link
Author

offbyone commented Jan 4, 2011

I would expect the virtualenv to prefix the prompt; probably something like this:

(D) [email protected] ~> . venv/bin/activate
(venv) (D) [email protected] ~> deactivate
(D) [email protected] ~> 

@niklas
Copy link
Contributor

niklas commented Jan 4, 2011

@lvv I propose to only set PS1 when it has not been changed manually (check in prompt_command_function()). unset'ting it enables git-prompt again.

What do you think?

@lvv
Copy link
Owner

lvv commented May 26, 2011

Coreranger's fork have commits to support virtualenv. I am about to merge it. But I can't test it. Can someone try it if it works?

@lvv lvv closed this as completed May 26, 2011
@lvv lvv reopened this May 26, 2011
@navilan
Copy link

navilan commented Sep 14, 2011

I can confirm that it works as expected (quite nice).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants