Alon Bar-Lev has posted comments on this change.

Change subject: packaging: build: move to ant drop maven and jboss modules
......................................................................


Patch Set 4:

> whats the benefit of ant over maven here / how this compares to say, engine 
> build?

few of reasons...

pros

1. no pom.xml hell! we are just discussing that for engine.

2. there is no maven support for most distributions, well, all except 
>=fedora-19, so this provide single method for rhel, centos, fedora, gentoo and 
others.

3. maven fetches dependencies from external resources which is violation of 
release engineering best-practices, it cannot be forced [easily] to use local 
installed jar.

4. for mead we need to hack the spec files and have a fork of build process 
which cause a lot of overhead.

5. for reports we must import almost all jasper into our local maven repository 
instead of simply use the BuildRequires over jasper and use its artifacts.

6. much simpler to implement, and maintained custom functionality, such as 
codegen and ordering, as it is procedural flow.

7. total control over output, jar contents, naming etc...

cons

1. developers must have dependencies installed on their system.

2. runtime dependencies classpath is not built easily in red hat distributions, 
as the build-classpath script provides only leaf, manual work should be done to 
acquire the dependency tree. in other distributions such as Gentoo building 
classpath including dependencies is supported.

3. I suspect it has less of "ide integration", am not sure as I do not use ide.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I39550f694d7e8c37bbe189c1b9bfd944a5dad2e0
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to