Bug 107934 - [osgi] legacy R3 bundles with invalid Bundle-Version headers must not fail to install
Summary: [osgi] legacy R3 bundles with invalid Bundle-Version headers must not fail to...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Runtime (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1.1   Edit
Assignee: Thomas Watson CLA Friend
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-24 16:22 EDT by Thomas Watson CLA Friend
Modified: 2005-09-03 10:28 EDT (History)
1 user (show)

See Also:

Attachments
Proposed fix for 3.1.1 (2.38 KB, patch)
2005-08-24 17:39 EDT, Thomas Watson CLA Friend 		no flags Details | Diff
Improved fix (5.76 KB, patch)
2005-08-26 15:12 EDT, Thomas Watson CLA Friend 		no flags Details | Diff
Show Obsolete (1) View All 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 Thomas Watson CLA Friend 2005-08-24 16:22:12 EDT 
In OSGi R3 the Bundle-Version header was not interpreted by the Framework.  R3 
bundles should be able to have any value for Bundle-Version an not fail to 
install.  Currently the framework implementation fails with a 
NumberFormatException.
Comment 1 Thomas Watson CLA Friend 2005-08-24 17:39:12 EDT 
Created attachment 26429 [details]
Proposed fix for 3.1.1
Comment 2 Thomas Watson CLA Friend 2005-08-24 17:39:55 EDT 
Pascal, please review fix for the 3.1.1 maintenance branch.  Thanks.
Comment 3 Pascal Rapicault CLA Friend 2005-08-26 10:01:28 EDT 
The patch caches a bogus value for the Bundle-Version which prevents the user to
get the real value. An idea to get around this is to subclass Version to be able
to store the original information.
Comment 4 Thomas Watson CLA Friend 2005-08-26 15:12:19 EDT 
Created attachment 26550 [details]
Improved fix

This fix will return the correct Bundle-Version header from Bundle.getHeaders.
Comment 5 Pascal Rapicault CLA Friend 2005-09-02 19:06:32 EDT 
Patch reviewd, good to go.
Comment 6 Thomas Watson CLA Friend 2005-09-03 10:28:14 EDT 
Fixed in HEAD and 3.1 maintenance branch.