Thanks Rob, sounds very interesting indeed. But I need to look how to set it up. <div><br></div><div>so if I understand correctly that'd give us something like : </div><div>org.eclipse.m2e.wtp.jpa.feature </div><div> |___org.eclipse.m2e.wtp.jpa / contains activator / plugin.xml / m2e xml settings / common classes</div>
<div> |___org.eclipse.m2e.wtp.jpa.e42.feature (optional) </div><div> |___org.eclipse.m2e.wtp.jpa.e42 : pre-kepler compatible project configurator impl.</div><div> |___org.eclipse.m2e.wtp.jpa.e43.feature (optional) </div>
<div><div> |___org.eclipse.m2e.wtp.jpa.e43 : kepler compatible project configurator impl.</div></div><div><br></div><div>So that means if a juno user installs the JPA feature, only org.eclipse.m2e.wtp.jpa.e42.feature will be installed because the Dali version restriction range matches what's available in juno?</div>
<div><br></div><div>IF that's correct AND that scenario supports upgrades from juno to kepler, that's most probably the best way to go. Any one else can confirm?</div><div><br></div><div>Fred</div><div><br><div class="gmail_quote">
On Thu, Jan 10, 2013 at 6:32 PM, Rob Cernich <span dir="ltr"><<a href="mailto:rcernich@redhat.com" target="_blank">rcernich@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div style="font-size:12pt;font-family:times new roman,new york,times,serif">Have you thought about breaking out the jpa related code base so that you only need to branch those bits? You could then have e42 and e43 plugins, included by jpa e42 and e43 features that are declared as optional in the main feature. In that case, all the plugins and features have the same version and the correct child feature gets installed based on the wtp version. I'm not sure about the update process (I worry you'd have to uninstall the previous jpa feature prior to installing the new feature).<br>
<br><hr><blockquote style="padding-left:5px;font-size:12pt;font-style:normal;margin-left:5px;font-family:Helvetica,Arial,sans-serif;text-decoration:none;font-weight:normal;border-left:2px solid rgb(16,16,255)"><div><div class="h5">
Any input is welcome.<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Fred Bricon</b> <span dir="ltr"><<a href="mailto:fbricon@gmail.com" target="_blank">fbricon@gmail.com</a>></span><br>
Date: Thu, Jan 10, 2013 at 4:29 PM<br>Subject: Best way to handle 2 code streams<br>To: M2E-WTP Developers mailing list <<a href="mailto:m2e-wtp-dev@eclipse.org" target="_blank">m2e-wtp-dev@eclipse.org</a>><br><br>
<br>Hi,<div><br>
</div>
<div>first of all, let me wish you all a very Happy New Year. I hope it'll see m2e-wtp graduate from the eclipse incubator next June ;-)</div><div><br></div><div>During last call, we decided m2e-wtp would provide a JPA configurator for both the Kepler and pre-Kepler streams, due to Dali moving its provisional API, meaning we now need to maintain two branches of code. So I've been trying to think at the best strategy that would lead to the minimum maintenance overhead while allowing to provide new cool features for everybody.</div>
<div><br></div><div>1/ have 2 different JPA features in 1 git repo :</div><div>Somethink like org.eclipse.m2e.wtp.jpa.feature (for kepler and onward) and org.eclipse.m2e.wtp.jpa.e42.feature (pre-kepler). Both features would be built and appear in the same update site (version 0.17.0). This gives us all the code easily accessible for maintenance (from my point of view) as we wouldn't have to switch between branches in order to apply fixes in both JPA configurators. </div>
<div>The biggest problem though, is we wouldn't be able to provide a way to upgrade from jpa.e42 to jpa. There's already a p2.inf hack allowing the upgrade from the jboss tools jpa feature to the eclipse one, that hack prevents us from adding the jpa (juno) to jpa (kepler) upgrade path (Tycho breaks seeing the same IU being built twice).</div>
<div><br></div><div>2/ Have 2 m2e-wtp versions in 2 branches of a git repo</div><div>We'd provide 2 separate update sites: </div><div>* a "pre-kepler" p2 repo with m2e-wtp 0.17.0, built from a git 0.17.0 branch</div>
<div>* a kepler repo with m2e-wtp 0.18.0/1.0, built from master.</div><div><br></div><div>This would be easier to build, from a tycho perspective, and we'd still be able to upgrade from juno to kepler AND from jboss tools. But it would bring a *huge* overhead in non-JPA maintenance.</div>
<div><br></div><div>3/ Have JPA in a dedicated git repo, use specific qualifiers :</div><div><div>We'd provide 2 separate update sites: </div><div>* a "pre-kepler" aggregated p2 repo with m2e-wtp 0.17.0, built from master and a jpa.0.17.0-e42.timestamp site, built from the e42 branch of the m2e-wtp-jpa git repo, </div>
<div>* a kepler aggegated p2 repo with m2e-wtp 0.17.0, built from master and a jpa.0.17.0-e43.timestamp site, built from the master branch of the m2e-wtp-jpa git repo, </div></div><div><br></div><div>It requires 2 tycho builds and more manual hacking of the aggregated sites, but would be easier to maintain, from a code perspective. And we'd still be able to upgrade from 0.17.0-e42 to 0.17.0-e43.</div>
<div><br></div><div>I think I like #3 better (but I haven't tested the approach works yet). What do you think?</div><span><font color="#888888"><div><br></div><div>Fred Bricon</div><div><br></div><div>
-- </div><div>"Have you tried turning it off and on again" - The IT Crowd
</div>
</font></span></div><br><br clear="all"><div><br></div>-- <br>"Have you tried turning it off and on again" - The IT Crowd
<br></div></div>_______________________________________________<br>jbosstools-dev mailing list<br><a href="mailto:jbosstools-dev@lists.jboss.org" target="_blank">jbosstools-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/jbosstools-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/jbosstools-dev</a></blockquote>
<br></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>"Have you tried turning it off and on again" - The IT Crowd
</div>