This wasn't your fault - you were actually doing the "right thing" considering the current state of SVN, however this is not backwards compatible and so changes need to be considered in that context too.

What happened is that in r399801 maven-archetype-bundles/pom.xml changes the group ID of all the archetypes. None have been released from this POM yet.

I'll correct SVN now, so no need to review the patch - however I will ask that before making changes (and same goes for reviewers of patches), that the consequences be considered. If you are unsure - just ask - someone will know here.

Thanks for the patch!

Cheers,
Brett

On 22/07/2006 5:55 AM, FA Valencia See wrote:
Good day to you,

I see. Thanks for the correction and sorry about that. Everything related to
it was all about "archetype" that I thought it was just a typo-error.

Anyway, I'd revise the patch to accomadate your comments. However, I'd just
like to clarify your comment regarding the parent pom. are you referring to
the artifact-id of the parent pom (meaning convert "maven-archetype-parent"
to "maven-archetypes-parent")?

Thanks

- Franz

On 7/21/06, Brett Porter <[EMAIL PROTECTED]> wrote:

-1 on the first change. Please revert that part of the patch.

All of the released archetypes so far are under "archetypes", not
"archetype".

http://repo1.maven.org/maven2/org/apache/maven/archetypes/

It seems that the parent pom for the archetypes was also changed to
"archetype" at some point in the past. It should be changed back to
maintain compatibility instead.

- Brett

On 21/07/2006 7:35 PM, [EMAIL PROTECTED] wrote:
> Author: aramirez
> Date: Fri Jul 21 02:35:32 2006
> New Revision: 424243
>
> URL: http://svn.apache.org/viewvc?rev=424243&view=rev
> Log:
> PR: ARCHETYPE-36
> Submitted By: Franz Allan See
>
> -changed the default-value of the archetypeGroupId from "
org.apache.maven.archetypes" to "org.apache.maven.archetype"
>
> -removed some lines in the <resources> tag
>
> Modified:
>
maven/archetype/trunk/maven-archetype-bundles/maven-archetype-j2ee-simple/src/main/resources/META-INF/archetype.xml
>
maven/archetype/trunk/maven-archetype/maven-archetype-plugin/src/main/java/org/apache/maven/plugin/archetype/MavenArchetypeMojo.java
>
> Modified:
maven/archetype/trunk/maven-archetype-bundles/maven-archetype-j2ee-simple/src/main/resources/META-INF/archetype.xml
> URL:
http://svn.apache.org/viewvc/maven/archetype/trunk/maven-archetype-bundles/maven-archetype-j2ee-simple/src/main/resources/META-INF/archetype.xml?rev=424243&r1=424242&r2=424243&view=diff
>
==============================================================================
> ---
maven/archetype/trunk/maven-archetype-bundles/maven-archetype-j2ee-simple/src/main/resources/META-INF/archetype.xml
(original)
> +++
maven/archetype/trunk/maven-archetype-bundles/maven-archetype-j2ee-simple/src/main/resources/META-INF/archetype.xml
Fri Jul 21 02:35:32 2006
> @@ -12,8 +12,5 @@
>      <resource>servlets/servlet/pom.xml</resource>
>      <resource>servlets/servlet/src/main/webapp/index.jsp</resource>

> <resource>servlets/servlet/src/main/webapp/WEB-INF/web.xml</resource>
> -    <resource>site/pom.xml</resource>
> -    <resource>site/src/site/site.xml</resource>
> -    <resource>site/src/site/apt/site.apt</resource>
>    </resources>
>  </archetype>
>
> Modified:
maven/archetype/trunk/maven-archetype/maven-archetype-plugin/src/main/java/org/apache/maven/plugin/archetype/MavenArchetypeMojo.java
> URL:
http://svn.apache.org/viewvc/maven/archetype/trunk/maven-archetype/maven-archetype-plugin/src/main/java/org/apache/maven/plugin/archetype/MavenArchetypeMojo.java?rev=424243&r1=424242&r2=424243&view=diff
>
==============================================================================
> ---
maven/archetype/trunk/maven-archetype/maven-archetype-plugin/src/main/java/org/apache/maven/plugin/archetype/MavenArchetypeMojo.java
(original)
> +++
maven/archetype/trunk/maven-archetype/maven-archetype-plugin/src/main/java/org/apache/maven/plugin/archetype/MavenArchetypeMojo.java
Fri Jul 21 02:35:32 2006
> @@ -68,7 +68,7 @@
>      private ArtifactRepository localRepository;
>
>      /**
> -     * @parameter expression="${archetypeGroupId}" default-value="
org.apache.maven.archetypes"
> +     * @parameter expression="${archetypeGroupId}" default-value="
org.apache.maven.archetype"
>       * @required
>       */
>      private String archetypeGroupId;
>
>


--
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





--
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to