airblade / voom

A simplest-thing-that-works Vim plugin manager. Use with Vim 8 or Pathogen.
MIT License
29 stars 9 forks source link

Added -q option to update to suppress commit logs #15

Closed hugoh closed 6 years ago

hugoh commented 6 years ago

voom update always shows the commit log. Added a -q option to hide it.

airblade commented 6 years ago

Thanks for this.

I pulled your changes and ran voom update. Most of my plugins updated by a handful of commits but vim-rails listed 1533 commits and vim-go 2065. I can't explain that but I don't think it's related to your change.

Anyway, I've had trouble in the past when updating plugins with the output from each getting interleaved, because each plugin updates in the background. It's hard to reproduce, but I wonder whether introducing two printf statements (when -q is not given) would open a window in which interleaving could occur. I think it would be safer to have a single printf on each side of the quietness conditional. Would you mind implementing that?

hugoh commented 6 years ago

Oh, I completely forgot about the concurrency of those updates. I'll fix it.

hugoh commented 6 years ago

@airblade - see the updated patch

airblade commented 6 years ago

Thanks again.