Allon Mureinik has posted comments on this change.
Change subject: packaging: add find security bugs to pom.xml
......................................................................
Patch Set 1: Code-Review-1
(3 comments)
....................................................
File pom.xml
Line 580: </plugin>
Line 581: <!-- FindBugs Static Analysis -->
Line 582: <plugin>
Line 583: <groupId>org.codehaus.mojo</groupId>
Line 584: <artifactId>findbugs-maven-plugin</artifactId>
Indeed. This is redundant.
Line 585: <version>2.5.2</version>
Line 586: <configuration>
Line 587: <effort>Max</effort>
Line 588: <threshold>Low</threshold>
Line 586: <configuration>
Line 587: <effort>Max</effort>
Line 588: <threshold>Low</threshold>
Line 589: <failOnError>true</failOnError>
Line 590:
<includeFilterFile>${session.executionRootDirectory}/findbugs-security-include.xml</includeFilterFile>
I can't see any reason to separate the exclusion filters - see my comment below.
Line 591:
<excludeFilterFile>${session.executionRootDirectory}/findbugs-security-exclude.xml</excludeFilterFile>
Line 592: <plugins>
Line 593: <plugin>
Line 594: <groupId>com.h3xstream.findsecbugs</groupId>
Line 593: <plugin>
Line 594: <groupId>com.h3xstream.findsecbugs</groupId>
Line 595: <artifactId>plugin</artifactId>
Line 596: <version>1.1.0</version>
Line 597: </plugin>
Frankly this is the only thing you should add - to he EXISTING plugin.
Line 598: </plugins>
Line 599: </configuration>
Line 600: </plugin>
Line 601: </plugins>
--
To view, visit http://gerrit.ovirt.org/21828
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6af60c1a080fcfc40a9b4ac82dbf4e857eca61ac
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
