Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 249174 - [ds]New Service Component Wizards corrupts manifest Service-Component header
Summary: [ds]New Service Component Wizards corrupts manifest Service-Component header
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.5   Edit
Hardware: PC Windows Vista
: P3 major (vote)
Target Milestone: 3.5 M3   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 263450 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-09-30 15:14 EDT by Andreas Dolk CLA
Modified: 2009-02-03 10:17 EST (History)
3 users (show)

See Also:

Attachments
Fixed Manifest updating. (2.08 KB, text/plain)
2008-09-30 15:47 EDT, Rafael Oliveira Nóbrega CLA 		caniszczyk: iplog+
Details
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 Andreas Dolk CLA 2008-09-30 15:14:27 EDT 
Steps to reproduce

(1) Create a new plugin project
(2) Create folder 'OSGI-INF'
(3) use New Service Component Wizard to create Service Component in OSGI-INF named 'component1' and with random class. Press Finish.
(4) Verify that manifest file now has a Service-Component header with a path to this component file.
(5) use New Service Component Wizard to create another Service Component in OSGI-INF named 'component2' and with random class. Press Finish.
(6) Verify that the complete Service-Component header has been recreated. It only contains a path to the last created component file.

The wizard should append the new path to an existing header rather than recreating the header.

The value of Service-Component is:
Service-Component ::= path ( ',' path )*
Comment 1 Andreas Dolk CLA 2008-09-30 15:22:57 EDT 
244088 mentions that this problem is known - but can't find the proposed new bug.

Changed Product/Component to settings from bug 244088.

Sorry for this inconvenience. Should have done it upfront.
Comment 2 Chris Aniszczyk CLA 2008-09-30 15:39:51 EDT 
Rafael, can you look into this?
Comment 3 Rafael Oliveira Nóbrega CLA 2008-09-30 15:40:47 EDT 
Sure! :)
Comment 4 Rafael Oliveira Nóbrega CLA 2008-09-30 15:47:03 EDT 
Created attachment 113911 [details]
Fixed Manifest updating.
Comment 5 Chris Aniszczyk CLA 2008-09-30 16:06:41 EDT 
I cleaned up the patch a bit and made it platform  neutral.

Thanks Rafael!
Comment 6 Chris Aniszczyk CLA 2008-09-30 16:07:04 EDT 
done.

> 20090930
Comment 7 Benjamin Cabé CLA 2009-02-03 10:17:08 EST 
*** Bug 263450 has been marked as a duplicate of this bug. ***