Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 316078

Summary: Extra ending comment tag in helloworld-feature.xml
Product: [Eclipse Project] PDE Reporter: Kit Lo <kitlo>
Component: UIAssignee: Darin Wright <darin.eclipse>
Status: VERIFIED FIXED QA Contact:
Severity: major    
Priority: P3 CC: ankur_sharma, curtis.windatt.public, daniel_megert, darin.eclipse, d_l_hall, john.arthorne, kim.moir, pwebster, tieTYT
Version: 3.6Flags: daniel_megert: pmc_approved+
darin.eclipse: review+
Target Milestone: 3.6.1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch
none
screen shot of broken cheat sheet
none
updated copyright none

Description Kit Lo CLA 2010-06-07 22:46:14 EDT
File: eclipse\plugins\org.eclipse.pde\cheatsheets\helloworld\helloworld-feature.xml

Extra ending comment tag on line 11 in helloworld-feature.xml causing the xml file to be non well formed and resulted in XML parsing error.
Comment 1 Darin Wright CLA 2010-06-08 08:54:30 EDT
Created attachment 171395 [details]
patch
Comment 2 Darin Wright CLA 2010-06-08 08:55:37 EDT
This problem was introduced updating copyrights. We should probably fix this for 3.6.
Comment 3 Darin Wright CLA 2010-06-08 09:07:30 EDT
Due to this, the "Create a feature" step of the "Creating an Eclipse plug-in" cheat sheet is broken. The user ends up with an error message in the cheat sheet: "Error loading cheat sheet content. Cheat sheet content file ".../helloworld-feature.xml" could not be parsed, error at line 12, column 2".
Comment 4 Darin Wright CLA 2010-06-08 09:08:47 EDT
I guess the question is whether to fix for 3.6 or 3.6.1.
Comment 5 Darin Wright CLA 2010-06-08 11:26:21 EDT
Created attachment 171419 [details]
screen shot of broken cheat sheet
Comment 6 Darin Wright CLA 2010-06-08 11:28:03 EDT
Created attachment 171421 [details]
updated copyright

Updated patch to reflect 2010 copyright year.
Comment 7 Darin Wright CLA 2010-06-08 11:33:13 EDT
John, do you think we should fix this one for 3.6? It's not a critical stop-ship bug, but it is a definite glitch in the UI.
Comment 8 Dani Megert CLA 2010-06-08 11:38:12 EDT
John, when I talked to Darin my take was: fix if there's a rebuild but don't trigger a rebuild just because of that fix. It's not a must fix for me i.e. I can live with a fix in 3.6.1.
Comment 9 John Arthorne CLA 2010-06-08 14:32:48 EDT
(In reply to comment #8)
> John, when I talked to Darin my take was: fix if there's a rebuild but don't
> trigger a rebuild just because of that fix. It's not a must fix for me i.e. I
> can live with a fix in 3.6.1.

Yes, makes sense to me.

I'm wondering why our CHKPII test didn't catch this - is CHKPII configured to ignore these files? If so can we get them included so we can avoid problems like this in the future?
Comment 10 Kit Lo CLA 2010-06-08 14:39:43 EDT
I'm not sure if CHKPII will check the well-form-ness of the XML files. But, we can always request for an enhancement.
Comment 11 Darin Wright CLA 2010-06-08 15:32:38 EDT
Kim, if there's going to be another build towards 3.6, please let us know, as we will contribute to fix this bug. Adding PMC approval flag.
Comment 12 Darin Wright CLA 2010-06-08 15:36:04 EDT
Note: we won't release to HEAD until we get notification of the build. That way HEAD stays in synch with the build contents.
Comment 13 Dani Megert CLA 2010-06-09 03:40:49 EDT
>CHKPII configured to ignore these files?
I don't think so (at least it's not the case on my machine).
Comment 14 Darin Wright CLA 2010-06-11 13:41:35 EDT
This is a 3.6.1 candidate
Comment 15 Darin Wright CLA 2010-06-25 09:34:58 EDT
Applied and released to 3.6.1. Applied to HEAD.
Comment 16 Darin Wright CLA 2010-08-03 09:12:50 EDT
*** Bug 321420 has been marked as a duplicate of this bug. ***
Comment 17 Dean CLA 2010-08-05 20:02:21 EDT
Question:  Has the fix for this issue been released?  I am run Helios for RCP and I am having this issue while going through the plugin tutorial.  The only helloworld-feature.xml file I can find is in the /configuration/bundles/210/1/.cp/cheatsheets/helloworld.

I tried to copy the fixed copyright lines into the file.  Did not work.

Thank you.
Comment 18 Ankur Sharma CLA 2010-08-05 23:46:47 EDT
The fix is available in builds for 3.7 M1 (I20100804-0100) and 3.6 Maintenance release (M20100728-0800) or later.
Comment 19 Dean CLA 2010-08-10 07:12:42 EDT
(In reply to comment #18)
> The fix is available in builds for 3.7 M1 (I20100804-0100) and 3.6 Maintenance
> release (M20100728-0800) or later.

What is the URL for getting these fixes?
Comment 20 Ankur Sharma CLA 2010-08-10 07:36:02 EDT
http://download.eclipse.org/eclipse/downloads
Comment 21 Curtis Windatt CLA 2010-08-30 14:23:45 EDT
Verified.