Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326436 - Add import Package to the Manifest Quickfix, ignores package version
Summary: Add import Package to the Manifest Quickfix, ignores package version
Status: RESOLVED WORKSFORME
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:
Depends on:
Blocks:
 
Reported: 2010-09-28 12:21 EDT by Zina Mostafia CLA
Modified: 2011-02-28 12:52 EST (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 Zina Mostafia CLA 2010-09-28 12:21:21 EDT 
Build Identifier: 3.6

I am using JavaResolutionFactory.createImportPackageProposal to create a quickfix proposal to add an import package to the bundle Manifest.
The proposal I get contains a package name, and a version number.

If I execute the quickfix, only the package name gets added to the Manifest, version number is ignored !!!

Reproducible: Always
Comment 1 Ankur Sharma CLA 2010-09-28 12:28:23 EDT 
The fix might be extension to fix for Bug 321891. Looking into it.
Comment 2 Ankur Sharma CLA 2010-09-29 08:55:46 EDT 
Which build you are using? We fixed similar one for 3.7 M1. (Bug 320173: Version not picked up on quickfix "Add <package> to imported packaged")
Comment 3 Curtis Windatt CLA 2010-09-29 10:21:08 EDT 
(In reply to comment #2)
> Which build you are using? We fixed similar one for 3.7 M1. (Bug 320173:
> Version not picked up on quickfix "Add <package> to imported packaged")

Good catch Ankur.  Zina, is this something that can wait until 3.7?
Comment 4 Ankur Sharma CLA 2010-11-29 05:34:00 EST 
Pinging Zina

Can you confirm if the problem persists post 3.7 M1?
Comment 5 Curtis Windatt CLA 2011-01-31 14:45:20 EST 
Zina, can you try this in 3.7?

Ankur, please try and investigate this for M6.
Comment 6 Curtis Windatt CLA 2011-02-28 12:52:46 EST 
AFAICT this is working properly now.