Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 342463 - Debugger opens ".java" files instead of ".class" files if target platform is inside the workspace
Summary: Debugger opens ".java" files instead of ".class" files if target platform is ...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.5.1   Edit
Hardware: PC Windows All
: P3 normal (vote)
Target Milestone: 3.4.2+   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 297039
Blocks:
  Show dependency tree
 
Reported: 2011-04-11 12:20 EDT by Michael Rennie CLA
Modified: 2011-05-24 15:03 EDT (History)
4 users (show)

See Also:
curtis.windatt.public: review+


Attachments
back-ported fix (7.08 KB, patch)
2011-04-11 12:20 EDT, Michael Rennie CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2011-04-11 12:20:08 EDT
Created attachment 192943 [details]
back-ported fix

+++ This bug was initially created as a clone of Bug #297039 +++

This would be a very good fix to back-port to 3.4.x as well. It is small and confined and makes the source lookup behavior consistent for products built on 3.4.x.

There are some additional steps to reproduce the problem on 3.4.x:

1. leave the default target platform alone
2. add the '/target/plugins/' folder as a folder entry to your target platform and *do not* remove any of the default entries, otherwise JUnit stops working and you can't run the tests.
3. follow the remainder of the steps from bug 297039

Steps to Reproduce (from bug 297039):
1. Import projects (from https://bugs.eclipse.org/bugs/attachment.cgi?id=153903) into a workspace.
2. Set as Target Platform target as mentioned above.
3. Open Type "foo.Foo". Set a breakpoint at line 5.
4. Select "foo.tests" project and Debug it as JUnit Plugin Test.
5. The debugger stops in "Foo.java" instead of "Foo.class".
Comment 1 Michael Rennie CLA 2011-04-11 14:28:03 EDT
applied patch to R3_4_maintenance, please verify Curtis.
Comment 2 Curtis Windatt CLA 2011-05-24 15:03:34 EDT
Verified