Mailing-List: contact dev-help@myfaces.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "MyFaces Development" <dev@myfaces.apache.org>
Received-SPF: pass (nike.apache.org: domain of darkarena@gmail.com designates
 64.233.166.181 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=from:to:in-reply-to:x-mailer:subject:content-transfer-encoding:references:message-id:content-type:mime-version:date:cc;
        b=hc6TldsiGdi1CG+WdoaBTt3c98GXPgdd52ig7oRigug50HnwFm/BgSKYgXXdmddfyCXekr+pseYARHs0nHBrOGTYg55UnOA39rhfVfKkitekV5Sap7vatIE1GjoRgbOsBoBePnY58h5aOShqOEIcHkICmtS8aofYLKL1hRVM23Y=
From: Scott O'Bryan <darkarena@gmail.com>
To: MyFaces Development <dev@myfaces.apache.org>
In-Reply-To: <48337F29.8040307@apache.org>
Subject: Re: [VOTE] Release of Trinidad 1.2.8
Content-Transfer-Encoding: 7bit
References: <482CE441.6070509@gmail.com>
	 <fa024af00805190835r5ae3e64fm917f888f51d0049@mail.gmail.com>
	 <4831C26D.4020104@gmail.com>
	 <fa024af00805191146s7c541d05x538be96e09085ead@mail.gmail.com>
	 <fa024af00805191205h71381e41m76f69afe47430488@mail.gmail.com>
	 <4831DAFF.1020404@mindspring.com>
 <fa024af00805191317m1aef1974h6f34dc6473c38b47@mail.gmail.com>
 <4831EC12.7080105@apache.org> <483215AD.5010804@gmail.com>
 <48322189.6040208@apache.org> <48337F29.8040307@apache.org>
Message-Id: <2E16D2A7-6729-46E5-BC39-EED15EFD1DE7@gmail.com>
Content-Type: text/plain;
	charset=US-ASCII;
	format=flowed;
	delsp=yes
Mime-Version: 1.0 (iPod Mail 4A102)
Date: Tue, 20 May 2008 20:01:51 -0600
Cc: MyFaces Development <dev@myfaces.apache.org>

Thanks Paul.

On May 20, 2008, at 7:47 PM, Paul Spencer <paulsp@apache.org> wrote:

> In light of the fix for TRINIDAD-1083 and issuance of TRINIDAD-1085,  
> I remove my -1 and change it to a +1.
>
> Paul Spencer
>
> Paul Spencer wrote:
>> Scott,
>> Venkata has created a JIRA, TRINIDAD-1083, and attached a patch.   
>> Please regenerate the artifacts. I should be able to retest, and  
>> alter my vote, tomorrow evening.
>> Thank you,
>> Paul Spencer
>> Scott O'Bryan wrote:
>>> Venkata, we'll need a JIRA issue and a patch if possible.  I can  
>>> apply it asap.
>>>
>>> To the community, I do have a question..   Concerning this vote we  
>>> had 2.5 +1's and 1 -1.  Technically I think that allows us to  
>>> release but I suspect people would want to get this one fixed  
>>> first.  I'll allow people to chime in as needed on both this and  
>>> the 1.0.8 thread if you'd like to stop the release.  Once Venkata  
>>> submits the patch, I can certainly apply it and generate the  
>>> artifacts, but it will delay our release another 72 hours as I  
>>> call another vote.   So let me know what you think and I'll either  
>>> complete the release tomorrow generate some new artifacts and  
>>> start the vote again.
>>>
>>> Current results for 1.2.8:
>>>
>>> +1 Scott O'Bryan, Matthias Wessendorf
>>> +.5 Andrew Robinson
>>> -1 Paul Spencer because the Chart component doesn't work.
>>>
>>> Current results for 1.0.8
>>> +1 Scott O'Bryan, Matthias Wessendorf, Andrew Robinson
>>>
>>> I'll go ahead and keep votes open for one more day to allow people  
>>> to chime in or change their vote.
>>>
>>> Scott
>>>
>>> Paul Spencer wrote:
>>>
>>>> Venkata,
>>>> Thank you for your work on this issue.
>>>>
>>>> Paul Spencer
>>>>
>>>> venkata guddanti wrote:
>>>>
>>>>> I believe yes.
>>>>>
>>>>>
>>>>> On Mon, May 19, 2008 at 1:54 PM, Paul Spencer <paulspencer@mindspring.com 
>>>>> >
>>>>> wrote:
>>>>>
>>>>>> Venkata,
>>>>>> Is this also an issue in 1.0.x?
>>>>>>
>>>>>> Paul Spencer
>>>>>>
>>>>>> venkata guddanti wrote:
>>>>>>
>>>>>>> I think I found out the reason why this is not working. The  
>>>>>>> ApacheChart.js
>>>>>>> is not registered in CoreCommonScriptsResourceLoader. I added  
>>>>>>> the file to
>>>>>>> the debug and normal libraries list and it works. Do we need a  
>>>>>>> JIRA ticket
>>>>>>> for this?
>>>>>>>
>>>>>>> Venkata
>>>>>>>
>>>>>>> On Mon, May 19, 2008 at 12:46 PM, venkata guddanti <
>>>>>>> ven.guddanti@gmail.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>> OK, I looked at the issue. It seems that the scriptlet output  
>>>>>>> is broken
>>>>>>>
>>>>>>>> in
>>>>>>>> the latest trunk and 1.2 trunk. The chart  is rendered using
>>>>>>>> ApacheChart.js
>>>>>>>> on the client. The  Renderer creates a  new  Linbrary  
>>>>>>>> Scriptlet( new
>>>>>>>> LibraryScriptlet("ApacheChart", null)). It the uses
>>>>>>>> chartLib.outputScriptlet
>>>>>>>> to send the library to the browser. This seems to be broken.  
>>>>>>>> Looking at
>>>>>>>> the
>>>>>>>> page source The library written is
>>>>>>>>
>>>>>>>> "/trinidad-demo-context-root/adf/jsLibs/ 
>>>>>>>> DebugApacheChart1_2_8. 
>>>>>>>> js; 
>>>>>>>> jsessionid= 
>>>>>>>> 0bef3a120e9dc7a98a8e960e4626cd56ee02bc717325f73e5bd3a0ae88bfd133" 
>>>>>>>> .
>>>>>>>> However this file seems to be invalid from the browser  
>>>>>>>> viewpoint. I only
>>>>>>>> see
>>>>>>>> "jsLibsDebugs/ApacheChart.js" in the output director.
>>>>>>>>
>>>>>>>> Does anybody know if the resource loader or something has  
>>>>>>>> changed in the
>>>>>>>> latest release to cause this to happen?
>>>>>>>>
>>>>>>>> Venkata
>>>>>>>>
>>>>>>>>
>>>>>>>> On Mon, May 19, 2008 at 12:09 PM, Scott O'Bryan <darkarena@gmail.com 
>>>>>>>> >
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>> Cool, thanks Venkata, I'll wait to hear back as to whether  
>>>>>>>> the issue is
>>>>>>>>
>>>>>>>>> in
>>>>>>>>> the demo or the component before I close the vote.
>>>>>>>>>
>>>>>>>>> Scott
>>>>>>>>>
>>>>>>>>> venkata guddanti wrote:
>>>>>>>>>
>>>>>>>>> I will investigate the chart not working in 1.2.8 today.
>>>>>>>>>
>>>>>>>>>> Venkata
>>>>>>>>>>
>>>>>>>>>> On Thu, May 15, 2008 at 7:32 PM, Scott O'Bryan <darkarena@gmail.com
>>>>>>>>>> <mailto:
>>>>>>>>>> darkarena@gmail.com>> wrote:
>>>>>>>>>>
>>>>>>>>>>  Hi,
>>>>>>>>>>
>>>>>>>>>>  I was running the needed tasks to get the 1.2.8 release of  
>>>>>>>>>> the
>>>>>>>>>>  Apache MyFaces Trinidad out. The artifacts are deployed to  
>>>>>>>>>> my
>>>>>>>>>>  private Apache account ([1]).
>>>>>>>>>>
>>>>>>>>>>  Please take a look at the "1.2.8" artifacts and vote.   
>>>>>>>>>> Please note
>>>>>>>>>>  that this is my first time putting together a release for  
>>>>>>>>>> Trinidad
>>>>>>>>>>  so if you could pay extra attention, I would be much  
>>>>>>>>>> appreciative.
>>>>>>>>>>  ------------------------------------------------
>>>>>>>>>>  [ ] +1 for community members who have reviewed the bits
>>>>>>>>>>  [ ] +0
>>>>>>>>>>  [ ] -1 for fatal flaws that should cause these bits not to  
>>>>>>>>>> be
>>>>>>>>>>  released,
>>>>>>>>>>  and why..............
>>>>>>>>>>  ------------------------------------------------
>>>>>>>>>>
>>>>>>>>>>  Thanks,
>>>>>>>>>>  Scott
>>>>>>>>>>
>>>>>>>>>>  [1] http://people.apache.org/~sobryan/trinidad/1.2.8<http://people.apache.org/%7Esobryan/trinidad/1.2.8 
>>>>>>>>>> >
>>>>>>>>>> <http://people.apache.org/%7Esobryan/trinidad/1.2.8>
>>>>>>>>>>  <http://people.apache.org/%7Esobryan/trinidad/1.2.8>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>
>>>>
>>>
>>>
>>>
>