Skip to content

Comments

Fix: Prompt change in newer versions of git; README file #5

Open
mrflip wants to merge 4 commits intosamuelkadolph:masterfrom
mrflip:master
Open

Fix: Prompt change in newer versions of git; README file #5
mrflip wants to merge 4 commits intosamuelkadolph:masterfrom
mrflip:master

Conversation

@mrflip
Copy link

@mrflip mrflip commented Apr 10, 2012

Recent versions of git cause the program to emit Password for 'https://mrflip@awesome-git-repo.com': ' not a simple Password:.

I made it only compare the leading part of the prompt, to re-use the same prompt it was given, and to give slightly more descriptive error messages.

Also added a README.md file (cribbed from your blog post) and a .gitignored ./build directory (to allow project to build in a relative directory.

Thanks for this -- really helpful!

Philip (flip) Kromer added 4 commits April 10, 2012 15:27
So we check only that it looks like a Username or Password prompt, and recycle that for re-prompting the user.
Also made the various error messages slightly more descriptive
@crishoj
Copy link

crishoj commented Aug 15, 2012

Please pull this one – the Homebrew guys don't like formula which point to forks

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

Successfully merging this pull request may close these issues.

2 participants