Maven Changes Plugin
  1. Maven Changes Plugin
  2. MCHANGES-46

There is no link to the RSS feed of changes in the changes report

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9
    • Component/s: changes.xml
    • Labels:
      None
    1. MCHANGES-46.patch
      24 kB
      Lukas JOSEFIK
    2. MCHANGES-46-maven-changes-plugin.patch
      29 kB
      Denis Cabasson

      Activity

      Dennis Lundberg created issue -
      Hide
      Dennis Lundberg added a comment -

      I have added code to create the link to the RSS feed, but I have commented it out. The reason being that there is no RSS feed file. So the changes.rss file needs to be created as well.

      In the Maven 1 version of the plugin this is done in Jelly, but using XSL should work. Perhaps an Ant mojo using the xslt task can do the job.

      Show
      Dennis Lundberg added a comment - I have added code to create the link to the RSS feed, but I have commented it out. The reason being that there is no RSS feed file. So the changes.rss file needs to be created as well. In the Maven 1 version of the plugin this is done in Jelly, but using XSL should work. Perhaps an Ant mojo using the xslt task can do the job.
      Hide
      Denis Cabasson added a comment -

      Proposed patch to enable RSS feed generations.

      This patch uses the new changes model, as given in MCHANGES-47

      TODO : Are those really the significant RSS feeds we want to generate?

      TODO : Look more closely to the changes-actions-rss.xml feed, and wether or not it should be generated, depending on the current Version of the project. A snpashot version of the project should generate this feed, as well as a alpha/beta release while a stable one should not. Depends too on the fact that the most recent release in the changes.xml is not necessarily up to date with current Version of project.

      Anyway, this patch is more food-for-thought than a working patch.

      Futher work should be delayed until MCHANGES-47 model is accepted or refused.

      Show
      Denis Cabasson added a comment - Proposed patch to enable RSS feed generations. This patch uses the new changes model, as given in MCHANGES-47 TODO : Are those really the significant RSS feeds we want to generate? TODO : Look more closely to the changes-actions-rss.xml feed, and wether or not it should be generated, depending on the current Version of the project. A snpashot version of the project should generate this feed, as well as a alpha/beta release while a stable one should not. Depends too on the fact that the most recent release in the changes.xml is not necessarily up to date with current Version of project. Anyway, this patch is more food-for-thought than a working patch. Futher work should be delayed until MCHANGES-47 model is accepted or refused.
      Denis Cabasson made changes -
      Field Original Value New Value
      Attachment MCHANGES-46-maven-changes-plugin.patch [ 21979 ]
      Dennis Lundberg made changes -
      Component/s changes-report [ 12877 ]
      Dennis Lundberg made changes -
      Issue Type Bug [ 1 ] Improvement [ 4 ]
      Hide
      Lukas Theussl added a comment -

      Attaching an alternative that uses the rome library for rss generation. Since I haven't checked what other work went into this issue before, and the changes plugin just had some release, I leave it here for review.

      Show
      Lukas Theussl added a comment - Attaching an alternative that uses the rome library for rss generation. Since I haven't checked what other work went into this issue before, and the changes plugin just had some release, I leave it here for review.
      Lukas Theussl made changes -
      Attachment MCHANGES-46.patch [ 55691 ]
      Hide
      Melloware added a comment -

      It has been years for this and it used to be in the old Maven 1 reports and it was useful. Someone please implement this patch!

      Show
      Melloware added a comment - It has been years for this and it used to be in the old Maven 1 reports and it was useful. Someone please implement this patch!
      Hide
      Lukas Theussl added a comment -

      I have committed my own patch after review and slight adaptations. However, I didn't do any extensive testing, so would appreciate any feedback. Use it with a configuration parameter 'feedType' with supported values: "rss_0.9", "rss_0.91N" (RSS 0.91 Netscape), "rss_0.91U" (RSS 0.91 Userland), "rss_0.92", "rss_0.93", "rss_0.94", "rss_1.0", "rss_2.0", "atom_0.3", "atom_1.0"

      Show
      Lukas Theussl added a comment - I have committed my own patch after review and slight adaptations. However, I didn't do any extensive testing, so would appreciate any feedback. Use it with a configuration parameter 'feedType' with supported values: "rss_0.9", "rss_0.91N" (RSS 0.91 Netscape), "rss_0.91U" (RSS 0.91 Userland), "rss_0.92", "rss_0.93", "rss_0.94", "rss_1.0", "rss_2.0", "atom_0.3", "atom_1.0"
      Lukas Theussl made changes -
      Fix Version/s 2.9 [ 18785 ]
      Resolution Fixed [ 1 ]
      Assignee Lukas Theussl [ lukas ]
      Status Open [ 1 ] Closed [ 6 ]
      Hide
      Melloware added a comment -

      Thanks looks good to me!

      Show
      Melloware added a comment - Thanks looks good to me!
      Mark Thomas made changes -
      Project Import Sun Apr 05 09:14:45 UTC 2015 [ 1428225285377 ]
      Mark Thomas made changes -
      Workflow jira [ 12717881 ] Default workflow, editable Closed status [ 12749503 ]
      Mark Thomas made changes -
      Project Import Sun Apr 05 22:40:15 UTC 2015 [ 1428273615853 ]
      Mark Thomas made changes -
      Workflow jira [ 12955353 ] Default workflow, editable Closed status [ 12992379 ]
      Mark Thomas made changes -
      Assignee Lukas JOSEFIK [ lukas ] Lukas Theussl [ ltheussl ]
      Transition Time In Source Status Execution Times Last Executer Last Execution Date
      Open Open Closed Closed
      2346d 22h 55m 1 Lukas Theussl 18/Dec/12 03:07

        People

        • Assignee:
          Lukas Theussl
          Reporter:
          Dennis Lundberg
        • Votes:
          2 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development