Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323910 - [target] Don't fail silently when error in resolving with slicer
Summary: [target] Don't fail silently when error in resolving with slicer
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.7 M2   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 350772 352806
  Show dependency tree
 
Reported: 2010-08-29 10:48 EDT by Gunnar Wagenknecht CLA
Modified: 2011-07-21 17:22 EDT (History)
2 users (show)

See Also:

Attachments
Patch to not swallow slicers issues (8.97 KB, patch)
2010-08-29 10:49 EDT, Gunnar Wagenknecht 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 Gunnar Wagenknecht CLA 2010-08-29 10:48:18 EDT 
I have a target that suddenly started resolving "0" plug-ins for one of the referenced software site. Through a long try and catch session I figured an issue with one of the IUs. However, only until I debugged through the resolution code I hit an issue where an error in the slicer is silently ignored.
Comment 1 Gunnar Wagenknecht CLA 2010-08-29 10:49:58 EDT 
Created attachment 177687 [details]
Patch to not swallow slicers issues

The attached patch simply throws the slicer status. It's reported nicely in the target editor.
Comment 2 Gunnar Wagenknecht CLA 2010-08-29 10:51:18 EDT 
Note, I don't know which formatting setting is causing the additional patch changes. I suspect it's something in the pde.core project setting that causes the additional formattings on save.
Comment 3 Ankur Sharma CLA 2010-08-30 02:01:26 EDT 
Keeping on radar for 3.7. Patch looks fairly simple.
Comment 4 Curtis Windatt CLA 2010-08-30 10:20:13 EDT 
Agreed, I'll look at it for 3.7
Comment 5 Curtis Windatt CLA 2010-09-01 14:34:02 EDT 
Applied the fix (not the comment changes) to HEAD.  Fix makes sense and works for me.