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 Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9
    • Component/s: changes.xml
    • Labels:
      None
    • Number of attachments :
      2
    1. MCHANGES-46.patch
      24 kB
      Lukas Theussl
    2. MCHANGES-46-maven-changes-plugin.patch
      29 kB
      Denis Cabasson

      Activity

      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.
      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.
      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"
      Hide
      Melloware added a comment -

      Thanks looks good to me!

      Show
      Melloware added a comment - Thanks looks good to me!

        People

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

          Dates

          • Created:
            Updated:
            Resolved: