XBean
  1. XBean
  2. XBEAN-31

[RTC] it would be great if the m2 plugin would automatically add the XSD and .xsd.html files as artifacts into the build

    Details

    • Type: Wish Wish
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.6
    • Component/s: maven-plugin
    • Labels:
      None

      Description

      So that the XSD and HTML reference would be automatically deployed into the m2 repository.

      See the attach-artifact for how to do it manually in each POM. But given how many projects are using the m2 plugin it would simplify our lives if the m2 plugin did this for us

      http://mojo.codehaus.org/build-helper-maven-plugin/howto.html

      1. XBEAN-31.matt.diff
        2 kB
        Matt Hogstrom
      2. XBEAN-31.diff
        2 kB
        Guillaume Nodet
      3. XBEAN-31.diff
        2 kB
        Guillaume Nodet

        Issue Links

          Activity

          Hide
          Guillaume Nodet added a comment -

          Attach patch uploads the generated xsd and documentation by default.
          This can be turned off by using the schemaAsArtifact boolean attribute on the mojo.

          Show
          Guillaume Nodet added a comment - Attach patch uploads the generated xsd and documentation by default. This can be turned off by using the schemaAsArtifact boolean attribute on the mojo.
          Hide
          james strachan added a comment -

          +1

          This is awesome!

          Show
          james strachan added a comment - +1 This is awesome!
          Hide
          Jeff Genender added a comment -

          +1...this looks good to me.

          Show
          Jeff Genender added a comment - +1...this looks good to me.
          Hide
          Dain Sundstrom added a comment -

          +1 very cool

          Show
          Dain Sundstrom added a comment - +1 very cool
          Hide
          Matt Hogstrom added a comment -

          +1 ...

          Show
          Matt Hogstrom added a comment - +1 ...
          Hide
          David Jencks added a comment -

          This looks like a good idea to me and I think it should be applied but I can't apply the patch: all chunks are rejected. Maybe this is an eol-style issue?

          I'm going to vote +1 on the patch and -1 on using patch to apply changes.

          Show
          David Jencks added a comment - This looks like a good idea to me and I think it should be applied but I can't apply the patch: all chunks are rejected. Maybe this is an eol-style issue? I'm going to vote +1 on the patch and -1 on using patch to apply changes.
          Hide
          Guillaume Nodet added a comment -

          Here is a new patch for review with the conflict solved.
          The conflict was cause by tabs / space changes, so the content is exactly the same
          as the previous patch (but whitespaces)

          Show
          Guillaume Nodet added a comment - Here is a new patch for review with the conflict solved. The conflict was cause by tabs / space changes, so the content is exactly the same as the previous patch (but whitespaces)
          Hide
          Matt Hogstrom added a comment -

          Guillaume, I tried the patch and it didn't work. Since it was fairly small I went ahead and manually applied the changes. And I think this will work.

          The patch is applied from geronimo/xbean/trunk via patch -p0 < XBEAN-31.matt.diff

          I also corrected a typo in one of the System.outs for AsArtifactId.

          Since this is your patch, I'll reset the vote and give this my +1

          Cheers.

          Show
          Matt Hogstrom added a comment - Guillaume, I tried the patch and it didn't work. Since it was fairly small I went ahead and manually applied the changes. And I think this will work. The patch is applied from geronimo/xbean/trunk via patch -p0 < XBEAN-31 .matt.diff I also corrected a typo in one of the System.outs for AsArtifactId. Since this is your patch, I'll reset the vote and give this my +1 Cheers.
          Hide
          Matt Hogstrom added a comment -

          Resetting vote.

          Show
          Matt Hogstrom added a comment - Resetting vote.
          Hide
          Matt Hogstrom added a comment -

          Starting again, here is my +1. Jeff, Jencks, anyone, can you vote this one.

          Thanks

          Show
          Matt Hogstrom added a comment - Starting again, here is my +1. Jeff, Jencks, anyone, can you vote this one. Thanks
          Hide
          David Jencks added a comment -

          Matt's version of the patch applies and builds for me, I'll repeat my +1

          Show
          David Jencks added a comment - Matt's version of the patch applies and builds for me, I'll repeat my +1
          Hide
          Guillaume Nodet added a comment -

          Author: gnodet
          Date: Fri Aug 18 02:03:06 2006
          New Revision: 432531

          URL: http://svn.apache.org/viewvc?rev=432531&view=rev
          Log:
          XBEAN-31: m2 plugin should upload XSD and xsd.html when deploying

          Modified:
          geronimo/xbean/trunk/maven-xbean-plugin/src/main/java/org/apache/xbean/maven/XBeanMojo.java

          Show
          Guillaume Nodet added a comment - Author: gnodet Date: Fri Aug 18 02:03:06 2006 New Revision: 432531 URL: http://svn.apache.org/viewvc?rev=432531&view=rev Log: XBEAN-31 : m2 plugin should upload XSD and xsd.html when deploying Modified: geronimo/xbean/trunk/maven-xbean-plugin/src/main/java/org/apache/xbean/maven/XBeanMojo.java

            People

            • Assignee:
              Guillaume Nodet
              Reporter:
              james strachan
            • Votes:
              5 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development