geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Donald Woods <dwo...@apache.org>
Subject Re: svn commit: r953208 - in /geronimo/server/branches/2.2: framework/configs/ framework/configs/client-system/ plugins/ plugins/client/ plugins/client/client-security/ plugins/client/client/ plugins/connector/client-transaction/ plugins/corba/client
Date Thu, 10 Jun 2010 12:50:27 GMT
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 <xiaming@apache.org <mailto:xiaming@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

Mime
View raw message