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
Build console progress bar doesn't show when using 'gitbash on Windows' terminals #416
Comments
The fix for visionmedia/node-progress#170 has been merged, but the bug still happens even on After some investigation, it seems like this bug is actually related to visionmedia/node-progress#110. The PR referenced in the issue visionmedia/node-progress#168 fixes this bug with the |
Hello, I tried running
Not sure if the issue has already been resolved. If the issue is related to the bug referenced in the above comment, then a quick patch is possible, as suggested here. If not perhaps we can safely close this issue:) |
Thanks for investigating this! @tlylt Good to see it works on a whole bunch of terminals. I'm still able to produce this via gitbash running on mintty 3.5, although strangely wsltty works fine. In any case, I think this issue is a little out of our control. Rather maybe we could insert a one / 2 liner into the |
Hi @ang-zeyu, hmm I am not familiar with terminals or console stuff, but I could try to apply the patch as suggested by others facing similar issues and send up a PR for you to verify if that works for "gitbash on mintty 3.5". Do you think that will be ok? Alternatively, I can add a note in the docs to warn users (perhaps as a footnote in the getting started section). Please let me know which is preferred :) |
👀 if its fixable, let's try that!
it should come packaged in the latest version from here: |
Hi @ang-zeyu , could you help verify if it works with #1654 pls? At the moment I have added node-progress with the relevant patch into I am unable to reproduce the result even though I think I am using gitbash coming from git. I tried git-bash and git-cmd and both are fine with or without the fix: |
@tlylt no dice, still no progress bar on your PR branch: |
Might be mistaken but the terminal in your screenshot dosen't seem like the default mintty that comes with the git installation as well (blue bar on top). Might be what's causing a false positive 🤔 |
Hi @ang-zeyu, thank you for your kind help in verifying the PR. I finally made some progress on this :) For the false positive, it might be due to the fact that I did not choose the option with mintty when installing git (see screenshot below): I did that because some tutorials that I have seen recommended the second option for windows 10, as the introduction of windows terminal and wsl probably makes working with cli easier nowadays. I have tested on my laptop and I was getting the same result as you, probably because on my laptop I went with the default git installation. On further inspection of the patch code, I found out that I misunderstood the fix PR raised over at node-progress and I did not invoke the code properly (Big warning for me to be caution with code that I don't understand fully 😂 ). I have updated the patch and verified that it works on my laptop. I shall continue the conversation over on my patch PR and make it ready to merge soon:) |
Tell us about your environment
What did you do? Please include the actual source code causing the issue.
When building pages, a progress bar is shown. However, when using gitbash on Windows terminals (or any mintty terminals), the progress bar does not show.
Issue is related to visionmedia/node-progress#170.
What did you expect to happen?
What actually happened? Please include the actual, raw output.
The text was updated successfully, but these errors were encountered: