Bug 204924 - Execution Environment select does not make sense for Eclipse 3.0 plug-ins
Summary: Execution Environment select does not make sense for Eclipse 3.0 plug-ins
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M3   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday
Depends on:
Blocks:
 
Reported: 2007-09-28 12:03 EDT by Brian Bauman CLA
Modified: 2007-10-24 11:48 EDT (History)
4 users (show)

See Also:

Attachments
Patch for review (1.20 KB, patch)
2007-09-28 13:36 EDT, Curtis Windatt CLA 		no flags Details | Diff
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 Brian Bauman CLA 2007-09-28 12:03:33 EDT 
We should disable the Execution Environment selection if the user is creating a Eclipse 3.0 plug-in.  This could cause confusion since the EE is not actually stored anywhere.
Comment 1 Chris Aniszczyk CLA 2007-09-28 12:25:24 EDT 
Curtis, are you screwing around :)?
Comment 2 Curtis Windatt CLA 2007-09-28 13:19:29 EDT 
Sorry, not today.  I can try to look at this next week.

Where is this execution environment selection?  In the new plug-in wizard?
Comment 3 Curtis Windatt CLA 2007-09-28 13:22:06 EDT 
Nevermind, answered my own question, should be quick to fix, so maybe I can look at it ;)
Comment 4 Curtis Windatt CLA 2007-09-28 13:36:17 EDT 
Created attachment 79414 [details]
Patch for review

Happy Bug Day!  Hopefully this does what you need.
Comment 5 Brian Bauman CLA 2007-09-28 16:57:25 EDT 
Hey Curtis, this is exactly what we are looking for.

I tweaked it just a little bit to handle the case where the wizard is initialized with the Eclipse version of 3.0.

Thanks for the quick turn around!
Comment 6 Brian Bauman CLA 2007-09-28 17:50:29 EDT 
Fixed in HEAD.  Available in 3.4M3
Comment 7 Brian Bauman CLA 2007-10-24 11:48:19 EDT 
verified on I20071023-0800