Maven Compiler Plugin
  1. Maven Compiler Plugin
  2. MCOMPILER-140

Compiler error messages do not contain drive letter on windows

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2, 2.3, 2.3.1, 2.3.2
    • Fix Version/s: None
    • Labels:
      None
    • Environment:
      Maven 2.2.1, Windows
    • Number of attachments :
      2

      Description

      See original bug request: http://youtrack.jetbrains.net/issue/IDEA-59521

      User description:
      I tried it with a toy project and it worked as you say. The errors in the console for the toy project had the complete path in the error:
      [INFO] Compilation failure

      C:\projects\java\eclipseOnly\WidgetCore\src\main\java\com\acme\core\CoreClass.java:[6,8] not a statement

      C:\projects\java\eclipseOnly\WidgetCore\src\main\java\com\acme\core\CoreClass.java:[6,14] ';' expected

      In my non-toy project where I'm seeing the error, the C: is not there.

      [ERROR] COMPILATION ERROR :
      [INFO]
      [ERROR] \projects\java\eclipseproj\MyProj\unittests\com\acme\messaging\configuration\ConfigurationTest.java:[12,8] not a statement

      [ERROR] \projects\java\eclipseproj\MyProj\unittests\com\acme\messaging\configuration\ConfigurationTest.java:[12,14] ';' expected

        Issue Links

          Activity

          Hide
          Jesse Glick added a comment -

          Patch for PLXCOMP-87 seems to have broken parsing of the Windows drive letter. Unit test coverage for this rather subtle parsing code is quite poor.

          Show
          Jesse Glick added a comment - Patch for PLXCOMP-87 seems to have broken parsing of the Windows drive letter. Unit test coverage for this rather subtle parsing code is quite poor.
          Hide
          Jesse Glick added a comment -

          Patch incl. unit test which seems to fix the issue. Confirmed by building patched version of plexus-compiler-javac, including that as a dep in maven-compiler-plugin, and requesting the patched plugin in a Maven project on which http://netbeans.org/bugzilla/show_bug.cgi?id=197381 could previously be reproduced.

          Show
          Jesse Glick added a comment - Patch incl. unit test which seems to fix the issue. Confirmed by building patched version of plexus-compiler-javac , including that as a dep in maven-compiler-plugin , and requesting the patched plugin in a Maven project on which http://netbeans.org/bugzilla/show_bug.cgi?id=197381 could previously be reproduced.
          Hide
          Kristian Rosenvold added a comment -

          Fixed in 32ef15b5a277015a4071486c3a00db6853fab2d4, thanks for the patch !

          Show
          Kristian Rosenvold added a comment - Fixed in 32ef15b5a277015a4071486c3a00db6853fab2d4, thanks for the patch !
          Hide
          Kristian Rosenvold added a comment -

          And r1293132 for compiler-plugin

          Show
          Kristian Rosenvold added a comment - And r1293132 for compiler-plugin

            People

            • Assignee:
              Kristian Rosenvold
              Reporter:
              Anton Makeev
            • Votes:
              5 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: