cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nikhil Khandelwal <nikhi...@microsoft.com>
Subject RE: Jira CB-831: File transfer tests crash on Android L
Date Wed, 08 Apr 2015 01:14:57 GMT
Why am I seeing this crash with the latest master on the BuildBot CI?

I understand that we're not bundling OkHttp, but it's being used with the File Transfer plugin
on an emulator for Android 5.0.1 (API 21). 

The crash is happening with the stack at com.android.okio.OkBuffer.write(OkBuffer.java:574)
and not com.squareup.okhttp which Android 3.7.1 platform used to package. 

Does Android ship with a version okhttp? Here's the issue description and a fix for it in
Okio: https://github.com/square/okio/issues/79. It does not look like this fix is part of
Android 5.0.1 - the emulator that was being used to run tests on our CI server.

(I'm quite new to Android development - bear with me if I'm missing something obvious here).

Thanks,
Nikhil


-----Original Message-----
From: Joe Bowser [mailto:bowserj@gmail.com] 
Sent: Tuesday, April 7, 2015 5:29 PM
To: dev@cordova.apache.org
Subject: Re: Jira CB-831: File transfer tests crash on Android L

This is a Won't Fix because we're not bundling OkHttp anymore.  This will be resolved when
we release Cordova-Android 4.0.0

On Tue, Apr 7, 2015 at 5:27 PM Nikhil Khandelwal <nikhilkh@microsoft.com>
wrote:

> The JIRA which was marked as 'Won't fix' is currently blocking Android 
> test runs on http://ci.cordova.io:  https://issues.apache.org/ 
> jira/browse/CB-8431.
>
> It looks like Android L's implementation of OkHttp has a race 
> condition which hits quite consistently with our current file transfer 
> tests. There is a comment by Andrew in the bug that points to using 
> this plugin on Android L: https://github.com/MobileChromeApps/cordova-plugin-okhttp.
>
> Should this plugin be a dependency to the file transfer plugin? Or the 
> code for it be rolled in as part of file transfer plugin itself?
>
> For the short term I'm leaning towards working around this issue, by 
> using an older AVD (instead of Android L) on our CI setup. However, 
> there is a chance developers running Cordova apps on Android L will 
> run into this and would expect some guidance here.
>
> Thanks,
> Nikhil
>
>
>
Mime
View raw message