Oved Ourfali has submitted this change and it was merged.

Change subject: core: Make dependency on log4j explicit
......................................................................


core: Make dependency on log4j explicit

There are several places in the code where we are importing
classes from log4j, but we don't have an explicit dependency on
log4j in the main POM file or in the specific POM files for the
affected components. Right now this is working because Maven is
bringing that transitive dependency (incorrectly, I guess). In
Maven 3 this doesn't work.

This patch adds explict dependecies where needed.

Change-Id: Id2e85a154bf5eb94a6710a17933d6160b12daec4
---
M backend/manager/modules/utils/pom.xml
M backend/manager/tools/engine-config/pom.xml
M backend/manager/tools/engine-tools-common/pom.xml
M pom.xml
4 files changed, 24 insertions(+), 0 deletions(-)

Approvals:
  Oved Ourfali: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id2e85a154bf5eb94a6710a17933d6160b12daec4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to