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
    • 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

      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

      1. WidgetCore.zip
        13 kB
        Anton Makeev
      2. MCOMPILER-140.diff
        3 kB
        Jesse Glick

        Activity

        Hide
        Kristian Rosenvold added a comment -

        And r1293132 for compiler-plugin

        Show
        Kristian Rosenvold added a comment - And r1293132 for compiler-plugin
        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
        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
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development