----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52429/ -----------------------------------------------------------
(Updated Oct. 26, 2016, 9:23 a.m.) Review request for oozie and Robert Kanter. Changes ------- `bin/*` is excluded from RAT analysis. Repository: oozie-git Description ------- We go for that solution: 1. create a new script as `bin/test-patch-11-findbugs-diff` so that it comes right after compile 2. `mvn clean verify -DskipTests` before and after applying the patch, and saving each and every module's `target/findbugs/findbugsXml.xml` to a secure folder 3. call the `findbugs-diff` app for all the before / after FindBugs xml pairs 4. analyze all the `findbugs-new.xml` entries for nodes with XPath `/BugCollection/BugInstance` entries. Stop on any BugInstance w/ `priority` and / or `rank` lower than threshold 5. have the reports merged and be downloadable in any case in following scenarios: * -1 for bugs that are too serious to apply the patch with them * 0 for other new bugs * +1 if there is no new bug Diffs (updated) ----- bin/findbugs-diff-0.1.0-all.jar.md5 PRE-CREATION bin/test-patch-11-findbugs-diff 0376ab0788644633982bfb099e740566b3b7118e bin/test-patch-11-findbugs-diff-0.1.0-all.jar.md5 28179b6921d1b3ead4f87fca581b03bd62c0f013 pom.xml ade1f250106daff83022dc9954cadea5addcacfd Diff: https://reviews.apache.org/r/52429/diff/ Testing ------- Thanks, András Piros
