Fix precommit

This commit is contained in:
Ryan Ernst 2016-03-08 22:55:24 -08:00
parent cb607a8fae
commit 1dafead2eb
3 changed files with 22 additions and 0 deletions

View File

@ -29,6 +29,7 @@ import org.elasticsearch.cli.ExitCodes;
import org.elasticsearch.cli.UserError; import org.elasticsearch.cli.UserError;
import org.elasticsearch.common.Strings; import org.elasticsearch.common.Strings;
import org.elasticsearch.cli.Terminal; import org.elasticsearch.cli.Terminal;
import org.elasticsearch.common.SuppressForbidden;
import org.elasticsearch.monitor.jvm.JvmInfo; import org.elasticsearch.monitor.jvm.JvmInfo;
final class BootstrapCliParser extends Command { final class BootstrapCliParser extends Command {
@ -54,6 +55,8 @@ final class BootstrapCliParser extends Command {
.withRequiredArg(); .withRequiredArg();
} }
// TODO: don't use system properties as a way to do this, its horrible...
@SuppressForbidden(reason = "Sets system properties passed as CLI parameters")
@Override @Override
protected void execute(Terminal terminal, OptionSet options) throws Exception { protected void execute(Terminal terminal, OptionSet options) throws Exception {
if (options.has(versionOption)) { if (options.has(versionOption)) {

View File

@ -38,6 +38,15 @@ thirdPartyAudit.excludes = [
// for example we do not need ivy, scripts arent allowed to download code // for example we do not need ivy, scripts arent allowed to download code
'com.thoughtworks.xstream.XStream', 'com.thoughtworks.xstream.XStream',
'groovyjarjarasm.asm.util.Textifiable', 'groovyjarjarasm.asm.util.Textifiable',
'org.apache.commons.cli.CommandLine',
'org.apache.commons.cli.CommandLineParser',
'org.apache.commons.cli.GnuParser',
'org.apache.commons.cli.HelpFormatter',
'org.apache.commons.cli.Option',
'org.apache.commons.cli.OptionBuilder',
'org.apache.commons.cli.Options',
'org.apache.commons.cli.Parser',
'org.apache.commons.cli.PosixParser',
'org.apache.ivy.Ivy', 'org.apache.ivy.Ivy',
'org.apache.ivy.core.event.IvyListener', 'org.apache.ivy.core.event.IvyListener',
'org.apache.ivy.core.event.download.PrepareDownloadEvent', 'org.apache.ivy.core.event.download.PrepareDownloadEvent',

View File

@ -191,6 +191,16 @@ thirdPartyAudit.excludes = [
'org.apache.commons.beanutils.DynaClass', 'org.apache.commons.beanutils.DynaClass',
'org.apache.commons.beanutils.DynaProperty', 'org.apache.commons.beanutils.DynaProperty',
'org.apache.commons.beanutils.PropertyUtils', 'org.apache.commons.beanutils.PropertyUtils',
'org.apache.commons.cli.CommandLine',
'org.apache.commons.cli.CommandLineParser',
'org.apache.commons.cli.GnuParser',
'org.apache.commons.cli.HelpFormatter',
'org.apache.commons.cli.Option',
'org.apache.commons.cli.OptionBuilder',
'org.apache.commons.cli.OptionGroup',
'org.apache.commons.cli.Options',
'org.apache.commons.cli.ParseException',
'org.apache.commons.cli.PosixParser',
'org.apache.commons.compress.archivers.tar.TarArchiveEntry', 'org.apache.commons.compress.archivers.tar.TarArchiveEntry',
'org.apache.commons.compress.archivers.tar.TarArchiveInputStream', 'org.apache.commons.compress.archivers.tar.TarArchiveInputStream',
'org.apache.commons.codec.DecoderException', 'org.apache.commons.codec.DecoderException',