Bug 214156

Summary:

Quickfix to remove Bundle-Activator if there's no lazy start

Product:

[Eclipse Project] PDE

Reporter:

Chris Aniszczyk <caniszczyk>

Component:

Assignee:

bartosz michalik <bartosz.michalik>

Status:

RESOLVED FIXED

QA Contact:

Severity:

enhancement

Priority:

CC:

bartosz.michalik

Version:

3.4

Keywords:

bugday, contributed

Target Milestone:

3.4 M5

Hardware:

OS:

Windows XP

Whiteboard:

Bug Depends on:

209432

Bug Blocks:

Attachments:

Description	Flags
mylyn/context/zip	none
fix with separate resolution classes	none
patch with single class for both quick fixes	none
pde.patch	none
mylyn/context/zip	none

Description Chris Aniszczyk

2008-01-02 16:20:02 EST

This is an enhancement for part of the work in bug 209432

Comment 1 bartosz michalik

2008-01-02 17:56:02 EST

I'll take care if it's not taken

Comment 2 Chris Aniszczyk

2008-01-02 18:02:49 EST

Here's a context for your Bartosz. Look at RemoveUnknownExecEnvironments as an example.

Thanks, it should be very straightforward.

Comment 3 Chris Aniszczyk

2008-01-02 18:02:54 EST

Created attachment 86038 [details]
mylyn/context/zip

Comment 4 bartosz michalik

2008-01-02 21:19:45 EST

Created attachment 86050 [details]
fix with separate resolution classes

Comment 5 bartosz michalik

2008-01-02 21:44:59 EST

Created attachment 86053 [details]
patch with single class for both quick fixes

Comment 6 Chris Aniszczyk

2008-01-02 22:38:59 EST

Created attachment 86056 [details]
pde.patch

I decided to just use the remove resolution as the disable one was effectively the same thing.

I also got rid of the modelChangedEvent addition, that wasn't necessary. See the updated patch to learn why :)

Comment 7 Chris Aniszczyk

2008-01-02 22:39:01 EST

Created attachment 86058 [details]
mylyn/context/zip

Comment 8 Chris Aniszczyk

2008-01-02 22:39:50 EST

Thanks again Bartosz!