Bug 320552 - API for bundle importers and bundle import pages
Summary: API for bundle importers and bundle import pages
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M6   Edit
Assignee: Ankur Sharma CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on: 326926 330490
Blocks:
  Show dependency tree
 
Reported: 2010-07-21 16:20 EDT by Darin Wright CLA
Modified: 2011-03-13 20:08 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Wright CLA 2010-07-21 16:20:18 EDT 
Internal/provisional API and extension points were created in 3.6 to support importing bundles from CVS:

Extension points:

org.eclipse.pde.core.bundleImporters
org.eclipse.pde.ui.bundleImportPages

Interfaces:

org.eclipse.pde.internal.core.importing.provisional.IBundleImporterDelegate
org.eclipse.pde.internal.ui.provisional.IBundeImportWizardPage

In 3.7, we should create an official API for this function to properly support other repository providers.
Comment 1 Darin Wright CLA 2010-09-20 16:52:55 EDT 
I've written a wiki page to describe the current implementation and desired API: http://wiki.eclipse.org/PDE/Importing_SCMURLs
Comment 2 Darin Wright CLA 2010-10-04 11:31:27 EDT 
The API will be provided by Team (bug 326926) and PDE will leverage it.
Comment 3 Curtis Windatt CLA 2010-12-01 15:43:31 EST 
API is now available from Team.
Comment 4 Ankur Sharma CLA 2011-01-20 09:46:04 EST 
We are almost there but its late for the move in M5. We will do it in early M6.
Comment 5 Curtis Windatt CLA 2011-01-20 09:59:42 EST 
What work is remaining?
Comment 6 Curtis Windatt CLA 2011-02-23 17:31:55 EST 
Reviewed the code changes.  Removed an unnecessary comment and fixed a non-logged error.  Testing went well, the UI handles just like the previous one and team integration works perfectly (similar repo matching, passward vault integration, overwriting existing projects).

Marking as FIXED.
Comment 7 Curtis Windatt CLA 2011-03-09 14:07:40 EST 
Verified in I20110307-2110.  There are still a few quirks in the PDE behaviour, but we already have bugs open for them in 3.7.