Bug 315686 - Standard Facelet Taglibs not being included in build
Summary: Standard Facelet Taglibs not being included in build
Status: RESOLVED FIXED
Alias: None
Product: Java Server Faces
Classification: WebTools
Component: Core (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P1 critical (vote)
Target Milestone: 3.2 RC4   Edit
Assignee: Cameron Bateman CLA
QA Contact:
URL:
Whiteboard: PMC_approved
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-03 17:47 EDT by Cameron Bateman CLA
Modified: 2010-06-04 15:30 EDT (History)
4 users (show)

See Also:
david_williams: pmc_approved+
raghunathan.srinivasan: pmc_approved? (naci.dai)
raghunathan.srinivasan: pmc_approved? (deboer)
neil.hauge: pmc_approved+
kaloyan: pmc_approved+
raghunathan.srinivasan: review+

Attachments
Add the std-libs folder to the build file. (662 bytes, text/plain)
2010-06-03 17:47 EDT, Cameron Bateman CLA 		no flags Details
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 Cameron Bateman CLA 2010-06-03 17:47:42 EDT 
Created attachment 171038 [details]
Add the std-libs folder to the build file.

I neglected to add a new folder to the build.properties when I added the standard libraries.  This is essential to fix for Helios.
Comment 1 Raghunathan Srinivasan CLA 2010-06-03 18:13:21 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. 
This is a stop-ship bug. Support for MyFaces 2.0 library is broken without this fix.
* Is there a work-around? If so, why do you believe the work-around is
insufficient? 
No workaround
* How has the fix been tested? Is there a test case attached to the bugzilla
record? Has a JUnit Test been added? 
Code review and manual testing.
* Give a brief technical overview. Who has reviewed this fix? 
See description
* What is the risk associated with this fix?
none
Comment 2 David Williams CLA 2010-06-04 12:47:58 EDT 
Thanks for catching this.
Comment 3 Cameron Bateman CLA 2010-06-04 15:30:45 EDT 
Patch applied to HEAD (M4).