Maven 2.x Compiler Plugin

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

relates to

Bug - A problem which impairs or prevents the functions of the product. PLXCOMP-87 The Javac Compiler Wrapper breaks the Build on Warnings in Java 6.

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Vote (5)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved: