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: r658264 - in /geronimo/samples/trunk/samples: ./ CustomerService/ bank/ calculator-stateless-pojo/ dbtester/ inventory/ jaxws-calculator/ jms-mdb-sample/ ldap-sample-app/ myphonebook/ mytime/ sendmail/ servlet-examples/ timereport/
Date Tue, 20 May 2008 16:38:24 GMT
Guess we should rethink the whole private repository model.  Maybe we 
need to move it to its own location in svn and have it contain depends 
for all releases (much like openejb has been doing), so we can reference 
it as a repo for sample (and other plugin) builds....  Otherwise, if we 
tell users that they have to build the server first, then very few will 
ever be willing to rebuild the samples, IMO.

Also, if Samples rely upon privately pinned depends, then how could we 
expect to only release Sample 2.1 and have it work with 2.1.1, 2.1.2, 
2.1.x, knowing that some depend versions have or will likely change in 
future releases?



-Donald


Joe Bohn wrote:
> Jarek Gawor wrote:
>> Joe,
>>
>> These apache-snapshot entires were there so that a user could checkout
>> a particular sample from svn and build it (otherwise the user had to
>> checkout the entire samples directory).
>>
>> Jarek
>>
> 
> 
> Thanks for the info Jarek.
> 
> Here were my reasons for removal:
> - The root sample pom now has the genesis project-config as a parent.  I 
> believe that we will pull a reference to the apache-snapshot repo from 
> there.
> - When moving to release the samples this snapshot reference would have 
> to be replaced with another repo that would contain released artifacts 
> to have the same effect.  I didn't want to add in both the snapshot and 
> released repos.
> - I don't think it's feasible to build just a sample (or even all 
> samples) any longer (if it ever was).  Samples builds require Geronimo 
> server builds to fulfill dependencies against our svn checked in 
> repository entries (which are not available from any published repo) ... 
> at least that has been my experience.
> 
> I'd be happy to consider some other alternatives to get some ability to 
> build individual samples but at the moment I'm not sure it is feasible.
> 
> Joe
> 
> 
> 
>> On Tue, May 20, 2008 at 10:47 AM,  <jbohn@apache.org> wrote:
>>> Author: jbohn
>>> Date: Tue May 20 07:47:33 2008
>>> New Revision: 658264
>>>
>>> URL: http://svn.apache.org/viewvc?rev=658264&view=rev
>>> Log:
>>> merge r658262 from branches/2.1 - remove references to 
>>> apache-snapshot repo and one more harded version reference
>>>
>>> Modified:
>>>    geronimo/samples/trunk/samples/CustomerService/pom.xml
>>>    geronimo/samples/trunk/samples/bank/pom.xml
>>>    geronimo/samples/trunk/samples/calculator-stateless-pojo/pom.xml
>>>    geronimo/samples/trunk/samples/dbtester/pom.xml
>>>    geronimo/samples/trunk/samples/inventory/pom.xml
>>>    geronimo/samples/trunk/samples/jaxws-calculator/pom.xml
>>>    geronimo/samples/trunk/samples/jms-mdb-sample/pom.xml
>>>    geronimo/samples/trunk/samples/ldap-sample-app/pom.xml
>>>    geronimo/samples/trunk/samples/myphonebook/pom.xml
>>>    geronimo/samples/trunk/samples/mytime/pom.xml
>>>    geronimo/samples/trunk/samples/pom.xml
>>>    geronimo/samples/trunk/samples/sendmail/pom.xml
>>>    geronimo/samples/trunk/samples/servlet-examples/pom.xml
>>>    geronimo/samples/trunk/samples/timereport/pom.xml
>>>
>>> Modified: geronimo/samples/trunk/samples/CustomerService/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/CustomerService/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/CustomerService/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/CustomerService/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -81,22 +81,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/bank/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/bank/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/bank/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/bank/pom.xml Tue May 20 07:47:33 2008
>>> @@ -76,22 +76,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: 
>>> geronimo/samples/trunk/samples/calculator-stateless-pojo/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/calculator-stateless-pojo/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/calculator-stateless-pojo/pom.xml 
>>> (original)
>>> +++ geronimo/samples/trunk/samples/calculator-stateless-pojo/pom.xml 
>>> Tue May 20 07:47:33 2008
>>> @@ -54,22 +54,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/dbtester/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/dbtester/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/dbtester/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/dbtester/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>            </plugin>
>>>        </plugins>
>>>    </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/inventory/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/inventory/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/inventory/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/inventory/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/jaxws-calculator/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/jaxws-calculator/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/jaxws-calculator/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/jaxws-calculator/pom.xml Tue May 
>>> 20 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/jms-mdb-sample/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/jms-mdb-sample/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/jms-mdb-sample/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/jms-mdb-sample/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -67,22 +67,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/ldap-sample-app/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/ldap-sample-app/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/ldap-sample-app/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/ldap-sample-app/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/myphonebook/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/myphonebook/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/myphonebook/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/myphonebook/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -75,22 +75,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/mytime/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/mytime/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/mytime/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/mytime/pom.xml Tue May 20 07:47:33 
>>> 2008
>>> @@ -57,22 +57,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/pom.xml Tue May 20 07:47:33 2008
>>> @@ -301,7 +301,7 @@
>>>                                             <arg 
>>> value="archetype:create"/>
>>>                                             <arg 
>>> value="-DarchetypeGroupId=org.apache.geronimo.samples"/>
>>>                                             <arg 
>>> value="-DarchetypeArtifactId=geronimo-samples-archetype"/>
>>> -                                            <arg 
>>> value="-DarchetypeVersion=2.2-SNAPSHOT"/>
>>> +                                            <arg 
>>> value="-DarchetypeVersion=${version}"/>
>>>                                             <arg 
>>> value="-DartifactId=${sampleName}"/>
>>>                                             <arg 
>>> value="-Dversion=${sampleVersion}"/>
>>>                                         </exec>
>>>
>>> Modified: geronimo/samples/trunk/samples/sendmail/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/sendmail/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/sendmail/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/sendmail/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/servlet-examples/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/servlet-examples/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/servlet-examples/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/servlet-examples/pom.xml Tue May 
>>> 20 07:47:33 2008
>>> @@ -43,22 +43,4 @@
>>>         <module>servlet-examples-jetty</module>
>>>         <module>servlet-examples-tomcat</module>
>>>     </modules>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>> Modified: geronimo/samples/trunk/samples/timereport/pom.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/geronimo/samples/trunk/samples/timereport/pom.xml?rev=658264&r1=658263&r2=658264&view=diff

>>>
>>> ==============================================================================

>>>
>>> --- geronimo/samples/trunk/samples/timereport/pom.xml (original)
>>> +++ geronimo/samples/trunk/samples/timereport/pom.xml Tue May 20 
>>> 07:47:33 2008
>>> @@ -56,22 +56,4 @@
>>>             </plugin>
>>>         </plugins>
>>>     </build>
>>> -
>>> -    <repositories>
>>> -        <repository>
>>> -            <id>apache-snapshots</id>
>>> -            <name>Apache Snapshots Repository</name>
>>> -            
>>> <url>http://people.apache.org/repo/m2-snapshot-repository</url>
>>> -            <layout>default</layout>
>>> -            <snapshots>
>>> -                <enabled>true</enabled>
>>> -                <updatePolicy>daily</updatePolicy>
>>> -                <checksumPolicy>ignore</checksumPolicy>
>>> -            </snapshots>
>>> -            <releases>
>>> -                <enabled>false</enabled>
>>> -            </releases>
>>> -        </repository>
>>> -    </repositories>
>>> -
>>>  </project>
>>>
>>>
>>>
>>
> 
> 

Mime
View raw message