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

Strange output on Windows #24

Closed
itavero opened this issue Apr 13, 2015 · 16 comments
Closed

Strange output on Windows #24

itavero opened this issue Apr 13, 2015 · 16 comments

Comments

@itavero
Copy link

itavero commented Apr 13, 2015

I'm getting the following output on Windows (both in Command Prompt, as well as in Git Bash). Looks like I'm missing something?

F←[0me←[0mt←[0mc←[0mh←[0mi←[0mn←[0mg←[0m ←[0mo←[0mr←[0mi←[0mg←[0mi←[0mn←[0m
←[0m←[0m←[1mdevelop     ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mfeature/528 ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mfeature/551 ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mfeature/575 ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mhotfix/545  ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mhotfix/580  ←[0m←[0m ←[0m←[32mup to date←[0m←[0m
←[0mThe system cannot find the path specified.
←[0m
@msiemens
Copy link
Owner

Strange... Which version of Windows are you using? And which version of PyGitUp?

@msiemens msiemens added the bug label Apr 13, 2015
@itavero
Copy link
Author

itavero commented Apr 14, 2015

I'm using Windows 7 (x64, Service Pack 1), with Python 2.7.8. GitUp is at version v1.3.0.

@pampersrocker
Copy link

Same Problem

  • Windows 8.1
  • Cygwin 64
  • Oh-My-Zsh
  • Python 3.4.3
  • PyGitUp 1.3.0

@msiemens
Copy link
Owner

I can now reproduce this behaviour. Will investigate...

@msiemens
Copy link
Owner

Looks like it's a regression of some kind in colorama introduced in tartley/colorama@961428e. I'm not sure what the right fix is here (besides downgrading the dependency). Could you both run echo %TERM% (in bash it's echo $TERM of course) in your consoles and report the result here?

@pampersrocker
Copy link

echo $TERM returns cygwin. On the default windows command line TERM is not set:
image
image

(Also wrong output on windows command line:)

image

@msiemens
Copy link
Owner

For now we'll have to live with a workaround. Could you try the latest dev version (pip install git+https://github.com/msiemens/PyGitUp.git) and report whether it actually fixes the bug?

@Predelnik
Copy link

It seems like it's still the same for me with dev version, also I suggest to add an option to disable coloring entirely since it would be useful not only in that case but to help users annoyed by colors 😄

@msiemens msiemens reopened this Apr 17, 2015
@msiemens
Copy link
Owner

Okay, this is getting really interesting. For reference, the latest dev version fixed this on my computer.

@Predelnik Could you uninstall PyGitUp and then reinstall the dev version to make sure?

@Predelnik
Copy link

Yeah reinstalling seems to fix this issue, I'm not very active user of pip so didn't know that it may be required sorry.

Though there's some leftover ←[0m at the bottom I don't know if it's intended
image

@msiemens
Copy link
Owner

Yeah, I noticed the ←[0m too. Will investigate this soon.

@msiemens
Copy link
Owner

I've filed an issue in colorama (which is responsible for converting ANSI codes to Win32 calls): tartley/colorama#49

@pampersrocker
Copy link

(Just for info you need to add --upgrade to the pip call to update the existing version. Instead of manually uninstalling)

Yes I can confirm the the latest dev version fixes the colors, just as @Predelnik said the ←[0m is left.

@msiemens
Copy link
Owner

Is that still an issue with PyGitUp 1.4.0?

@hugovk
Copy link
Contributor

hugovk commented Feb 29, 2016

Looks fixed for me: cmd.exe and MINGW32 are much improved and have switched the [0m← stuff for colours.

But Git Bash (MINGW64) has lost its colours. I've made a new issue for this different problem: #48.

@msiemens
Copy link
Owner

Okay, we'll track that on the issue you've opened.

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

No branches or pull requests

5 participants