It can go either way. Most of the time, we expect plugin providers to update their CAR builds to add in the new server releases (see geronimo/plugins/openjpa2/). Ideally, the Samples would be released with every server release, but we have never done that in the past, but it is something we need to start doing.
If you add artifact-alias entries so the server can support older Samples, then I'm afraid this will become a maintenance problem, as you may also need to add in mappings to handle upgraded levels of Jetty, Tomcat, OpenEJB, OpenJPA, .... Then the question becomes, how many intra-release levels do you support? Only 2.2 plugins working on 2.2.1 or any 2.2.x plugins working on all future levels of 2.2.x forever. I'm really against this scenario, as it greatly complicates creating new maintenance releases, as you have to go in and find/add new alias entries after every release. -Donald On 6/9/10 11:42 PM, Ivan wrote: > I am not sure whether we really need to keep the old plugins work in the > new Geronimo release. Do we have corresponding samples for the new release ? > e.g. between different releases, there maybe GBean signatures changes, > so the serialized data will not work correctly. > > 2010/6/10 <xiam...@apache.org <mailto:xiam...@apache.org>> > > Author: xiaming > Date: Thu Jun 10 03:22:19 2010 > New Revision: 953208 > > URL: http://svn.apache.org/viewvc?rev=953208&view=rev > <http://svn.apache.org/viewvc?rev=953208&view=rev> > Log: > GERONIMO-5373 Make 2.2 samples plugins work on G 2.2.1, though only > part of samples plugins works after this change. > > Modified: > geronimo/server/branches/2.2/framework/configs/client-system/pom.xml > geronimo/server/branches/2.2/framework/configs/pom.xml > geronimo/server/branches/2.2/plugins/client/client-security/pom.xml > geronimo/server/branches/2.2/plugins/client/client/pom.xml > geronimo/server/branches/2.2/plugins/client/pom.xml > > geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml > geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml > geronimo/server/branches/2.2/plugins/pom.xml > > Modified: > geronimo/server/branches/2.2/framework/configs/client-system/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/client-system/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/client-system/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- > geronimo/server/branches/2.2/framework/configs/client-system/pom.xml > (original) > +++ > geronimo/server/branches/2.2/framework/configs/client-system/pom.xml > Thu Jun 10 03:22:19 2010 > @@ -207,6 +207,7 @@ > <config-xml-content load="false" /> > <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-system//car">org.apache.geronimo.framework/client-system/${version}/car</artifact-alias> > <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-system/${version}/car">org.apache.geronimo.framework/client-system/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-system/2.2/car">org.apache.geronimo.framework/client-system/${version}/car</artifact-alias> > </plugin-artifact> > </instance> > > > Modified: geronimo/server/branches/2.2/framework/configs/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- geronimo/server/branches/2.2/framework/configs/pom.xml (original) > +++ geronimo/server/branches/2.2/framework/configs/pom.xml Thu Jun > 10 03:22:19 2010 > @@ -85,6 +85,7 @@ > > <source-repository>${PluginSrcRepoCentral}</source-repository> > > <source-repository>${PluginSrcRepoApacheSnapshots}</source-repository> > <artifact-alias > > key="${groupId}/${artifactId}//car">${groupId}/${artifactId}/${version}/car</artifact-alias> > + <artifact-alias > > key="${groupId}/${artifactId}/2.2/car">${groupId}/${artifactId}/${version}/car</artifact-alias> > </plugin-artifact> > </commonInstance> > </configuration> > > Modified: > geronimo/server/branches/2.2/plugins/client/client-security/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/client-security/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/client-security/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- > geronimo/server/branches/2.2/plugins/client/client-security/pom.xml > (original) > +++ > geronimo/server/branches/2.2/plugins/client/client-security/pom.xml > Thu Jun 10 03:22:19 2010 > @@ -78,6 +78,7 @@ > <config-xml-content load="false" /> > <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-security//car">org.apache.geronimo.configs/client-security/${version}/car</artifact-alias> > <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-security/${version}/car">org.apache.geronimo.configs/client-security/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="org.apache.geronimo.framework/j2ee-security/2.2/car">org.apache.geronimo.configs/client-security/${version}/car</artifact-alias> > </plugin-artifact> > </instance> > </configuration> > > Modified: geronimo/server/branches/2.2/plugins/client/client/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/client/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/client/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- geronimo/server/branches/2.2/plugins/client/client/pom.xml > (original) > +++ geronimo/server/branches/2.2/plugins/client/client/pom.xml Thu > Jun 10 03:22:19 2010 > @@ -153,6 +153,7 @@ > <config-xml-content load="false" /> > <artifact-alias server="client" > > key="org.apache.geronimo.configs/j2ee-server//car">org.apache.geronimo.configs/client/${version}/car</artifact-alias> > <artifact-alias server="client" > > key="org.apache.geronimo.configs/j2ee-server/${version}/car">org.apache.geronimo.configs/client/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="org.apache.geronimo.configs/j2ee-server/2.2/car">org.apache.geronimo.configs/client/${version}/car</artifact-alias> > </plugin-artifact> > </instance> > </configuration> > > Modified: geronimo/server/branches/2.2/plugins/client/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/client/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- geronimo/server/branches/2.2/plugins/client/pom.xml (original) > +++ geronimo/server/branches/2.2/plugins/client/pom.xml Thu Jun 10 > 03:22:19 2010 > @@ -60,6 +60,7 @@ > > <source-repository>${PluginSrcRepoApacheSnapshots}</source-repository> > > <artifact-alias server="client" > > key="${groupId}/${artifactId}//car">${groupId}/${artifactId}/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="${groupId}/${artifactId}/2.2/car">${groupId}/${artifactId}/${version}/car</artifact-alias> > </plugin-artifact> > </commonInstance> > </configuration> > > Modified: > geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- > geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml > (original) > +++ > geronimo/server/branches/2.2/plugins/connector/client-transaction/pom.xml > Thu Jun 10 03:22:19 2010 > @@ -105,6 +105,7 @@ > <config-xml-content load="false" /> > <artifact-alias server="client" > > key="org.apache.geronimo.configs/transaction//car">org.apache.geronimo.configs/client-transaction/${version}/car</artifact-alias> > <artifact-alias server="client" > > key="org.apache.geronimo.configs/transaction/${version}/car">org.apache.geronimo.configs/client-transaction/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="org.apache.geronimo.configs/transaction/2.2/car">org.apache.geronimo.configs/client-transaction/${version}/car</artifact-alias> > </plugin-artifact> > </instance> > </configuration> > > Modified: > geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- > geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml > (original) > +++ > geronimo/server/branches/2.2/plugins/corba/client-corba-yoko/pom.xml > Thu Jun 10 03:22:19 2010 > @@ -178,6 +178,7 @@ > <plugin-artifact> > <config-xml-content load="false" /> > <artifact-alias server="client" > > key="${groupId}/${artifactId}//car">${groupId}/${artifactId}/${version}/car</artifact-alias> > + <artifact-alias server="client" > > key="${groupId}/${artifactId}/2.2/car">${groupId}/${artifactId}/${version}/car</artifact-alias> > </plugin-artifact> > </instance> > </configuration> > > Modified: geronimo/server/branches/2.2/plugins/pom.xml > URL: > > http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/pom.xml?rev=953208&r1=953207&r2=953208&view=diff > > <http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/pom.xml?rev=953208&r1=953207&r2=953208&view=diff> > > ============================================================================== > --- geronimo/server/branches/2.2/plugins/pom.xml (original) > +++ geronimo/server/branches/2.2/plugins/pom.xml Thu Jun 10 03:22:19 > 2010 > @@ -125,6 +125,7 @@ > > <source-repository>${PluginSrcRepoCentral}</source-repository> > > <source-repository>${PluginSrcRepoApacheSnapshots}</source-repository> > <artifact-alias > > key="${groupId}/${artifactId}//car">${groupId}/${artifactId}/${version}/car</artifact-alias> > + <artifact-alias > > key="${groupId}/${artifactId}/2.2/car">${groupId}/${artifactId}/${version}/car</artifact-alias> > </plugin-artifact> > </commonInstance> > </configuration> > > > > > > -- > Ivan