Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: PDF-REPORT-1.1
    • Fix Version/s: PDF-REPORT-1.2
    • Component/s: PDF Report
    • Labels:
      None
    • Number of attachments :
      0

      Activity

      Hide
      Antonio Manuel Muñiz Martín added a comment -

      Hi,

      As you know PDF Report Plugin is at same time a Maven plugin and a Sonar plugin, so actually his packaging is maven-plugin and it can't be changed. Currently I only know one solution, write separate plugins, one for Maven (called as post job action) and other for Sonar, containing all Sonar related classes and configuration.

      Do you know another option?

      Thanks,
      Antonio.

      Show
      Antonio Manuel Muñiz Martín added a comment - Hi, As you know PDF Report Plugin is at same time a Maven plugin and a Sonar plugin, so actually his packaging is maven-plugin and it can't be changed. Currently I only know one solution, write separate plugins, one for Maven (called as post job action) and other for Sonar, containing all Sonar related classes and configuration. Do you know another option? Thanks, Antonio.
      Hide
      Evgeny Mandrikov added a comment -

      Hi Antonio,

      sonar-plugin packaging needed to create correct metadata for your plugin, in it's turn those metadata required for update center. Of course you can add them manually, but for me it sounds like a hack, which may lead to problems. So my suggestion is to create separate plugins.

      Show
      Evgeny Mandrikov added a comment - Hi Antonio, sonar-plugin packaging needed to create correct metadata for your plugin, in it's turn those metadata required for update center. Of course you can add them manually, but for me it sounds like a hack, which may lead to problems. So my suggestion is to create separate plugins.
      Hide
      Evgeny Mandrikov added a comment -

      Also note that your plugin should have key "pdfreport", since this key was reserved for you in update center.

      Show
      Evgeny Mandrikov added a comment - Also note that your plugin should have key "pdfreport", since this key was reserved for you in update center.
      Hide
      Antonio Manuel Muñiz Martín added a comment -

      Ok. I will divide the plugin in two, Sonar and Maven plugin.
      Thanks for the reserve

      Show
      Antonio Manuel Muñiz Martín added a comment - Ok. I will divide the plugin in two, Sonar and Maven plugin. Thanks for the reserve
      Hide
      Evgeny Mandrikov added a comment -

      Hi Antonio,
      I see that you closed this issue - congratulations!
      But since you made significant changes in plugin, could you please request vote/review before release ?

      Show
      Evgeny Mandrikov added a comment - Hi Antonio, I see that you closed this issue - congratulations! But since you made significant changes in plugin, could you please request vote/review before release ?
      Hide
      Antonio Manuel Muñiz Martín added a comment -

      Hi Evgeny,

      Of course. I'm waiting for a reply in http://jira.codehaus.org/browse/SONARPLUGINS-456. If there is no reply then I will mark it as invalid and will call for review.

      Show
      Antonio Manuel Muñiz Martín added a comment - Hi Evgeny, Of course. I'm waiting for a reply in http://jira.codehaus.org/browse/SONARPLUGINS-456 . If there is no reply then I will mark it as invalid and will call for review.

        People

        • Assignee:
          Antonio Manuel Muñiz Martín
          Reporter:
          Evgeny Mandrikov
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: