Bug 91772 - Exported plugin dependencies are missing from the classpath
Summary: Exported plugin dependencies are missing from the classpath
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: 3.1 M7   Edit
Assignee: David Audel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 91979 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-04-18 14:33 EDT by Stefan Xenos CLA
Modified: 2005-05-12 14:16 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Xenos CLA 2005-04-18 14:33:48 EDT
N20050418-0010

I was just working on o.e.ui.tests and tried to write some code that used
MessageDialog, but it could not be found. 

The o.e.ui.tests project is no longer able to reference classes from o.e.jface
or o.e.ui.workbench. Note that o.e.ui.tests depends on o.e.ui which exports
o.e.jface and o.e.ui.workbench, so they should be there.

The builder, content assist, and organize imports all cannot see the classes
from this plugin. Expanding the "Plug-in Dependencies" tab for o.e.ui.tests
reveals that o.e.jface and o.e.ui.workbench are both missing from the PDE
classpath container.
Comment 1 Stefan Xenos CLA 2005-04-18 14:38:14 EDT
Correction: the builder seems to be working fine. If I add the import by hand,
the code still compiles and I don't get red X's. It's only the UI (content
assist, organize imports, package explorer, project properties, etc.) that seems
confused.
Comment 2 Jeff McAffer CLA 2005-04-19 13:06:20 EDT
seems like a UI problem...
Comment 3 Wassim Melhem CLA 2005-04-19 13:11:53 EDT
was org.eclipse.ui a project in your workspace?  If so, expanding the 
classpath container for o.e.ui.tests indeed should not show o.e.jface, etc. as 
they are not directly on the classpath.  They are available to your plugin by 
being reexported by the org.eclipse.ui project.

right-click on the Pde container for org.eclipse.ui and select configure..., 
tell me what you see there.
Comment 4 Stefan Xenos CLA 2005-04-19 14:04:33 EDT
When I right-click on org.eclipse.ui/Plug-in Dependencies and go to
configure..., I see jface and workbench.

When I right-click on org.eclipse.ui.tests/Plug-in Dependencies and go to
configure..., I see org.eclipse.ui but not jface or workbench.

o.e.ui was indeed a project in my workspace.
Comment 5 Dejan Glozic CLA 2005-04-19 14:11:45 EDT
I am seeing a similar problem:

org.eclipse.help.base requires org.eclipse.help and re-exports it
org.eclipse.help.ui requires org.eclipse.help.base

I can see classes in API packages like 'ITopic', but not internal classes 
like 'Topic' from org.eclipse.help.
Comment 6 Wassim Melhem CLA 2005-04-19 15:40:13 EDT
Stefan, your classpath is correct.  This might have to do with access 
restrictions that were on for the nightly build, but turned off for today's I-
build.

Dejan, what build are you using?
Comment 7 Dejan Glozic CLA 2005-04-19 15:48:03 EDT
8am integration
Comment 8 Wassim Melhem CLA 2005-04-19 15:50:12 EDT
Dejan, access restrictions are off in the 8 am I-build.  I suggest opening a 
separate bug report against JDT/Core.  

The nightly build may be related to access restrictions, so I would like to 
investigate that case more.
Comment 9 Wassim Melhem CLA 2005-04-19 16:01:26 EDT
actually, I just confirmed that access restrictions were off for the nightly 
build too.

The fact that comment 1 says that everything compiles fine indicates that the 
classpath is complete.  I just verified that it is.

Moving to JDT/Core to investigate the code assist.
Comment 10 Philipe Mulet CLA 2005-04-19 18:31:58 EDT
*** Bug 91979 has been marked as a duplicate of this bug. ***
Comment 11 David Audel CLA 2005-04-20 06:45:14 EDT
The bug was introduced with the fix of bug 79288.
Export must not be check on rawEntry.
Comment 12 David Audel CLA 2005-04-20 11:49:21 EDT
Fix is released and solve the code assist and organize import problem.

Needs to add a regression test.
Comment 13 David Audel CLA 2005-04-21 04:33:40 EDT
Add test CompletionTests2#testBug91772()
Comment 14 Olivier Thomann CLA 2005-05-12 14:16:24 EDT
Verified in I20050510-0010 + JDT/Core v_556