Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta-2
    • Fix Version/s: 2.1
    • Component/s: changes.xml
    • Labels:
      None
    • Number of attachments :
      0

      Description

      A DTD/XSD for changes.xml would be extremely useful for IDE auto-completion. It should be hosted on the apache site.

        Issue Links

          Activity

          Show
          Lukas Theussl added a comment - The m1 plugin has an xsd already: http://svn.apache.org/viewvc/maven/maven-1/plugins/trunk/changes/src/plugin-resources/xsd/changes.xsd?revision=354742&view=markup
          Hide
          Mark Hobson added a comment -

          Cool, any chance of hosting this somewhere convenient? We would also ideally need a namespace.

          Show
          Mark Hobson added a comment - Cool, any chance of hosting this somewhere convenient? We would also ideally need a namespace.
          Hide
          Dennis Lundberg added a comment -

          Please note that the Maven 2 version of the plugin does not yet have all the features of the Maven 1 plugin.

          Work has been done by Denis Cabasson in MCHANGES-47 to try to use a Modello model in the changes plugin. With this in place it would be easy to generate an xsd for changes.xml. I've been working a bit on Modello to apply patches that are required by Denis' patch for the changes plugin.

          Show
          Dennis Lundberg added a comment - Please note that the Maven 2 version of the plugin does not yet have all the features of the Maven 1 plugin. Work has been done by Denis Cabasson in MCHANGES-47 to try to use a Modello model in the changes plugin. With this in place it would be easy to generate an xsd for changes.xml. I've been working a bit on Modello to apply patches that are required by Denis' patch for the changes plugin.
          Hide
          Olivier Lamy added a comment -

          If we use the patch from MCHANGES-47 the changes xml need changes and this will break backward compatibility.
          To preserve backward comp, we have to fix MODELLO-113 (I'm working on providing a patch for this).

          Show
          Olivier Lamy added a comment - If we use the patch from MCHANGES-47 the changes xml need changes and this will break backward compatibility. To preserve backward comp, we have to fix MODELLO-113 (I'm working on providing a patch for this).
          Hide
          Olivier Lamy added a comment -

          fixed in rev 680301.

          Show
          Olivier Lamy added a comment - fixed in rev 680301.
          Hide
          Milos Kleint added a comment -

          will the xsd be uploaded to http://maven.apache.org/xsd/ or will it be elsewhere?

          Show
          Milos Kleint added a comment - will the xsd be uploaded to http://maven.apache.org/xsd/ or will it be elsewhere?
          Hide
          Olivier Lamy added a comment -

          Yep good idea !
          It will included in the site generation too.

          Show
          Olivier Lamy added a comment - Yep good idea ! It will included in the site generation too.
          Hide
          Olivier Lamy added a comment -

          Note using the Xpp3Reader has the following side effect.
          People must add

          <action type="fix" dev="toto">
            <![CDATA[ 
             Add <i>foo</i> in the bean .
            ]]>
          </action>
          Whereas the following format was supported before :
          <action type="fix" dev="toto">
             Add <i>foo</i> in the bean .
          </action>
          

          I will see if I can find a solution in modello if not I will add note in the release note.

          Show
          Olivier Lamy added a comment - Note using the Xpp3Reader has the following side effect. People must add <action type= "fix" dev= "toto" > <![CDATA[ Add <i> foo </i> in the bean . ]]> </action> Whereas the following format was supported before : <action type= "fix" dev= "toto" > Add <i> foo </i> in the bean . </action> I will see if I can find a solution in modello if not I will add note in the release note.

            People

            • Assignee:
              Olivier Lamy
              Reporter:
              Mark Hobson
            • Votes:
              3 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: