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

mid3v2: add verbose mode and command line options [with patch] #40

Closed
lazka opened this issue Jul 4, 2014 · 5 comments
Closed

mid3v2: add verbose mode and command line options [with patch] #40

lazka opened this issue Jul 4, 2014 · 5 comments
Labels

Comments

@lazka
Copy link
Member

@lazka lazka commented Jul 4, 2014

Originally reported by: Christoph Reiter (Bitbucket: lazka, GitHub: lazka)


From hanse...@gmail.com on November 17, 2009 19:16:11

When driven by a script, mid3v2 can be rather verbose in its output. I don't really care to know
everything it is doing, only if there is a problem.

The attached patch adds command line options -v (--verbose) and -q (--quiet) to control the
wordiness.

Cheers

Hans

Original issue: http://code.google.com/p/mutagen/issues/detail?id=40


@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From hanse...@gmail.com on November 17, 2009 11:01:01

Missed one place where a "print" had to be prefixed with "if verbose:".

I'm now running this script-driven on my music collection and the results are very
clean. As far as I can see this should be the last version of the patch.

Of course, you might have a different opinion on what the default value for
options.verbose should be....

Attachment: mid3v2.verbose.diff

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From hanse...@gmail.com on November 17, 2009 11:14:03

Here is a patch for the manpage mid3v2.1

Attachment: mid3v2.1.verbose.diff

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From joe.wreschnig@gmail.com on December 12, 2009 20:11:57

This issue was closed by revision r67 .

Status: Fixed
Mergedinto: -

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From joe.wreschnig@gmail.com on November 24, 2009 16:02:47

Status: Accepted

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From hanse...@gmail.com on December 28, 2009 09:55:25

Thank you! Sorry for the bad python. Never written it before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.