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

Commands like Get, Init, Update can't retrieve live out/err? #39

Open
tony opened this issue May 4, 2016 · 1 comment
Open

Commands like Get, Init, Update can't retrieve live out/err? #39

tony opened this issue May 4, 2016 · 1 comment

Comments

@tony
Copy link
Contributor

tony commented May 4, 2016

In git, initial checkout and pulls, these are often "buffered". That information is valuable in my case (www.github.com/tony/vcsync) because I want to forward the progress of the update to stdout. Ideally, a couple of repos at a time.

At the present time, Get, Init and Update is hard-coded to use CombinedOutput, which runs the command and waits for completion. This lends me no ability run the command and observe the live output of a git pull, and due to the command and error handling for the (Get/Init/Update) on the repo being tightly-coupled, I can't trivially compose a command for my case without duplicating 90% of the code.

I'm going to present a PR for this for your opinion.

tony added a commit to tony/vcs that referenced this issue May 4, 2016
tony added a commit to tony/vcs that referenced this issue May 4, 2016
@mattfarina
Copy link
Member

@tony I look forward to your PR.

tony added a commit to tony/vcs that referenced this issue May 5, 2016
tony added a commit to tony/vcs that referenced this issue May 5, 2016
@tony tony mentioned this issue May 5, 2016
tony added a commit to tony/vcs that referenced this issue May 6, 2016
tony added a commit to tony/vcs that referenced this issue May 6, 2016
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

2 participants