Roy Golan has posted comments on this change. Change subject: core: Infra for injecting beans in external JARs ......................................................................
Patch Set 2: (3 comments) http://gerrit.ovirt.org/#/c/37439/2/backend/manager/dependencies/common/pom.xml File backend/manager/dependencies/common/pom.xml: Line 379: <version>${jackson.version}</version> Line 380: </dependency> Line 381: Line 382: <dependency> Line 383: <groupId>org.apache.deltaspike.core</groupId> > Of course. Already run a developer job with it sorry confused. I meant packaging wise, not build time. I had the exact same issue with antlr4. a dev job will pass but packaging will fail. we can continue that offline, its a d/s issue Line 384: <artifactId>deltaspike-core-api</artifactId> Line 385: <version>${deltaspike.version}</version> Line 386: </dependency> Line 387: http://gerrit.ovirt.org/#/c/37439/2/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/ModuleConfigurationExtension.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/ModuleConfigurationExtension.java: Line 21: import org.reflections.Reflections; Line 22: import org.slf4j.Logger; Line 23: import org.slf4j.LoggerFactory; Line 24: Line 25: import com.google.common.base.Strings; > We already have guava (my patch didn't add it) - so why not use it? thats new to me. didn't realize somebody added that Line 26: Line 27: public class ModuleConfigurationExtension implements Extension { Line 28: private static final Logger log = LoggerFactory.getLogger(ModuleConfigurationExtension.class); Line 29: private final Map<String, AnnotatedType<Object>> beans = new HashMap<>(); Line 32: * This method is automatically activated by CDI, and loads all classes in the org.ovirt package that has NAMED or Line 33: * SINGLETON annotations. Line 34: * @param bdd Line 35: */ Line 36: void readAllConfigurations(final @Observes BeforeBeanDiscovery bdd, BeanManager bm) { > It's not there, so no problem. great Line 37: log.info("Starting to load beans from modules"); Line 38: addBeansFromPackage(bdd, bm, "org.ovirt.engine.core.dal"); Line 39: addBeansFromPackage(bdd, bm, "org.ovirt.engine.core.dao"); Line 40: } -- To view, visit http://gerrit.ovirt.org/37439 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b3c2c57178b5df4f98b4191ac9a06f9129414d0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
