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,  <[EMAIL PROTECTED]> 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>






Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to