Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Labels:
      None
    • Environment:

      Description

      I'm attaching the build console output and surefire output file for the failing unit test.

      1. build.log
        11 kB
        John Casey
      2. MJAVADOC-210_new.patch
        5 kB
        Siveton Vincent
      3. MJAVADOC-210.patch
        6 kB
        Benjamin Bentmann
      4. org.apache.maven.plugin.javadoc.JavadocReportTest.txt
        0.8 kB
        John Casey

        Activity

        John Casey created issue -
        Hide
        Benjamin Bentmann added a comment -

        IIRC, Mac has no tools.jar but a similar thing named classes.jar which is already part of the bootstrap class path I believe. So I coded up this patch. I have no Mac box to test it, so it's all up to you John. If it works, I leave it to Vincent to do a final review and maybe apply.

        Show
        Benjamin Bentmann added a comment - IIRC, Mac has no tools.jar but a similar thing named classes.jar which is already part of the bootstrap class path I believe. So I coded up this patch. I have no Mac box to test it, so it's all up to you John. If it works, I leave it to Vincent to do a final review and maybe apply.
        Benjamin Bentmann made changes -
        Field Original Value New Value
        Attachment MJAVADOC-210.patch [ 36246 ]
        Show
        Siveton Vincent added a comment - According [1] , I am not sure if it is classes.jar. John? [1] https://svn.codehaus.org/plexus/plexus-components/trunk/plexus-compiler/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java
        Hide
        Siveton Vincent added a comment -

        John, could you zip your target dir?

        Show
        Siveton Vincent added a comment - John, could you zip your target dir?
        Hide
        Siveton Vincent added a comment -

        Thanks Benjamin! I modified your patch to make getToolsJar() always returns tools.jar (win unix) or classes.jar (mac) based on [1].
        John, could you review this patch since I have no QA env that uses Mac?

        [1] http://developer.apple.com/documentation/Java/Conceptual/Java14Development/02-JavaDevTools/JavaDevTools.html#//apple_ref/doc/uid/TP40001884-208117

        Show
        Siveton Vincent added a comment - Thanks Benjamin! I modified your patch to make getToolsJar() always returns tools.jar (win unix) or classes.jar (mac) based on [1] . John, could you review this patch since I have no QA env that uses Mac? [1] http://developer.apple.com/documentation/Java/Conceptual/Java14Development/02-JavaDevTools/JavaDevTools.html#//apple_ref/doc/uid/TP40001884-208117
        Siveton Vincent made changes -
        Attachment MJAVADOC-210_new.patch [ 36259 ]
        Siveton Vincent made changes -
        Fix Version/s 2.5 [ 14120 ]
        Brett Porter made changes -
        Assignee Brett Porter [ brettporter ]
        Benjamin Bentmann made changes -
        Link This issue is related to MNGFAQ-73 [ MNGFAQ-73 ]
        Brett Porter made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 11:56:47 UTC 2015 [ 1428235007093 ]
        Mark Thomas made changes -
        Workflow jira [ 12722448 ] Default workflow, editable Closed status [ 12762426 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 00:11:46 UTC 2015 [ 1428279106587 ]
        Mark Thomas made changes -
        Workflow jira [ 12959943 ] Default workflow, editable Closed status [ 12996791 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        14h 56m 1 Brett Porter 01/Aug/08 08:33

          People

          • Assignee:
            Brett Porter
            Reporter:
            John Casey
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development