Comment 1 Olivier Thomann 2006-10-11 11:26:33 EDT

Builder issue.

Comment 2 Philipe Mulet 2006-10-13 02:58:07 EDT

We also have markers issued by the JavaModel, ie. buildpath markers.
All of them should be stamped accordingly.

Comment 3 Maxime Daniel 2006-11-30 08:19:24 EST

I have prepared a patch that sets that attribute to JavaCore.PLUGIN_ID (currently under test, instrumented to catch failed cases).
Frédéric, please let me know if this is the intended value for all our markers, or if you want another one.

Comment 4 Maxime Daniel 2006-11-30 09:21:00 EST

Created attachment 54777 [details]
Fix (using JavaCore.PLUGIN_ID as marker)

Would still need a few tests to check we do not miss that attribute under diverse circumstances.

Comment 5 Maxime Daniel 2006-12-01 04:30:26 EST

Created attachment 54877 [details]
Fix + test cases - still using JavaCore.PLUGIN_ID

Note:
- need to discuss with clients to define what is needed (JavaCore.PLUGIN_ID, something else...);
- need to dig deeper the compiler participant generated problems case.

Comment 6 Maxime Daniel 2006-12-01 04:44:44 EST

Pinged JohnA.

Comment 7 Maxime Daniel 2006-12-05 03:56:45 EST

Feedback from JohnA is that we should use a shorter key if possible, aka "JDT". Discussed with Philippe: we'll check other potential uses (as of today) and go for a JDT key if no indication that something else would be better.
As far as compilation participants are concerned, we'll hand out the responsibility to them, carrying the attribute if they set it on the categorized problems (need to document that).

Comment 8 Maxime Daniel 2006-12-05 11:53:50 EST

Found no reference to IMarker#GENERATED_BY in a full Eclipse source workspace, hence will go for "JDT".

Comment 9 Maxime Daniel 2006-12-07 01:53:57 EST

Created attachment 55194 [details]
Fix + test cases using JavaBuilder#GENERATED_BY

This patch does the following:
- IMarker-s generated by JDT get JavaBuilder#GENERATED_BY, currently valued to JDT, as their IMarker#GENERATED_BY attribute;
- IMarker-s originating from compiler participants' categorized problems which do not have the IMarker#GENERATED_BY extra attribute set do not have their GENERATED_BY attribute set;
- IMarker-s originating from compiler participants' categorized problems which have the IMarker#GENERATED_BY set to a given value get their GENERATED_BY attribute set to the said value.

Comment 10 Maxime Daniel 2006-12-07 01:55:43 EST

Verifier please consider ParticipantBuildTests #1001 and 1002 and ErrorsTests #100 to 104.

Comment 11 Maxime Daniel 2006-12-07 02:30:16 EST

Released for 3.3 M4.

Comment 12 Olivier Thomann 2006-12-11 16:47:26 EST

Verified for 3.3M4 with I20061211-1119.
Tests passed.