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

Bug 313848

Summary: about.html not included in build.properties
Product: [WebTools] Java Server Faces Reporter: Raghunathan Srinivasan <raghunathan.srinivasan>
Component: CoreAssignee: Raghunathan Srinivasan <raghunathan.srinivasan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: david_williams, neil.hauge
Version: 3.2Flags: david_williams: pmc_approved+
raghunathan.srinivasan: pmc_approved? (naci.dai)
raghunathan.srinivasan: pmc_approved? (deboer)
neil.hauge: pmc_approved+
raghunathan.srinivasan: pmc_approved? (kaloyan)
raghunathan.srinivasan: review+
Target Milestone: 3.2 RC3   
Hardware: PC   
OS: Windows XP   
See Also: https://git.eclipse.org/r/114223
https://git.eclipse.org/c/jsf/webtools.jsf.git/commit/?id=a59721ad5cf8068428eaad7ecd83d36155b1d8a9
Whiteboard: PMC_approved
Attachments:
Description Flags
Patch
none
Test Feature license
none
Updated Fix for license in test feature none

Description Raghunathan Srinivasan CLA 2010-05-21 01:29:23 EDT
Created attachment 169456 [details]
Patch

From the build :
# Missing pattern or file: about.html in dir: org.eclipse.jst.jsf.facesconfig.tests_1.1.2.v20100428
# Missing pattern or file: about.html in dir: org.eclipse.jst.jsf.facesconfig.ui.test_1.1.0.v20100428
# Missing pattern or file: about.html in dir: org.eclipse.jst.jsf.validation.el.tests_1.2.0.v20100428
# Missing pattern or file: about.html in dir: 
# Missing pattern or file: about.html in file: org.eclipse.jst.jsf.common.runtime.tests_1.1.0.v20100428.jar
# Missing pattern or file: about.html in file: org.eclipse.jst.jsf.facelet.ui_1.0.0.v20100510.jar
Comment 1 Raghunathan Srinivasan CLA 2010-05-21 01:32:36 EDT
* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug"
(requested by an adopter) please document it as such. 
Mission about.html files in build
* Is there a work-around? If so, why do you believe the work-around is
insufficient? 
No 
* How has the fix been tested? Is there a test case attached to the bugzilla
record? Has a JUnit Test been added? 
Manual
* Give a brief technical overview. Who has reviewed this fix? 
See description
* What is the risk associated with this fix?
none
Comment 2 Raghunathan Srinivasan CLA 2010-05-21 12:08:16 EDT
Created attachment 169518 [details]
Test Feature license
Comment 3 Raghunathan Srinivasan CLA 2010-05-21 12:11:26 EDT
Update license in the JSF Teast feature, though, I couldn't discern the issue with the current license. I copied the license from one of the 'correct' features. Please review to see if I have addressed the issue.
Comment 4 David Williams CLA 2010-05-21 15:15:32 EDT
(In reply to comment #3)
> Update license in the JSF Teast feature, though, I couldn't discern the issue
> with the current license. I copied the license from one of the 'correct'
> features. Please review to see if I have addressed the issue.

I think it is failing because in your feature.xml file, you have %feature.license, instead of %license. 

And, in case you are tempted to deviate from the norm :) it really does have to be %license ... you can not change feature.properties file to match feature.license, the feature.properties must be 'license' as it is now. 

Thank you for attending.
Comment 5 Raghunathan Srinivasan CLA 2010-05-21 15:20:30 EDT
Created attachment 169550 [details]
Updated Fix for license in test feature
Comment 6 Raghunathan Srinivasan CLA 2010-05-25 19:52:50 EDT
Fixed in S-3.2.0RC3-20100525145526
Comment 7 Eclipse Genie CLA 2017-12-18 20:15:42 EST
New Gerrit change created: https://git.eclipse.org/r/114223