-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
MNG-5486 hiding transfer logs #28
Conversation
+1 |
I like the idea of quiet transfert WDYT? |
@@ -137,6 +139,7 @@ public CLIManager() | |||
options.addOption( OptionBuilder.withLongOpt( "encrypt-master-password" ).hasOptionalArg().withDescription( "Encrypt master security password" ).create( ENCRYPT_MASTER_PASSWORD ) ); | |||
options.addOption( OptionBuilder.withLongOpt( "encrypt-password" ).hasOptionalArg().withDescription( "Encrypt server password" ).create( ENCRYPT_PASSWORD ) ); | |||
options.addOption( OptionBuilder.withLongOpt( "threads" ).hasArg().withDescription( "Thread count, for instance 2.0C where C is core multiplied" ).create( THREADS ) ); | |||
options.addOption( OptionBuilder.withLongOpt( "quiet_transfer_logs" ).withDescription( "quiets transfer logs, for example: downloading, uploading, bytes downloaded, etc.." ).create( QUIET_TRANSFER_LOGS) ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"quiet-transfer-logs", for consistency with other options, please
Why do we need this? There is a batch mode, minimal transfer output. |
@michael-o I think even |
@martin-g does the quiet option provide the behaviour you'd like? |
@jvanzyl Yes. I don't need the new option. |
quiet option is too quiet, I originally wanted just to hide the transfer logs, sometime it can be annoying in CI systems |
@JigarJoshi , you can use batch mode for that. |
Adds a sentence that explains what happens to plugin declarations which load extensions.
…n-docs Resolve apache#28: Improve Javadoc for flatten goal
The latest patch from https://jira.codehaus.org/browse/MNG-5486 as a Pull Request as requested by Jason van Zyl.