Bug 426872 - Java warnings generated by tycho-compiler-jdt 0.19.0 not detected
Summary: Java warnings generated by tycho-compiler-jdt 0.19.0 not detected
Status: NEW
Alias: None
Product: Hudson
Classification: Technology
Component: Plugins (show other bugs)
Version: 3.0.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Winston Prakash CLA
QA Contact: Geoff Waymark CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-29 05:29 EST by Gunnar Wagenknecht CLA
Modified: 2014-06-12 12:56 EDT (History)
5 users (show)

See Also:


Attachments
sample test project (2.90 KB, application/zip)
2014-04-12 07:13 EDT, Gunnar Wagenknecht CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gunnar Wagenknecht CLA 2014-01-29 05:29:49 EST
For some reasons, Tycho changed the output of compiler warnings. They are using the JDT compiler so it may affect all builds using the JDT compiler. This shmall change breaks the plug-in parsing compiler warnings in Hudson.


Before (Tycho 0.18.1):
---
[WARNING] /jobs/.../SomeClass.java:[72,0] 
  // TODO: we may need to wrap this into a doPriv call
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO: we may need to wrap this into a doPriv call
---

Now (Tycho 0.19.0):
---
[WARNING] /jobs/.../SomeClass.java:[72] 
  // TODO: we may need to wrap this into a doPriv call
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO: we may need to wrap this into a doPriv call
---

The change is the report of the error location: [72,0] vs [72]
Comment 1 Gunnar Wagenknecht CLA 2014-04-12 07:13:10 EDT
Created attachment 241917 [details]
sample test project
Comment 2 Gunnar Wagenknecht CLA 2014-06-05 04:31:58 EDT
Any updates on this?

Tobias, would it be possible to Tycho to revert back to the old style warnings?
Comment 3 Tobias Oberlies CLA 2014-06-12 12:41:17 EDT
(In reply to comment #2)
> Tobias, would it be possible to Tycho to revert back to the old style warnings?
AFAIK this would mean rolling back to an old version of JDT. I don't think that this is an option.

If you wanted, you could swap out the JDT compiler used by Tycho yourself (via POM configuration) though.
Comment 4 Gunnar Wagenknecht CLA 2014-06-12 12:56:08 EDT
(In reply to Tobias Oberlies from comment #3)
> (In reply to comment #2)
> > Tobias, would it be possible to Tycho to revert back to the old style warnings?
> AFAIK this would mean rolling back to an old version of JDT. I don't think
> that this is an option.


You are right. It likely isn't Tycho but Maven.
org.codehaus.plexus.compiler.CompilerMessage.toString()