Shahar Havivi has posted comments on this change.

Change subject: findbugs: adding category ignore list
......................................................................


Patch Set 1:

Yes,
There is two problems:
1. not all places that we have pom file we have the ignore project, so findbugs 
will try to run and will not find the excludeFileFilter.xml and raise and error.

2. the findbugs-general profile refer to findbugs-general.xml in a relative 
position to the project, not all projects in the same directory tree level so 
its change from project to project.

If you have other solution I will be happy to try and implement.

-- 
To view, visit http://gerrit.ovirt.org/15775
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idfbc37df07eff4c212cc5210965cabd79a51df36
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to