ActiveMQ
  1. ActiveMQ
  2. AMQ-3260

Javadoc refinements are necessary as exposed by the ActiveMQ build process

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.5.0
    • Fix Version/s: 5.6.0
    • Component/s: None
    • Labels:
      None
    • Environment:

      Product Version = NetBeans IDE 7.0 RC1 (Build 201103280000)
      Operating System = Windows Vista version 6.0 running on x86
      Java; VM; Vendor = 1.6.0_23
      Runtime = Java HotSpot(TM) Client VM 19.0-b09

      Description

      Javadoc refinements are necessary as exposed by the ActiveMQ build process

      Note that these refinements are trivial in nature and the only downside in performing the improvements is the time and commitment to make the changes.

      The benefits include a better Javadoc documentation set and removal of the warnings from the detailed build process.

      I will include the Javadoc related warnings seen in the build process as an attached comment.

      1. JAAS-DIFF.txt
        0.7 kB
        Robert Liguori
      2. CONSOLE-DIFFS.txt
        3 kB
        Robert Liguori
      3. JavadocWarnings.txt
        155 kB
        Robert Liguori

        Activity

        Hide
        Timothy Bish added a comment -

        All supplied patches have been applied.

        Show
        Timothy Bish added a comment - All supplied patches have been applied.
        Hide
        Timothy Bish added a comment -

        Applied the console diffs patch

        Show
        Timothy Bish added a comment - Applied the console diffs patch
        Hide
        Robert Liguori added a comment -

        I reattached the CONSOLE-DIFFS.txt file and checked the appropriate box granting the license.

        I also added the warning output, clearly organized (by the number of warnings) as an attachment.

        Show
        Robert Liguori added a comment - I reattached the CONSOLE-DIFFS.txt file and checked the appropriate box granting the license. I also added the warning output, clearly organized (by the number of warnings) as an attachment.
        Hide
        Timothy Bish added a comment -

        When you attach the file you need to check the box "Grant license to ASF for inclusion in ASF works"

        Show
        Timothy Bish added a comment - When you attach the file you need to check the box "Grant license to ASF for inclusion in ASF works"
        Hide
        Robert Liguori added a comment -

        "The console diffs file needs to have grants for the apache license."

        I don't know what this means, can you explain further... specifically, what is it that I need to do so the patch is valid.

        Show
        Robert Liguori added a comment - "The console diffs file needs to have grants for the apache license." I don't know what this means, can you explain further... specifically, what is it that I need to do so the patch is valid.
        Hide
        Timothy Bish added a comment -

        Applied the Jaas patch

        Show
        Timothy Bish added a comment - Applied the Jaas patch
        Hide
        Timothy Bish added a comment -

        The console diffs file needs to have grants for the apache license.

        Show
        Timothy Bish added a comment - The console diffs file needs to have grants for the apache license.
        Hide
        Timothy Bish added a comment -

        No patch supplied to address the issue. The amount of work needed to fix this issue would be wasteful and better spent on implementing the new openwire support in Apollo.

        Show
        Timothy Bish added a comment - No patch supplied to address the issue. The amount of work needed to fix this issue would be wasteful and better spent on implementing the new openwire support in Apollo.
        Hide
        Timothy Bish added a comment -

        Fix this now would be a very large undertaking and not sure how valuable it'd be. If you want to attach a patch that fixes them all we'd welcome it.

        Show
        Timothy Bish added a comment - Fix this now would be a very large undertaking and not sure how valuable it'd be. If you want to attach a patch that fixes them all we'd welcome it.

          People

          • Assignee:
            Unassigned
            Reporter:
            Robert Liguori
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development