<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Alexandre,<br>
<br>
Some of your recent pom.xml changes might introduce a new issue.<br>
<br>
What changes are an issue?<br>
=================<br>
<br>
Creating this repo<br>
<a class="moz-txt-link-freetext" href="https://github.com/droolsjbpm/kie-commons">https://github.com/droolsjbpm/kie-commons</a><br>
(in conjunction with adding a SNAPSHOT dependency to it from several
projects)<br>
<br>
Why is that an issue?<br>
=============<br>
<ul>
<li>Every extra git repository introduces overhead:</li>
<ul>
<li>slows down the git-all update<br>
</li>
<li>slows down the mvn-all build</li>
<li>add a new jenkins job (which triggers more jenkins jobs)</li>
<li>Introduces complexity to understand the repo list here:
<a class="moz-txt-link-freetext" href="https://github.com/droolsjbpm">https://github.com/droolsjbpm</a><br>
</li>
</ul>
<li>Much of the orchestration of a new git repo is missing</li>
<ul>
<li>jenkins jobs (so SNAPSHOTs are build and deployed to nexus)</li>
<li>Part of release procedure.</li>
<ul>
<li>Currently, master can't be released because kie-commons
needs to be released first<br>
</li>
</ul>
<li>...</li>
</ul>
<li>The kie-commons pom.xml duplicates the droolsjbpm-parent pom</li>
<ul>
<li>DRY<br>
</li>
<li>Out of sync issues</li>
<ul>
<li>Different dependency/plugin versions</li>
<li>...</li>
</ul>
</ul>
</ul>
How can you fix it?<br>
============<br>
<br>
Proposal:<br>
<ul>
<li>Put module kie-commons in same repo as kie-api<br>
</li>
<ul>
<li><a class="moz-txt-link-freetext" href="https://github.com/droolsjbpm/droolsjbpm-knowledge">https://github.com/droolsjbpm/droolsjbpm-knowledge</a></li>
</ul>
<li>Change kie-common/pom.xml: remove all duplicate pom code and
make it like kie-api/pom.xml</li>
<li>Delete git repo kie-commons</li>
</ul>
===<br>
<br>
Can you take a look at fixing it?<br>
(Let me know if you can't fix it in a timely manner.)<br>
<br>
With kind regards,<br>
Geoffrey<br>
<br>
</body>
</html>