Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (nike.apache.org: domain of shazron@gmail.com designates
 209.85.216.54 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAE6O_-a5pM-n2mi1MtYsg7=uWL5N80Zzu-2+oMnxOdLz159Bsg@mail.gmail.com>
References: 
 <CADZEuyEq+rK=29PP+KseTYNe33EDkaZhb5EvvQ7-JsjQF+eBcQ@mail.gmail.com>
 <CAE6O_-ZzQKGY0koSCxmiZTG8QoB8aZQ6G6-iBZy0LcS=VKQG8Q@mail.gmail.com>
 <CAE6O_-aPWo3E9G6qYV3uq60AQBrCCmoGgJYsfVMg42WHXEd4+A@mail.gmail.com>
 <CAE6O_-a6by9Zcz8YRRQr1znt1CJ58f99cyQqm==7+vZJ54uQBQ@mail.gmail.com>
 <CADZEuyGiLmMmzxmcT+2ZBSwCM8J6sEP0SjhsCN5FdfOkMrxTaQ@mail.gmail.com>
 <4763A1C0-182F-436E-A7CD-3E4FBA47A5DF@gmail.com>
 <CAE6O_-YX1KH7ut6SCx9FYcFFa8HwFHR_JXA=wxmDgcdOsZS+BA@mail.gmail.com>
 <CADjNb+mZ-_zxyOxZtFUD0ZnJvQuu88BVNZnMq3_hs0-FR2T6mg@mail.gmail.com>
 <CAE6O_-az3N+8pLK7B7rdLi1QJoh9eacPXuAE49rsOAWtwYz+oQ@mail.gmail.com>
 <CAE6O_-ZM4cQV25Z=2_y0LfsyT8wCJ4Mc+oZkAW3AAAvXBv1hjA@mail.gmail.com>
 <CAE6O_-a5pM-n2mi1MtYsg7=uWL5N80Zzu-2+oMnxOdLz159Bsg@mail.gmail.com>
From: Shazron <shazron@gmail.com>
Date: Fri, 14 Dec 2012 11:17:34 -0800
Message-ID: 
 <CAE6O_-Y_2vdAT_Sp+1b6Xo_cEowMMa1qvRZeO9rG-NAytVwUOA@mail.gmail.com>
Subject: Re: mobile-spec failures on iOS 6.0.0
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=bcaec51b18e7abfe2d04d0d4e448

--bcaec51b18e7abfe2d04d0d4e448
Content-Type: text/plain; charset=ISO-8859-1

Instead of doing the tag and test dance again for everybody, I'm going to
leave it as is for iOS rc2 even though it fails 2 File tests in rc2. These
should be addressed in the final 2.3.0. I'm away from Mon-Wed next week so
hopefully someone will address any iOS issues.

I'll send a pull-request to cordova-js for review since it affects all
platforms.


On Thu, Dec 13, 2012 at 2:23 PM, Shazron <shazron@gmail.com> wrote:

> Furthermore, it used to clobber first, then merge:
> https://github.com/apache/cordova-js/commit/99fadd12f74703a58e729664394570956d6ccbe4#lib/scripts/bootstrap.js
>
> Ok, I'd like to change this but this requires everyone to grab a new
> cordova-js and re-test -- or I just tag iOS now with this fail and leave
> the fix for 2.3.0 final. What does everyone think?
>
>
> On Thu, Dec 13, 2012 at 2:20 PM, Shazron <shazron@gmail.com> wrote:
>
>> Found the culprit -- these two lines have to be reversed (which makes
>> sense -- clobber first, then merge, not the other way around):
>>
>> https://github.com/apache/cordova-js/blob/99fadd12f74703a58e729664394570956d6ccbe4/lib/scripts/bootstrap.js#L48-49
>>
>> Unfortunately, this is an all-platform change to cordova-js :/
>>
>>
>> On Thu, Dec 13, 2012 at 2:07 PM, Shazron <shazron@gmail.com> wrote:
>>
>>> Becky and I were debugging this. In lib/ios/platform.js, it seems
>>> FileReader is clobbered in the "clobbers" section but the iOS specific
>>> FileReader is not merged in the "merges" section. Not sure why yet.
>>>
>>>
>>> On Thu, Dec 13, 2012 at 1:22 PM, Becky Gibson <gibson.becky@gmail.com>wrote:
>>>
>>>> So, it seems that the iOS specific version of readAsText is NOT being
>>>> called and the "standard" one is being used.  The iOS version of
>>>> readAsText
>>>> performs the necessary decode.  The iOS version IS in the cordova.js
>>>> file
>>>> but I haven't debugged enough to see why it is not being used.
>>>>
>>>> -becky
>>>>
>>>>
>>>> On Thu, Dec 13, 2012 at 4:08 PM, Shazron <shazron@gmail.com> wrote:
>>>>
>>>> > Thanks Becky -- i'll take a peek as well. Filed:
>>>> > http://issues.cordova.io/2058
>>>> >
>>>> >
>>>> > On Thu, Dec 13, 2012 at 12:18 PM, Becky Gibson <
>>>> gibson.becky@gmail.com
>>>> > >wrote:
>>>> >
>>>> > > So, I thought the encoding was necessary for internationalization
>>>> and any
>>>> > > special characters in the file data (for example a URL)?  Do all of
>>>> the
>>>> > > other file tests still run with this change?  We're there any
>>>> changes to
>>>> > > the file tests themselves?  I try to take a look this afternoon.
>>>> > >
>>>> > > - Becky
>>>> > >
>>>> > > Sent from my iPad
>>>> > >
>>>> > > On Dec 13, 2012, at 2:15 PM, Marcel Kinard <cmarcelk@gmail.com>
>>>> wrote:
>>>> > >
>>>> > > > Yup, I see that file-transfer.jitsu.com is returning a page
>>>> saying:
>>>> > > >
>>>> > > > An error has occurred:
>>>> > > > {"code":"ECONNREFUSED","errno":"ECONNREFUSED","syscall":"connect"}
>>>> > > >
>>>> > > > Something is obviously dead there. So yeah, ignore those test
>>>> failures
>>>> > as
>>>> > > > an infrastructure problem.
>>>> > > >
>>>> > > > Your File API failures look like the ones I was getting. If I
>>>> modify
>>>> > > > readAsText: from
>>>> > > >
>>>> > > > messageAsString:[pNStrBuff
>>>> > > > stringByAddingPercentEscapesUsingEncoding:NSUTF8StringEncoding]
>>>> > > >
>>>> > > > to
>>>> > > >
>>>> > > > messageAsString:[NSString stringWithString:pNStrBuff]
>>>> > > > then those failures disappear. So I'm still puzzled if it is
>>>> really
>>>> > > > supposed to be URL encoding the result message.
>>>> > > >
>>>> > > > BTW, I noticed in your screenshot that your testsuite has 95
>>>> tests.
>>>> > Mine
>>>> > > > has 238. Are we using the same version of mobile-spec?
>>>> > > >
>>>> > > > -- Marcel
>>>> > >
>>>> >
>>>>
>>>
>>>
>>
>

--bcaec51b18e7abfe2d04d0d4e448--