Re: failing tests / experiment to move to phantomjs for testing cordovajs

2013-12-11 Thread Bryan Bishop
On Thu, Dec 12, 2013 at 12:11 AM, Brian LeRoux  wrote:

> The tests all pass in Chrome and Node but some are failing in Phantom.
>

PhantomJS is using a fairly old version of qtwebkit
(Function.prototype.bind is missing, for example). Is there any reason you
can't use upstream webkit to do testing? For example, webkitgtk+ has pretty
good bindings into any language supported by gobject/gir stuff.

- Bryan
http://heybryan.org/
1 512 203 0507


failing tests / experiment to move to phantomjs for testing cordovajs

2013-12-11 Thread Brian LeRoux
I'm experimenting in a branch with PhantomJS for testing cordova-js.
Rational being that we are currently emulating a browser using JSDom when
our abstraction actually targets browsers. (Note: you can run the Jasmine
tests in the browser too.)

The tests all pass in Chrome and Node but some are failing in Phantom.

base64:: can base64 encode strings correctly: failed
  TypeError: 'undefined' is not a function (evaluating
'arrayBuffer.slice(0,1)') in
file:///Users/brianleroux/Desktop/cordova-js/test/test.base64.js (line 31)
(1)

modulemapper:: should clobber existing properties: failed
  TypeError: setting a property that has only a getter in
file:///Users/brianleroux/Desktop/cordova-js/pkg/cordova.test.js (line
1852) (1)
  TypeError: setting a property that has only a getter in
file:///Users/brianleroux/Desktop/cordova-js/pkg/cordova.test.js (line
1836) (2)

urlutil:: can handle root-relative URLs: failed
  Expected 'file:///' to be 'file:///Users/'. (1)
  Expected 'file:///foo?a#b' to be 'file:///Users/foo?a#b'. (2)
  Expected 'file:///foo/b%20ar' to be 'file:///Users/foo/b%20ar'. (3)

=/

Is this worth the effort or is the existing test solution good enough? Any
ideas why the above would fail in Phantom but not Chrome?

You can see the work in progress here:

https://github.com/brianleroux/cordova-js/tree/phantom


Re: plugins.cordova.io: UX redesign

2013-12-11 Thread Andrew Grieve
Shoot. I was going to go to bed, but now I'm too excited! These look
AWESOME!

Put some nitpicks below, but pretty much I think it's perfect!



On Wed, Dec 11, 2013 at 10:26 PM, Joni Rustulka  wrote:

> Hi all,
>
> I'm Joni. I work at Adobe as a UX designer on the PhoneGap team, and I
> have recently thrown some time at redesigning the
> http://plugins.cordova.io site. I'm looking for feedback prior to
> implementation - your thoughts are appreciated.
>
> The main screens that have been reworked are as follows:
>
>
>   1.  Home (http://cl.ly/image/2n3l1N0Z0g0w)
> The primary goal for the site is to help users find the plugins they need.
> As such, the main focus is on search/findability.
>

Might consider doing away with "Find Plugins" and "Analytics" top-nav links
  - Don't feel strongly about it, but we might be able to get away without
them. e.g. click the banner to get back to front page & have the front page
have a link to analytics on it somewhere (I don't think analytics needs to
be prominently linked to.)


>
>   2.  Search Results (http://cl.ly/image/0c3X3T1E1L0j)
> Upon search for a plugin, the user is returned results. The user can
> filter to include only desired platforms (http://cl.ly/image/3W2f321I211r),
> and the results table can be sorted by number of downloads, or plugin ID.
>

- Love that search results has a filter by platform
   - Would be neat if the search results page showed supported platforms as
well


>
>   3.  Plugin Details (http://cl.ly/image/2n1p2s3E0a2t)
> The details screen intends to provide all of the necessary information on
> a plugin.
> ** From what I understand, one piece of information that may be missing
> from this screen is "Installation Instructions". This being, config.xml
> feature tags. Is this something that you guys plan to automate, or should
> we be telling plugin authors to put this information in their Read Me?
> Please advise.
>
>
- Maybe get rid of the "Back to search results" link?
   - Users can use the browser back button for this
   - The link won't make sense if you went directly to the page (not
through search results)
- Ditch the "Copy to Clipboard" button.
   - AFAIK, we can't implement that


Further, we do intend to make this a responsive site - so it will be easily
> viewed on smaller screen sizes.
>
> Thanks for your input,
> -joni
>
>
> PS - big thanks and kudos to Yohei for putting together the pluggy robot!
>
>
>


Re: plugins.cordova.io: UX redesign

2013-12-11 Thread Carlos Santana
This looks awesome Joni

The plugin robot looks rad!

for #3 details view, now I looked at your mockup it looks we are missing
that info from the plugin

Maybe it will make sense that we take same approach as NPM and use a readme
markdown file per plugin, and render in details view

we could provide a template that people should follow to allow some sort of
consistency for the details view





On Wed, Dec 11, 2013 at 10:26 PM, Joni Rustulka  wrote:

> Hi all,
>
> I'm Joni. I work at Adobe as a UX designer on the PhoneGap team, and I
> have recently thrown some time at redesigning the
> http://plugins.cordova.io site. I'm looking for feedback prior to
> implementation - your thoughts are appreciated.
>
> The main screens that have been reworked are as follows:
>
>
>   1.  Home (http://cl.ly/image/2n3l1N0Z0g0w)
> The primary goal for the site is to help users find the plugins they need.
> As such, the main focus is on search/findability.
>
>   2.  Search Results (http://cl.ly/image/0c3X3T1E1L0j)
> Upon search for a plugin, the user is returned results. The user can
> filter to include only desired platforms (http://cl.ly/image/3W2f321I211r),
> and the results table can be sorted by number of downloads, or plugin ID.
>
>   3.  Plugin Details (http://cl.ly/image/2n1p2s3E0a2t)
> The details screen intends to provide all of the necessary information on
> a plugin.
> ** From what I understand, one piece of information that may be missing
> from this screen is "Installation Instructions". This being, config.xml
> feature tags. Is this something that you guys plan to automate, or should
> we be telling plugin authors to put this information in their Read Me?
> Please advise.
>
> Further, we do intend to make this a responsive site - so it will be
> easily viewed on smaller screen sizes.
>
> Thanks for your input,
> -joni
>
>
> PS - big thanks and kudos to Yohei for putting together the pluggy robot!
>
>
>


-- 
Carlos Santana



plugins.cordova.io: UX redesign

2013-12-11 Thread Joni Rustulka
Hi all,

I'm Joni. I work at Adobe as a UX designer on the PhoneGap team, and I have 
recently thrown some time at redesigning the http://plugins.cordova.io site. 
I'm looking for feedback prior to implementation - your thoughts are 
appreciated.

The main screens that have been reworked are as follows:


  1.  Home (http://cl.ly/image/2n3l1N0Z0g0w)
The primary goal for the site is to help users find the plugins they need. As 
such, the main focus is on search/findability.

  2.  Search Results (http://cl.ly/image/0c3X3T1E1L0j)
Upon search for a plugin, the user is returned results. The user can filter to 
include only desired platforms (http://cl.ly/image/3W2f321I211r), and the 
results table can be sorted by number of downloads, or plugin ID.

  3.  Plugin Details (http://cl.ly/image/2n1p2s3E0a2t)
The details screen intends to provide all of the necessary information on a 
plugin.
** From what I understand, one piece of information that may be missing from 
this screen is "Installation Instructions". This being, config.xml feature 
tags. Is this something that you guys plan to automate, or should we be telling 
plugin authors to put this information in their Read Me? Please advise.

Further, we do intend to make this a responsive site - so it will be easily 
viewed on smaller screen sizes.

Thanks for your input,
-joni


PS - big thanks and kudos to Yohei for putting together the pluggy robot!




Re: 3.3.0 Release

2013-12-11 Thread Carlos Santana
CLI design contract is to just run platforms//bin/create nothing
about running npm install on platform repo files.

If your platform scripts have dependencies platform is responsible to
satisfy its dependencies.

BlackBerry and Android have dependencies on npm modules, both handle
differently

Android include npm dependencies in the repo
https://github.com/apache/cordova-android/tree/master/bin/node_modules

BlackBerry runs npm install to install on demand as part of create script
https://github.com/apache/cordova-blackberry/blob/master/blackberry10/bin/create#L38

hope this clarifies




On Wed, Dec 11, 2013 at 5:49 PM, Steven Gill  wrote:

> The master branch on CLI now depends on the correct plugman. That problem
> should be good.
>
> I will review the two pull requests and merge in if they look good.
>
> I am not sure about the CLI not honoring the dependencies expressed in
> package.json. I don't think the CLI has any logic to do that. I think it
> should be handled in a platform level script. Maybe force npm install
> before the create command runs? Maybe create an init script like
> blackberry10 has and run that while running check reqs? Others might have
> better suggestions for this one.
>
> Other platforms ran into the permission issue you mentioned. I think
> https://issues.apache.org/jira/browse/CB-3812 should have fixed it. Maybe
> others can chime in with some suggestions here too.
>
>
>
>
> On Wed, Dec 11, 2013 at 10:44 AM, David Barth  >wrote:
>
> > Le 11/12/2013 16:39, David Barth a écrit :
> >
> >  Le 10/12/2013 19:37, Steven Gill a écrit :
> >>
> >>> Lets start the final round of tagging today!
> >>> https://issues.apache.org/jira/browse/CB-5538
> >>>
> >> +1, but we're still chasing down some dependency and chmod +x issues on
> >> the Ubuntu part. Will update this thread shortly.
> >>
> > Maxim added a workaround in those pull request to ensure that the
> > dependencies for the ubuntu commands are satisfied:
> >
> > https://github.com/apache/cordova-ubuntu/pull/2
> > https://github.com/apache/cordova-ubuntu/pull/1
> >
> > Somehow, cordova-cli is not honoring the dependencies expressed in the
> > package.json file here:
> >
> > https://github.com/apache/cordova-ubuntu/blob/master/bin/package.json
> >
> > maybe they should move one directory above, but I'm not so keen on
> > changing that this late.
> >
> > Please note also that the rc1 cordova-cli in the NPM registry still
> points
> > to an old plugman version that does not support ubuntu.
> >
> > Let me know if i'm missing something here.
> >
> > I'll send an update when the remaining command permission issue is fixed.
> > https://issues.apache.org/jira/browse/CB-5629
> >
> > David
> >
>



-- 
Carlos Santana



Re: 3.3.0 Release

2013-12-11 Thread Steven Gill
The master branch on CLI now depends on the correct plugman. That problem
should be good.

I will review the two pull requests and merge in if they look good.

I am not sure about the CLI not honoring the dependencies expressed in
package.json. I don't think the CLI has any logic to do that. I think it
should be handled in a platform level script. Maybe force npm install
before the create command runs? Maybe create an init script like
blackberry10 has and run that while running check reqs? Others might have
better suggestions for this one.

Other platforms ran into the permission issue you mentioned. I think
https://issues.apache.org/jira/browse/CB-3812 should have fixed it. Maybe
others can chime in with some suggestions here too.




On Wed, Dec 11, 2013 at 10:44 AM, David Barth wrote:

> Le 11/12/2013 16:39, David Barth a écrit :
>
>  Le 10/12/2013 19:37, Steven Gill a écrit :
>>
>>> Lets start the final round of tagging today!
>>> https://issues.apache.org/jira/browse/CB-5538
>>>
>> +1, but we're still chasing down some dependency and chmod +x issues on
>> the Ubuntu part. Will update this thread shortly.
>>
> Maxim added a workaround in those pull request to ensure that the
> dependencies for the ubuntu commands are satisfied:
>
> https://github.com/apache/cordova-ubuntu/pull/2
> https://github.com/apache/cordova-ubuntu/pull/1
>
> Somehow, cordova-cli is not honoring the dependencies expressed in the
> package.json file here:
>
> https://github.com/apache/cordova-ubuntu/blob/master/bin/package.json
>
> maybe they should move one directory above, but I'm not so keen on
> changing that this late.
>
> Please note also that the rc1 cordova-cli in the NPM registry still points
> to an old plugman version that does not support ubuntu.
>
> Let me know if i'm missing something here.
>
> I'll send an update when the remaining command permission issue is fixed.
> https://issues.apache.org/jira/browse/CB-5629
>
> David
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Brian LeRoux
Awesome! Case closed.


On Thu, Dec 12, 2013 at 8:54 AM, Steven Gill  wrote:

> I have already been running git log, curating and posting to the blog when
> I do releases. :)
>
> Syndication would be nice to setup.
>
>
> On Wed, Dec 11, 2013 at 1:34 PM, Brian LeRoux  wrote:
>
> > Ok, so I feel we have some consensus that:
> >
> > - release notes are sometimes helpful for some types of people
> > - release notes belong on the blog and not in the source tree
> > - we need a champion to run git log, curate, and post said release notes
> to
> > the blog
> >
> > Sound about right?
> >
> > (I feel blog syndication thing is a different issue.)
> >
> >
> > On Thu, Dec 12, 2013 at 5:58 AM, Michal Mocny 
> wrote:
> >
> > > Thanks for chiming in Dan!
> > >
> > >
> > > On Wed, Dec 11, 2013 at 1:48 PM, Dan Moore  wrote:
> > >
> > > > Hi folks,
> > > >
> > > > As a user, having all the release information in one place would be
> > > > fantastic.  I understand your desire to have a rapid release schedule
> > > > (documented, as best as I can tell, here:
> > > >
> > >
> >
> http://phonegap.com/2012/04/12/rolling-releases-how-apache-cordova-becomes-phonegap-and-why/
> > > ).
> > > >
> > > > But for normal app developers, having a clear understanding of what
> > they
> > > > get when upgrading (what features, bug fixes, improvements, etc have
> > > > happened since the last release) is extremely helpful.  Doing this
> will
> > > > probably increase adoption of the latest versions of cordova.
> > > >
> > >
> > > Absolutely Agree.
> > >
> > >
> > > >
> > > > Another option different from having everything on one blog would be
> to
> > > do
> > > > something like what phonegap.com/blog does--a mix of content hosted
> > > there
> > > > and content excerpts pulled in from other blogs.  (I don't know the
> > exact
> > > > mechanism nor how hard it would be to implement, though.)
> > > >
> > >
> > > Actually when we first discussed the cordova blog and how to post what,
> > > this exact suggestion came up.  We all agreed it was a good idea (well,
> > at
> > > least those who decided to voice opinions at the time..).
> > >
> > > At the time we mentioned that "release notes" are clearly the result
> of a
> > > group effort on platforms/plugins, and thus a cordova centric blog post
> > > should go out about it (but still posted from one specific Author with
> > name
> > > attribution).
> > >
> > > For other types of posts that come from community members' personal
> > blogs,
> > > like user guides, or tips and tricks, we should absolutely syndicate.
>  I
> > > don't think we've been doing this, though.  I'm not sure that we even
> > > discussed the way to submit something for syndication.  Maybe we should
> > > start on that, or maybe we should just leave it up to the phonegap
> blog,
> > > I'm not sure.
> > >
> > >
> > > >
> > > > Cheers,
> > > > Dan
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > On Wednesday, December 11, 2013 11:39 AM, Andrew Grieve <
> > > > agri...@chromium.org> wrote:
> > > >
> > > > Joe - would you be willing to write the blog post on Cordova's blog
> > > instead
> > > > of a personal blog? Each cordova blog post does have an author with
> an
> > > > optional link.
> > > >
> > > > I think having things on Cordova's blog rather than personal /
> > downstream
> > > > ones makes things more trusted & discoverable.
> > > >
> > > >
> > > >
> > > > On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser 
> wrote:
> > > >
> > > > > On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve <
> > agri...@chromium.org>
> > > > > wrote:
> > > > > > Yep, my main concern is communicating what's changed to our users
> > for
> > > > > > releases. Whether this file actually exists, or when it's
> updated,
> > I
> > > > care
> > > > > > less about.
> > > > > >
> > > > > > Joe - if you don't think a single blog post is a good way to
> > > > > communicating
> > > > > > this, what's a good alternative? Should we have each platform
> > write a
> > > > > blog
> > > > > > post as a part of the release instead of release notes?
> > > > > >
> > > > >
> > > > > Yes, because until recently that's what we did.  Shaz wrote the iOS
> > > > > one, and either Simon or I wrote a blog post about Android.  These
> > > > > would then be syndicated and put on the phonegap.com blog.  We
> used
> > to
> > > > > have a perfectly good solution to this problem which went away
> > roughly
> > > > > around when 3.0.0 came out.
> > > >  This mostly went away because of time
> > > > > constraints and the fact that my own blog sucks ass and needs to be
> > > > > migrated to a real server.  I think we need to go back to this.
> > > > >
> > > > > Also, this is a good way for people to get exposure and get their
> > name
> > > > > out there, so there's way more reward for doing this than just
> > writing
> > > > > RELEASENOTES.md which will be buried in the release and may or may
> > not
> > > > > be read.
> > > > >
> > > > > >
> > > > > > On Wed, Dec 11, 2013 at 11:05 AM, Josh S

Re: RELEASENOTES.md and coho

2013-12-11 Thread Steven Gill
I have already been running git log, curating and posting to the blog when
I do releases. :)

Syndication would be nice to setup.


On Wed, Dec 11, 2013 at 1:34 PM, Brian LeRoux  wrote:

> Ok, so I feel we have some consensus that:
>
> - release notes are sometimes helpful for some types of people
> - release notes belong on the blog and not in the source tree
> - we need a champion to run git log, curate, and post said release notes to
> the blog
>
> Sound about right?
>
> (I feel blog syndication thing is a different issue.)
>
>
> On Thu, Dec 12, 2013 at 5:58 AM, Michal Mocny  wrote:
>
> > Thanks for chiming in Dan!
> >
> >
> > On Wed, Dec 11, 2013 at 1:48 PM, Dan Moore  wrote:
> >
> > > Hi folks,
> > >
> > > As a user, having all the release information in one place would be
> > > fantastic.  I understand your desire to have a rapid release schedule
> > > (documented, as best as I can tell, here:
> > >
> >
> http://phonegap.com/2012/04/12/rolling-releases-how-apache-cordova-becomes-phonegap-and-why/
> > ).
> > >
> > > But for normal app developers, having a clear understanding of what
> they
> > > get when upgrading (what features, bug fixes, improvements, etc have
> > > happened since the last release) is extremely helpful.  Doing this will
> > > probably increase adoption of the latest versions of cordova.
> > >
> >
> > Absolutely Agree.
> >
> >
> > >
> > > Another option different from having everything on one blog would be to
> > do
> > > something like what phonegap.com/blog does--a mix of content hosted
> > there
> > > and content excerpts pulled in from other blogs.  (I don't know the
> exact
> > > mechanism nor how hard it would be to implement, though.)
> > >
> >
> > Actually when we first discussed the cordova blog and how to post what,
> > this exact suggestion came up.  We all agreed it was a good idea (well,
> at
> > least those who decided to voice opinions at the time..).
> >
> > At the time we mentioned that "release notes" are clearly the result of a
> > group effort on platforms/plugins, and thus a cordova centric blog post
> > should go out about it (but still posted from one specific Author with
> name
> > attribution).
> >
> > For other types of posts that come from community members' personal
> blogs,
> > like user guides, or tips and tricks, we should absolutely syndicate.  I
> > don't think we've been doing this, though.  I'm not sure that we even
> > discussed the way to submit something for syndication.  Maybe we should
> > start on that, or maybe we should just leave it up to the phonegap blog,
> > I'm not sure.
> >
> >
> > >
> > > Cheers,
> > > Dan
> > >
> > >
> > >
> > >
> > >
> > > On Wednesday, December 11, 2013 11:39 AM, Andrew Grieve <
> > > agri...@chromium.org> wrote:
> > >
> > > Joe - would you be willing to write the blog post on Cordova's blog
> > instead
> > > of a personal blog? Each cordova blog post does have an author with an
> > > optional link.
> > >
> > > I think having things on Cordova's blog rather than personal /
> downstream
> > > ones makes things more trusted & discoverable.
> > >
> > >
> > >
> > > On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:
> > >
> > > > On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve <
> agri...@chromium.org>
> > > > wrote:
> > > > > Yep, my main concern is communicating what's changed to our users
> for
> > > > > releases. Whether this file actually exists, or when it's updated,
> I
> > > care
> > > > > less about.
> > > > >
> > > > > Joe - if you don't think a single blog post is a good way to
> > > > communicating
> > > > > this, what's a good alternative? Should we have each platform
> write a
> > > > blog
> > > > > post as a part of the release instead of release notes?
> > > > >
> > > >
> > > > Yes, because until recently that's what we did.  Shaz wrote the iOS
> > > > one, and either Simon or I wrote a blog post about Android.  These
> > > > would then be syndicated and put on the phonegap.com blog.  We used
> to
> > > > have a perfectly good solution to this problem which went away
> roughly
> > > > around when 3.0.0 came out.
> > >  This mostly went away because of time
> > > > constraints and the fact that my own blog sucks ass and needs to be
> > > > migrated to a real server.  I think we need to go back to this.
> > > >
> > > > Also, this is a good way for people to get exposure and get their
> name
> > > > out there, so there's way more reward for doing this than just
> writing
> > > > RELEASENOTES.md which will be buried in the release and may or may
> not
> > > > be read.
> > > >
> > > > >
> > > > > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref <
> jso...@blackberry.com>
> > > > wrote:
> > > > >
> > > > >> Michal wrote:
> > > > >> > when doing a release, you usually have to make
> > >  a
> > > > >> > mental note of what is worth testing, which usually means going
> > > > through
> > > > >> the
> > > > >> > changelog anyway, which means it isn't really adding serious
> time
> > to
> > > > the
> > > > >> > releas

Re: Plugman update on npm?

2013-12-11 Thread Steven Gill
Done!


On Wed, Dec 11, 2013 at 1:34 PM, Max Woghiren  wrote:

> That would be great.  Thanks!
>
>
> On Wed, Dec 11, 2013 at 1:52 PM, Steven Gill 
> wrote:
>
> > I did do a small update last week with support for fireos & ubuntu so the
> > 3.3.0-rc.1 didn't flail. I see there are some new changes now. I could
> push
> > out version 0.17.0 today.
> >
> >
> > On Wed, Dec 11, 2013 at 10:39 AM, Max Woghiren 
> wrote:
> >
> > > Hi everyone—do we know when we plan to update npm with a newer version
> of
> > > plugman?  My understanding is that we have an "at most" weekly schedule
> > for
> > > this, but have we done it recently?
> > >
> > > Thanks!
> > > -Max
> > >
> >
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
Here's the log:
https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=log;h=97ad4d84ce333f05d6240b75f65da8536bcc3eb1

On Wed, Dec 11, 2013 at 1:22 PM, Axel Nennker  wrote:
> 3.3 will be released tomorrow, right?
>
> -Axel
>
> (Are there more patches put in after rc1?)
>
>
> 2013/12/11 Joe Bowser 
>>
>> It appears that your issue has been fixed by the changes made in
>> CB-5481.  This patch was put in AFTER rc1 was released.
>> https://issues.apache.org/jira/browse/CB-5481
>>
>> On Wed, Dec 11, 2013 at 12:44 PM, Axel Nennker 
>> wrote:
>> > Here is the issue on JIRA:
>> > https://issues.apache.org/jira/browse/CB-5633
>> >
>> > This code works with 3.3-rc1:
>> >
>> >public void loadPlugins() {
>> >   Activity activity = this.ctx.getActivity();
>> >   Log.d(this.getClass().getName(),
>> >
>> > "activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName());
>> >   Intent intent =
>> >
>> > activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName());
>> >   Log.d(this.getClass().getName(),
>> > "intent.getPackage()="+intent.getPackage());
>> >   int id = activity.getResources().getIdentifier("config", "xml",
>> > intent.getPackage());
>> > if (id == 0) {
>> >
>> > -Axel
>> >
>> >
>> >
>> > 2013/12/11 Joe Bowser 
>> >>
>> >> Your bug doesn't exist unless it has a JIRA issue.
>> >>
>> >> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker 
>> >> wrote:
>> >> > Jesse,
>> >> > I did not create an JIRA issue yet because I do not have a small
>> >> > sample
>> >> > project to reproduce this.
>> >> > I am using the command line to create this cordova app.
>> >> > The project has a config.xml and it starts correctly (which is prove
>> >> > that it
>> >> > has the config.xml. otherwise I would see the spinner eternally).
>> >> > The version is 3.3-rc1 (Installed 30 minutes ago).
>> >> >
>> >> > I have an ant script that does the work. I would not 100%-ly rule it
>> >> > out
>> >> > that this build.xml is missing some steps. It creates the projects
>> >> > folders
>> >> > (www, merges, etc) and then calls e.g. cordova platform add android
>> >> > and
>> >> > cordova build. "cordova create hello com.example.hello HelloWorld" is
>> >> > NOT
>> >> > called.
>> >> > I attached my build.xml and the AndroidManifest.xml
>> >> >
>> >> > -Axel
>> >> >
>> >> > config.xml is there where it belongs.
>> >> >
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l
>> >> > platforms/android/res/xml/
>> >> > insgesamt 4
>> >> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >> >
>> >> >
>> >> >
>> >> >
>> >> >
>> >> > 2013/12/11 Jesse 
>> >> >>
>> >> >> Is this a JIRA issue and reproducable steps yet?
>> >> >> Does the project that is failing actually have a config.xml file?
>> >> >> How are you creating the project? Platform script, cordova-cli? What
>> >> >> version of the cli? ...
>> >> >>
>> >> >> 3.3.0 is already tagged and ready to go, so this needs reproducible
>> >> >> steps
>> >> >> to get attention.
>> >> >>
>> >> >>
>> >> >> @purplecabbage
>> >> >> risingj.com
>> >> >>
>> >> >>
>> >> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
>> >> >> wrote:
>> >> >>
>> >> >> > Sorry no. It does not work on 3.3-rc1
>> >> >> > Axel
>> >> >> >
>> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
>> >> >> > 3.3.0-rc.1
>> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >> >> >
>> >> >> > same error:
>> >> >> > D/CordovaActivity( 4871): CordovaActivity.init()
>> >> >> > D/CordovaWebView( 4871): >>>
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> >> > D/PluginManager( 4871): init()
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =
>> >> >> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
>> >> >> > res/xml/config.xml to your project.
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =
>> >> >> > D/CordovaWebView( 4871): >>> loadUrlNow()
>> >> >> > D/ItemImport( 4871): onResume() loadURL started:
>> >> >> > D/CordovaActivity( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > 2013/12/11 Andrew Grieve 
>> >> >> >
>> >> >> > > If it worked in 3.1 and broke in 3.2, then it's very likely
>> >> >> > > CB-5481

Re: Plugman update on npm?

2013-12-11 Thread Max Woghiren
That would be great.  Thanks!


On Wed, Dec 11, 2013 at 1:52 PM, Steven Gill  wrote:

> I did do a small update last week with support for fireos & ubuntu so the
> 3.3.0-rc.1 didn't flail. I see there are some new changes now. I could push
> out version 0.17.0 today.
>
>
> On Wed, Dec 11, 2013 at 10:39 AM, Max Woghiren  wrote:
>
> > Hi everyone—do we know when we plan to update npm with a newer version of
> > plugman?  My understanding is that we have an "at most" weekly schedule
> for
> > this, but have we done it recently?
> >
> > Thanks!
> > -Max
> >
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Brian LeRoux
Ok, so I feel we have some consensus that:

- release notes are sometimes helpful for some types of people
- release notes belong on the blog and not in the source tree
- we need a champion to run git log, curate, and post said release notes to
the blog

Sound about right?

(I feel blog syndication thing is a different issue.)


On Thu, Dec 12, 2013 at 5:58 AM, Michal Mocny  wrote:

> Thanks for chiming in Dan!
>
>
> On Wed, Dec 11, 2013 at 1:48 PM, Dan Moore  wrote:
>
> > Hi folks,
> >
> > As a user, having all the release information in one place would be
> > fantastic.  I understand your desire to have a rapid release schedule
> > (documented, as best as I can tell, here:
> >
> http://phonegap.com/2012/04/12/rolling-releases-how-apache-cordova-becomes-phonegap-and-why/
> ).
> >
> > But for normal app developers, having a clear understanding of what they
> > get when upgrading (what features, bug fixes, improvements, etc have
> > happened since the last release) is extremely helpful.  Doing this will
> > probably increase adoption of the latest versions of cordova.
> >
>
> Absolutely Agree.
>
>
> >
> > Another option different from having everything on one blog would be to
> do
> > something like what phonegap.com/blog does--a mix of content hosted
> there
> > and content excerpts pulled in from other blogs.  (I don't know the exact
> > mechanism nor how hard it would be to implement, though.)
> >
>
> Actually when we first discussed the cordova blog and how to post what,
> this exact suggestion came up.  We all agreed it was a good idea (well, at
> least those who decided to voice opinions at the time..).
>
> At the time we mentioned that "release notes" are clearly the result of a
> group effort on platforms/plugins, and thus a cordova centric blog post
> should go out about it (but still posted from one specific Author with name
> attribution).
>
> For other types of posts that come from community members' personal blogs,
> like user guides, or tips and tricks, we should absolutely syndicate.  I
> don't think we've been doing this, though.  I'm not sure that we even
> discussed the way to submit something for syndication.  Maybe we should
> start on that, or maybe we should just leave it up to the phonegap blog,
> I'm not sure.
>
>
> >
> > Cheers,
> > Dan
> >
> >
> >
> >
> >
> > On Wednesday, December 11, 2013 11:39 AM, Andrew Grieve <
> > agri...@chromium.org> wrote:
> >
> > Joe - would you be willing to write the blog post on Cordova's blog
> instead
> > of a personal blog? Each cordova blog post does have an author with an
> > optional link.
> >
> > I think having things on Cordova's blog rather than personal / downstream
> > ones makes things more trusted & discoverable.
> >
> >
> >
> > On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:
> >
> > > On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve 
> > > wrote:
> > > > Yep, my main concern is communicating what's changed to our users for
> > > > releases. Whether this file actually exists, or when it's updated, I
> > care
> > > > less about.
> > > >
> > > > Joe - if you don't think a single blog post is a good way to
> > > communicating
> > > > this, what's a good alternative? Should we have each platform write a
> > > blog
> > > > post as a part of the release instead of release notes?
> > > >
> > >
> > > Yes, because until recently that's what we did.  Shaz wrote the iOS
> > > one, and either Simon or I wrote a blog post about Android.  These
> > > would then be syndicated and put on the phonegap.com blog.  We used to
> > > have a perfectly good solution to this problem which went away roughly
> > > around when 3.0.0 came out.
> >  This mostly went away because of time
> > > constraints and the fact that my own blog sucks ass and needs to be
> > > migrated to a real server.  I think we need to go back to this.
> > >
> > > Also, this is a good way for people to get exposure and get their name
> > > out there, so there's way more reward for doing this than just writing
> > > RELEASENOTES.md which will be buried in the release and may or may not
> > > be read.
> > >
> > > >
> > > > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref 
> > > wrote:
> > > >
> > > >> Michal wrote:
> > > >> > when doing a release, you usually have to make
> >  a
> > > >> > mental note of what is worth testing, which usually means going
> > > through
> > > >> the
> > > >> > changelog anyway, which means it isn't really adding serious time
> to
> > > the
> > > >> > release process.
> > > >>
> > > >> > However, this shouldn't be codified into our processes,
> > > >> > and should be the responsibility of whoever is doing the blog
> post,
> > > not
> > > >> > whoever is doing the release, and those two aren't always the
> same.
> > > >>
> > > >> +1
> > > >>
> > > >> One problem is that the release blog seems to be pro forma and
> > hurried.
> > > >>
> > > >> I've written
> >  release notes with blog entries. Doing them well is
> > > >> worthwhile.
> > > >>
> > > >> A few things that c

Re: cordova-amazon-fireos mirror on github?

2013-12-11 Thread Brian LeRoux
You certainly can! ;) Commenting on the infra ticket sometimes helps.
Camping in the Apache Infra IRC and directly requesting help also works.


On Thu, Dec 12, 2013 at 6:05 AM, Naik, Archana  wrote:

> Hi, Guys
>
> Could someone please followup on this?
> Thanks
> Archana
>
> On 12/2/13 12:09 PM, "Steven Gill"  wrote:
>
> >https://issues.apache.org/jira/browse/INFRA-7054
> >
> >
> >On Mon, Dec 2, 2013 at 11:54 AM, Naik, Archana  wrote:
> >
> >> Hello,
> >>
> >> Cordova-amazon-fireos doesn't have github mirror. What is the process of
> >> creating one? I need to periodically up level it from cordova-android
> >>and
> >> need to work on its fork.
> >> Please advise.
> >>
> >> Thanks
> >> Archana
> >>
>
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
3.3 will be released tomorrow, right?

-Axel

(Are there more patches put in after rc1?)


2013/12/11 Joe Bowser 

> It appears that your issue has been fixed by the changes made in
> CB-5481.  This patch was put in AFTER rc1 was released.
> https://issues.apache.org/jira/browse/CB-5481
>
> On Wed, Dec 11, 2013 at 12:44 PM, Axel Nennker 
> wrote:
> > Here is the issue on JIRA:
> > https://issues.apache.org/jira/browse/CB-5633
> >
> > This code works with 3.3-rc1:
> >
> >public void loadPlugins() {
> >   Activity activity = this.ctx.getActivity();
> >   Log.d(this.getClass().getName(),
> >
> "activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName());
> >   Intent intent =
> >
> activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName());
> >   Log.d(this.getClass().getName(),
> > "intent.getPackage()="+intent.getPackage());
> >   int id = activity.getResources().getIdentifier("config", "xml",
> > intent.getPackage());
> > if (id == 0) {
> >
> > -Axel
> >
> >
> >
> > 2013/12/11 Joe Bowser 
> >>
> >> Your bug doesn't exist unless it has a JIRA issue.
> >>
> >> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker 
> >> wrote:
> >> > Jesse,
> >> > I did not create an JIRA issue yet because I do not have a small
> sample
> >> > project to reproduce this.
> >> > I am using the command line to create this cordova app.
> >> > The project has a config.xml and it starts correctly (which is prove
> >> > that it
> >> > has the config.xml. otherwise I would see the spinner eternally).
> >> > The version is 3.3-rc1 (Installed 30 minutes ago).
> >> >
> >> > I have an ant script that does the work. I would not 100%-ly rule it
> out
> >> > that this build.xml is missing some steps. It creates the projects
> >> > folders
> >> > (www, merges, etc) and then calls e.g. cordova platform add android
> and
> >> > cordova build. "cordova create hello com.example.hello HelloWorld" is
> >> > NOT
> >> > called.
> >> > I attached my build.xml and the AndroidManifest.xml
> >> >
> >> > -Axel
> >> >
> >> > config.xml is there where it belongs.
> >> >
> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l
> >> > platforms/android/res/xml/
> >> > insgesamt 4
> >> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
> >> >
> >> >
> >> >
> >> >
> >> >
> >> > 2013/12/11 Jesse 
> >> >>
> >> >> Is this a JIRA issue and reproducable steps yet?
> >> >> Does the project that is failing actually have a config.xml file?
> >> >> How are you creating the project? Platform script, cordova-cli? What
> >> >> version of the cli? ...
> >> >>
> >> >> 3.3.0 is already tagged and ready to go, so this needs reproducible
> >> >> steps
> >> >> to get attention.
> >> >>
> >> >>
> >> >> @purplecabbage
> >> >> risingj.com
> >> >>
> >> >>
> >> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
> >> >> wrote:
> >> >>
> >> >> > Sorry no. It does not work on 3.3-rc1
> >> >> > Axel
> >> >> >
> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
> >> >> > 3.3.0-rc.1
> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
> >> >> >
> >> >> > same error:
> >> >> > D/CordovaActivity( 4871): CordovaActivity.init()
> >> >> > D/CordovaWebView( 4871): >>>
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> >> >> > D/PluginManager( 4871): init()
> >> >> > E/PluginManager( 4871):
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> =
> >> >> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
> >> >> > res/xml/config.xml to your project.
> >> >> > E/PluginManager( 4871):
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
> >> >> > E/PluginManager( 4871):
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> =
> >> >> > D/CordovaWebView( 4871): >>> loadUrlNow()
> >> >> > D/ItemImport( 4871): onResume() loadURL started:
> >> >> > D/CordovaActivity( 4871):
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> >> >> > 2013/12/11 Andrew Grieve 
> >> >> >
> >> >> > > If it worked in 3.1 and broke in 3.2, then it's very likely
> >> >> > > CB-5481 and
> >> >> > > is fixed in 3.3
> >> >> > >
> >> >> > >
> >> >> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser 
> >> >> > > wrote:
> >> >> > >
> >> >> > > > No. As far as I'm concerned this problem doesn't exist until it
> >> >> > > > is a
> >> >> > JIRA
> >> >> > > > issue with some sample code that I can use to reproduce the
> >> >> > > > error. I
> >> >> 

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
It appears that your issue has been fixed by the changes made in
CB-5481.  This patch was put in AFTER rc1 was released.
https://issues.apache.org/jira/browse/CB-5481

On Wed, Dec 11, 2013 at 12:44 PM, Axel Nennker  wrote:
> Here is the issue on JIRA:
> https://issues.apache.org/jira/browse/CB-5633
>
> This code works with 3.3-rc1:
>
>public void loadPlugins() {
>   Activity activity = this.ctx.getActivity();
>   Log.d(this.getClass().getName(),
> "activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName());
>   Intent intent =
> activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName());
>   Log.d(this.getClass().getName(),
> "intent.getPackage()="+intent.getPackage());
>   int id = activity.getResources().getIdentifier("config", "xml",
> intent.getPackage());
> if (id == 0) {
>
> -Axel
>
>
>
> 2013/12/11 Joe Bowser 
>>
>> Your bug doesn't exist unless it has a JIRA issue.
>>
>> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker 
>> wrote:
>> > Jesse,
>> > I did not create an JIRA issue yet because I do not have a small sample
>> > project to reproduce this.
>> > I am using the command line to create this cordova app.
>> > The project has a config.xml and it starts correctly (which is prove
>> > that it
>> > has the config.xml. otherwise I would see the spinner eternally).
>> > The version is 3.3-rc1 (Installed 30 minutes ago).
>> >
>> > I have an ant script that does the work. I would not 100%-ly rule it out
>> > that this build.xml is missing some steps. It creates the projects
>> > folders
>> > (www, merges, etc) and then calls e.g. cordova platform add android and
>> > cordova build. "cordova create hello com.example.hello HelloWorld" is
>> > NOT
>> > called.
>> > I attached my build.xml and the AndroidManifest.xml
>> >
>> > -Axel
>> >
>> > config.xml is there where it belongs.
>> >
>> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l
>> > platforms/android/res/xml/
>> > insgesamt 4
>> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
>> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >
>> >
>> >
>> >
>> >
>> > 2013/12/11 Jesse 
>> >>
>> >> Is this a JIRA issue and reproducable steps yet?
>> >> Does the project that is failing actually have a config.xml file?
>> >> How are you creating the project? Platform script, cordova-cli? What
>> >> version of the cli? ...
>> >>
>> >> 3.3.0 is already tagged and ready to go, so this needs reproducible
>> >> steps
>> >> to get attention.
>> >>
>> >>
>> >> @purplecabbage
>> >> risingj.com
>> >>
>> >>
>> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
>> >> wrote:
>> >>
>> >> > Sorry no. It does not work on 3.3-rc1
>> >> > Axel
>> >> >
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
>> >> > 3.3.0-rc.1
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >> >
>> >> > same error:
>> >> > D/CordovaActivity( 4871): CordovaActivity.init()
>> >> > D/CordovaWebView( 4871): >>>
>> >> >
>> >> >
>> >> >
>> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> > D/PluginManager( 4871): init()
>> >> > E/PluginManager( 4871):
>> >> >
>> >> >
>> >> >
>> >> > =
>> >> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
>> >> > res/xml/config.xml to your project.
>> >> > E/PluginManager( 4871):
>> >> >
>> >> >
>> >> >
>> >> > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
>> >> > E/PluginManager( 4871):
>> >> >
>> >> >
>> >> >
>> >> > =
>> >> > D/CordovaWebView( 4871): >>> loadUrlNow()
>> >> > D/ItemImport( 4871): onResume() loadURL started:
>> >> > D/CordovaActivity( 4871):
>> >> >
>> >> >
>> >> >
>> >> > onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> >
>> >> >
>> >> >
>> >> >
>> >> > 2013/12/11 Andrew Grieve 
>> >> >
>> >> > > If it worked in 3.1 and broke in 3.2, then it's very likely
>> >> > > CB-5481 and
>> >> > > is fixed in 3.3
>> >> > >
>> >> > >
>> >> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser 
>> >> > > wrote:
>> >> > >
>> >> > > > No. As far as I'm concerned this problem doesn't exist until it
>> >> > > > is a
>> >> > JIRA
>> >> > > > issue with some sample code that I can use to reproduce the
>> >> > > > error. I
>> >> > > doubt
>> >> > > > that this was a regression since we never explicitly supported
>> >> > > > your
>> >> > > > use
>> >> > > > case to begin with.
>> >> > > >
>> >> > > > Of course, patches are always welcome.
>> >> > > >  On 11 Dec 2013 07:58, "Axel Nennker" 
>> >> > > > wrote:
>> >> > > >
>> >> > > > > No. I intented this to be on the list. Gmail UI irks.
>> >> > > > > Could this be fixed in 3.3?
>> >> > > > > I

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
Here is the issue on JIRA:
https://issues.apache.org/jira/browse/CB-5633

This code works with 3.3-rc1:

   public void loadPlugins() {
  Activity activity = this.ctx.getActivity();
  Log.d(this.getClass().getName(),
"activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName());
  Intent intent =
activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName());
  Log.d(this.getClass().getName(),
"intent.getPackage()="+intent.getPackage());
  int id = activity.getResources().getIdentifier("config", "xml",
intent.getPackage());
if (id == 0) {

-Axel



2013/12/11 Joe Bowser 

> Your bug doesn't exist unless it has a JIRA issue.
>
> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker 
> wrote:
> > Jesse,
> > I did not create an JIRA issue yet because I do not have a small sample
> > project to reproduce this.
> > I am using the command line to create this cordova app.
> > The project has a config.xml and it starts correctly (which is prove
> that it
> > has the config.xml. otherwise I would see the spinner eternally).
> > The version is 3.3-rc1 (Installed 30 minutes ago).
> >
> > I have an ant script that does the work. I would not 100%-ly rule it out
> > that this build.xml is missing some steps. It creates the projects
> folders
> > (www, merges, etc) and then calls e.g. cordova platform add android and
> > cordova build. "cordova create hello com.example.hello HelloWorld" is NOT
> > called.
> > I attached my build.xml and the AndroidManifest.xml
> >
> > -Axel
> >
> > config.xml is there where it belongs.
> >
> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l
> platforms/android/res/xml/
> > insgesamt 4
> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
> > ignisvulpis@ubuntu:/host/20131210/phonegap$
> >
> >
> >
> >
> >
> > 2013/12/11 Jesse 
> >>
> >> Is this a JIRA issue and reproducable steps yet?
> >> Does the project that is failing actually have a config.xml file?
> >> How are you creating the project? Platform script, cordova-cli? What
> >> version of the cli? ...
> >>
> >> 3.3.0 is already tagged and ready to go, so this needs reproducible
> steps
> >> to get attention.
> >>
> >>
> >> @purplecabbage
> >> risingj.com
> >>
> >>
> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
> >> wrote:
> >>
> >> > Sorry no. It does not work on 3.3-rc1
> >> > Axel
> >> >
> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
> >> > 3.3.0-rc.1
> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
> >> >
> >> > same error:
> >> > D/CordovaActivity( 4871): CordovaActivity.init()
> >> > D/CordovaWebView( 4871): >>>
> >> >
> >> >
> >> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> >> > D/PluginManager( 4871): init()
> >> > E/PluginManager( 4871):
> >> >
> >> >
> >> >
> =
> >> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
> >> > res/xml/config.xml to your project.
> >> > E/PluginManager( 4871):
> >> >
> >> >
> >> >
> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
> >> > E/PluginManager( 4871):
> >> >
> >> >
> >> >
> =
> >> > D/CordovaWebView( 4871): >>> loadUrlNow()
> >> > D/ItemImport( 4871): onResume() loadURL started:
> >> > D/CordovaActivity( 4871):
> >> >
> >> >
> >> >
> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> >> >
> >> >
> >> >
> >> >
> >> > 2013/12/11 Andrew Grieve 
> >> >
> >> > > If it worked in 3.1 and broke in 3.2, then it's very likely
> >> > > CB-5481 and
> >> > > is fixed in 3.3
> >> > >
> >> > >
> >> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser 
> >> > > wrote:
> >> > >
> >> > > > No. As far as I'm concerned this problem doesn't exist until it
> is a
> >> > JIRA
> >> > > > issue with some sample code that I can use to reproduce the
> error. I
> >> > > doubt
> >> > > > that this was a regression since we never explicitly supported
> your
> >> > > > use
> >> > > > case to begin with.
> >> > > >
> >> > > > Of course, patches are always welcome.
> >> > > >  On 11 Dec 2013 07:58, "Axel Nennker" 
> wrote:
> >> > > >
> >> > > > > No. I intented this to be on the list. Gmail UI irks.
> >> > > > > Could this be fixed in 3.3?
> >> > > > > I know it is late but...
> >> > > > >
> >> > > > > -Axel
> >> > > > >
> >> > > > >
> >> > > > > 2013/12/11 Joe Bowser 
> >> > > > >
> >> > > > > > Did you mean to to send this to me personally?
> >> > > > > >
> >> > > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
> >> > ignisvul...@gmail.com
> >> > > >
> >> > > > > > wrote:
> >> > > > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> >> > > > > > > The config.xml is where it belongs but 

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
Your bug doesn't exist unless it has a JIRA issue.

On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker  wrote:
> Jesse,
> I did not create an JIRA issue yet because I do not have a small sample
> project to reproduce this.
> I am using the command line to create this cordova app.
> The project has a config.xml and it starts correctly (which is prove that it
> has the config.xml. otherwise I would see the spinner eternally).
> The version is 3.3-rc1 (Installed 30 minutes ago).
>
> I have an ant script that does the work. I would not 100%-ly rule it out
> that this build.xml is missing some steps. It creates the projects folders
> (www, merges, etc) and then calls e.g. cordova platform add android and
> cordova build. "cordova create hello com.example.hello HelloWorld" is NOT
> called.
> I attached my build.xml and the AndroidManifest.xml
>
> -Axel
>
> config.xml is there where it belongs.
>
> ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l platforms/android/res/xml/
> insgesamt 4
> -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
> ignisvulpis@ubuntu:/host/20131210/phonegap$
>
>
>
>
>
> 2013/12/11 Jesse 
>>
>> Is this a JIRA issue and reproducable steps yet?
>> Does the project that is failing actually have a config.xml file?
>> How are you creating the project? Platform script, cordova-cli? What
>> version of the cli? ...
>>
>> 3.3.0 is already tagged and ready to go, so this needs reproducible steps
>> to get attention.
>>
>>
>> @purplecabbage
>> risingj.com
>>
>>
>> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
>> wrote:
>>
>> > Sorry no. It does not work on 3.3-rc1
>> > Axel
>> >
>> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
>> > 3.3.0-rc.1
>> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >
>> > same error:
>> > D/CordovaActivity( 4871): CordovaActivity.init()
>> > D/CordovaWebView( 4871): >>>
>> >
>> >
>> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> > D/PluginManager( 4871): init()
>> > E/PluginManager( 4871):
>> >
>> >
>> > =
>> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
>> > res/xml/config.xml to your project.
>> > E/PluginManager( 4871):
>> >
>> >
>> > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
>> > E/PluginManager( 4871):
>> >
>> >
>> > =
>> > D/CordovaWebView( 4871): >>> loadUrlNow()
>> > D/ItemImport( 4871): onResume() loadURL started:
>> > D/CordovaActivity( 4871):
>> >
>> >
>> > onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >
>> >
>> >
>> >
>> > 2013/12/11 Andrew Grieve 
>> >
>> > > If it worked in 3.1 and broke in 3.2, then it's very likely
>> > > CB-5481 and
>> > > is fixed in 3.3
>> > >
>> > >
>> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser 
>> > > wrote:
>> > >
>> > > > No. As far as I'm concerned this problem doesn't exist until it is a
>> > JIRA
>> > > > issue with some sample code that I can use to reproduce the error. I
>> > > doubt
>> > > > that this was a regression since we never explicitly supported your
>> > > > use
>> > > > case to begin with.
>> > > >
>> > > > Of course, patches are always welcome.
>> > > >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
>> > > >
>> > > > > No. I intented this to be on the list. Gmail UI irks.
>> > > > > Could this be fixed in 3.3?
>> > > > > I know it is late but...
>> > > > >
>> > > > > -Axel
>> > > > >
>> > > > >
>> > > > > 2013/12/11 Joe Bowser 
>> > > > >
>> > > > > > Did you mean to to send this to me personally?
>> > > > > >
>> > > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
>> > ignisvul...@gmail.com
>> > > >
>> > > > > > wrote:
>> > > > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
>> > > > > > > The config.xml is where it belongs but PluginManager does not
>> > find
>> > > it
>> > > > > in
>> > > > > > a
>> > > > > > > certain (legal) setting.
>> > > > > > > When the app is started "normally" PluginManager finds
>> > > > > > > config.xml
>> > > > > (which
>> > > > > > > proves that is at the correct location)
>> > > > > > > When the app is started because an IntentFilter triggers then
>> > > > > > > PluginInManager does not find it.
>> > > > > > >
>> > > > > > > The "main" activity is in one class like a.b.c.Main while the
>> > > > > > IntentFilter
>> > > > > > > activity is in another e.g. u.v.w.IF
>> > > > > > > IF starts Main successfully but - my guess is - that
>> > PluginManager
>> > > > > tries
>> > > > > > to
>> > > > > > > find config.xml in package u.v.w
>> > > > > > >
>> > > > > > > The JUNIT/spec test should be updated to catch this situation
>> > > > > > > if
>> > > > > > possible.
>> > > > > > > I guess I could change the package nam

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
Jesse,
I did not create an JIRA issue yet because I do not have a small sample
project to reproduce this.
I am using the command line to create this cordova app.
The project has a config.xml and it starts correctly (which is prove that
it has the config.xml. otherwise I would see the spinner eternally).
The version is 3.3-rc1 (Installed 30 minutes ago).

I have an ant script that does the work. I would not 100%-ly rule it out
that this build.xml is missing some steps. It creates the projects folders
(www, merges, etc) and then calls e.g. cordova platform add android and
cordova build. "cordova create hello com.example.hello HelloWorld" is NOT
called.
I attached my build.xml and the AndroidManifest.xml

-Axel

config.xml is there where it belongs.

ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l platforms/android/res/xml/
insgesamt 4
-rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
ignisvulpis@ubuntu:/host/20131210/phonegap$





2013/12/11 Jesse 

> Is this a JIRA issue and reproducable steps yet?
> Does the project that is failing actually have a config.xml file?
> How are you creating the project? Platform script, cordova-cli? What
> version of the cli? ...
>
> 3.3.0 is already tagged and ready to go, so this needs reproducible steps
> to get attention.
>
>
> @purplecabbage
> risingj.com
>
>
> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker  >wrote:
>
> > Sorry no. It does not work on 3.3-rc1
> > Axel
> >
> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
> > 3.3.0-rc.1
> > ignisvulpis@ubuntu:/host/20131210/phonegap$
> >
> > same error:
> > D/CordovaActivity( 4871): CordovaActivity.init()
> > D/CordovaWebView( 4871): >>>
> >
> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> > D/PluginManager( 4871): init()
> > E/PluginManager( 4871):
> >
> >
> =
> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
> > res/xml/config.xml to your project.
> > E/PluginManager( 4871):
> >
> >
> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
> > E/PluginManager( 4871):
> >
> >
> =
> > D/CordovaWebView( 4871): >>> loadUrlNow()
> > D/ItemImport( 4871): onResume() loadURL started:
> > D/CordovaActivity( 4871):
> >
> >
> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> >
> >
> >
> >
> > 2013/12/11 Andrew Grieve 
> >
> > > If it worked in 3.1 and broke in 3.2, then it's very likely
> > > CB-5481 and
> > > is fixed in 3.3
> > >
> > >
> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser 
> wrote:
> > >
> > > > No. As far as I'm concerned this problem doesn't exist until it is a
> > JIRA
> > > > issue with some sample code that I can use to reproduce the error. I
> > > doubt
> > > > that this was a regression since we never explicitly supported your
> use
> > > > case to begin with.
> > > >
> > > > Of course, patches are always welcome.
> > > >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
> > > >
> > > > > No. I intented this to be on the list. Gmail UI irks.
> > > > > Could this be fixed in 3.3?
> > > > > I know it is late but...
> > > > >
> > > > > -Axel
> > > > >
> > > > >
> > > > > 2013/12/11 Joe Bowser 
> > > > >
> > > > > > Did you mean to to send this to me personally?
> > > > > >
> > > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
> > ignisvul...@gmail.com
> > > >
> > > > > > wrote:
> > > > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > > > > > > The config.xml is where it belongs but PluginManager does not
> > find
> > > it
> > > > > in
> > > > > > a
> > > > > > > certain (legal) setting.
> > > > > > > When the app is started "normally" PluginManager finds
> config.xml
> > > > > (which
> > > > > > > proves that is at the correct location)
> > > > > > > When the app is started because an IntentFilter triggers then
> > > > > > > PluginInManager does not find it.
> > > > > > >
> > > > > > > The "main" activity is in one class like a.b.c.Main while the
> > > > > > IntentFilter
> > > > > > > activity is in another e.g. u.v.w.IF
> > > > > > > IF starts Main successfully but - my guess is - that
> > PluginManager
> > > > > tries
> > > > > > to
> > > > > > > find config.xml in package u.v.w
> > > > > > >
> > > > > > > The JUNIT/spec test should be updated to catch this situation
> if
> > > > > > possible.
> > > > > > > I guess I could change the package names so that IF is in a.b.c
> > too
> > > > but
> > > > > > it
> > > > > > > worked in 3.1 and I see no reason why this is "complex"
> > > > > > > The u.v.w.IF features are implemented in a plugin that
> registers
> > > the
> > > > > > intent
> > > > > > > filter in the app's AndroidManifest. It is developed by my
> > >

Re: review request for cordova-js gruntfile refactor

2013-12-11 Thread Brian LeRoux
Cool that is literally all those 8 commits do. (Rename, separate out into
discreet modules.)

I'll merge.
On Dec 12, 2013 2:04 AM, "Andrew Grieve"  wrote:

> Brian - the 8 commits change is pretty overwhelming to look at all at once.
> Could you split them up into those that you want looked at, and those that
> you don't (e.g. lib->src & splitting gruntfile into separate files don't
> really need to be looked at).
>
>
> On Tue, Dec 10, 2013 at 11:04 PM, Brian LeRoux  wrote:
>
> > https://github.com/apache/cordova-js/pull/61
> >
> > In my quest to add the browser platform I got a little sidetracked
> cleaning
> > up the gruntfile. Please have a look. The _btest fails currently but I
> > think I want to move to phantom anyhow. Otherwise everything is
> identical.
> >
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
OK, so what's the JIRA issue?

On Wed, Dec 11, 2013 at 12:01 PM, Axel Nennker  wrote:
> I added a Log.d (code below) to PluginManager and get this output:
> 12-11 20:47:37.430: D/org.apache.cordova.PluginManager(25673):
> this.ctx.getActivity().getClass().getPackage().getName()=de.telekom.tlabs.wallet.fileimport
>
> This (de.telekom.tlabs.wallet.fileimport) is the package name of the
> activity that was started by Android but this is not the applications main
> activity's package which is "de.telekom.tlabs.wallet.cordova".
> 12-11 20:47:24.325: D/org.apache.cordova.PluginManager(25673):
> this.ctx.getActivity().getClass().getPackage().getName()=de.telekom.tlabs.wallet.cordova
>
> As you can see from the time stamps the second line is output earlier. This
> is the output when the app starts normally. Then I open the Downloads
> Android app and select a downloaded file that triggers "fileimport"'s
> intent filter. Android now starts an activity in the package
> "de.telekom.tlabs.wallet.fileimport" which then (re-)starts the main
> activity in the package "de.telekom.tlabs.wallet.cordova" which leads to
> the first output and the wrong package from beeing used to find config.xml
>
> -axel
>
>/**
>  * Load plugins from res/xml/config.xml
>  */
> public void loadPlugins() {
>   Log.d(this.getClass().getName(),
> "this.ctx.getActivity().getClass().getPackage().getName()="+this.ctx.getActivity().getClass().getPackage().getName());
> int id =
> this.ctx.getActivity().getResources().getIdentifier("config", "xml",
> this.ctx.getActivity().getClass().getPackage().getName());
> if (id == 0) {
>
>
>
> 2013/12/11 Axel Nennker 
>
>> Sorry no. It does not work on 3.3-rc1
>> Axel
>>
>> ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
>> 3.3.0-rc.1
>> ignisvulpis@ubuntu:/host/20131210/phonegap$
>>
>> same error:
>> D/CordovaActivity( 4871): CordovaActivity.init()
>> D/CordovaWebView( 4871): >>>
>> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> D/PluginManager( 4871): init()
>> E/PluginManager( 4871):
>> =
>> E/PluginManager( 4871): ERROR: config.xml is missing.  Add
>> res/xml/config.xml to your project.
>> E/PluginManager( 4871):
>> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
>> E/PluginManager( 4871):
>> =
>> D/CordovaWebView( 4871): >>> loadUrlNow()
>> D/ItemImport( 4871): onResume() loadURL started:
>> D/CordovaActivity( 4871):
>> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>>
>>
>>
>>
>> 2013/12/11 Andrew Grieve 
>>
>>> If it worked in 3.1 and broke in 3.2, then it's very likely
>>> CB-5481 and
>>> is fixed in 3.3
>>>
>>>
>>> On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser  wrote:
>>>
>>> > No. As far as I'm concerned this problem doesn't exist until it is a
>>> JIRA
>>> > issue with some sample code that I can use to reproduce the error. I
>>> doubt
>>> > that this was a regression since we never explicitly supported your use
>>> > case to begin with.
>>> >
>>> > Of course, patches are always welcome.
>>> >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
>>> >
>>> > > No. I intented this to be on the list. Gmail UI irks.
>>> > > Could this be fixed in 3.3?
>>> > > I know it is late but...
>>> > >
>>> > > -Axel
>>> > >
>>> > >
>>> > > 2013/12/11 Joe Bowser 
>>> > >
>>> > > > Did you mean to to send this to me personally?
>>> > > >
>>> > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
>>> ignisvul...@gmail.com>
>>> > > > wrote:
>>> > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
>>> > > > > The config.xml is where it belongs but PluginManager does not
>>> find it
>>> > > in
>>> > > > a
>>> > > > > certain (legal) setting.
>>> > > > > When the app is started "normally" PluginManager finds config.xml
>>> > > (which
>>> > > > > proves that is at the correct location)
>>> > > > > When the app is started because an IntentFilter triggers then
>>> > > > > PluginInManager does not find it.
>>> > > > >
>>> > > > > The "main" activity is in one class like a.b.c.Main while the
>>> > > > IntentFilter
>>> > > > > activity is in another e.g. u.v.w.IF
>>> > > > > IF starts Main successfully but - my guess is - that PluginManager
>>> > > tries
>>> > > > to
>>> > > > > find config.xml in package u.v.w
>>> > > > >
>>> > > > > The JUNIT/spec test should be updated to catch this situation if
>>> > > > possible.
>>> > > > > I guess I could change the package names so that IF is in a.b.c
>>> too
>>> > but
>>> > > > it
>>> > > > > worked in 3.1 and I see no reason why this is "complex"
>>> > > > > The u.v.w.IF features are implemented in a plugin that registers
>>> t

Debugging on Android 4.4 with multiple users

2013-12-11 Thread David Kemp
Just a cautionary note for anyone using multiple accounts on an Android
device.

If you have a secondary user set up that has the ability to install apps,
you can deploy apps to either the primary or secondary user (whichever is
logged in). The primary user works as expected, but when deploying to the
secondary user adb does not correctly start the app. In my testing it
usually brings the battery monitor to the front instead of the app.

This has nothing to do with Cordova, I have replicated it with the sample
accelerometerplay android application.
The step that starts the app in the android tools is:

adb -s  shell am start -W -a android.intent.action.MAIN -n


It works OK if you remove the -W (but returns immediately instead of
waiting)

Reported as a bug in adb.


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
I added a Log.d (code below) to PluginManager and get this output:
12-11 20:47:37.430: D/org.apache.cordova.PluginManager(25673):
this.ctx.getActivity().getClass().getPackage().getName()=de.telekom.tlabs.wallet.fileimport

This (de.telekom.tlabs.wallet.fileimport) is the package name of the
activity that was started by Android but this is not the applications main
activity's package which is "de.telekom.tlabs.wallet.cordova".
12-11 20:47:24.325: D/org.apache.cordova.PluginManager(25673):
this.ctx.getActivity().getClass().getPackage().getName()=de.telekom.tlabs.wallet.cordova

As you can see from the time stamps the second line is output earlier. This
is the output when the app starts normally. Then I open the Downloads
Android app and select a downloaded file that triggers "fileimport"'s
intent filter. Android now starts an activity in the package
"de.telekom.tlabs.wallet.fileimport" which then (re-)starts the main
activity in the package "de.telekom.tlabs.wallet.cordova" which leads to
the first output and the wrong package from beeing used to find config.xml

-axel

   /**
 * Load plugins from res/xml/config.xml
 */
public void loadPlugins() {
  Log.d(this.getClass().getName(),
"this.ctx.getActivity().getClass().getPackage().getName()="+this.ctx.getActivity().getClass().getPackage().getName());
int id =
this.ctx.getActivity().getResources().getIdentifier("config", "xml",
this.ctx.getActivity().getClass().getPackage().getName());
if (id == 0) {



2013/12/11 Axel Nennker 

> Sorry no. It does not work on 3.3-rc1
> Axel
>
> ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
> 3.3.0-rc.1
> ignisvulpis@ubuntu:/host/20131210/phonegap$
>
> same error:
> D/CordovaActivity( 4871): CordovaActivity.init()
> D/CordovaWebView( 4871): >>>
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> D/PluginManager( 4871): init()
> E/PluginManager( 4871):
> =
> E/PluginManager( 4871): ERROR: config.xml is missing.  Add
> res/xml/config.xml to your project.
> E/PluginManager( 4871):
> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
> E/PluginManager( 4871):
> =
> D/CordovaWebView( 4871): >>> loadUrlNow()
> D/ItemImport( 4871): onResume() loadURL started:
> D/CordovaActivity( 4871):
> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>
>
>
>
> 2013/12/11 Andrew Grieve 
>
>> If it worked in 3.1 and broke in 3.2, then it's very likely
>> CB-5481 and
>> is fixed in 3.3
>>
>>
>> On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser  wrote:
>>
>> > No. As far as I'm concerned this problem doesn't exist until it is a
>> JIRA
>> > issue with some sample code that I can use to reproduce the error. I
>> doubt
>> > that this was a regression since we never explicitly supported your use
>> > case to begin with.
>> >
>> > Of course, patches are always welcome.
>> >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
>> >
>> > > No. I intented this to be on the list. Gmail UI irks.
>> > > Could this be fixed in 3.3?
>> > > I know it is late but...
>> > >
>> > > -Axel
>> > >
>> > >
>> > > 2013/12/11 Joe Bowser 
>> > >
>> > > > Did you mean to to send this to me personally?
>> > > >
>> > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
>> ignisvul...@gmail.com>
>> > > > wrote:
>> > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
>> > > > > The config.xml is where it belongs but PluginManager does not
>> find it
>> > > in
>> > > > a
>> > > > > certain (legal) setting.
>> > > > > When the app is started "normally" PluginManager finds config.xml
>> > > (which
>> > > > > proves that is at the correct location)
>> > > > > When the app is started because an IntentFilter triggers then
>> > > > > PluginInManager does not find it.
>> > > > >
>> > > > > The "main" activity is in one class like a.b.c.Main while the
>> > > > IntentFilter
>> > > > > activity is in another e.g. u.v.w.IF
>> > > > > IF starts Main successfully but - my guess is - that PluginManager
>> > > tries
>> > > > to
>> > > > > find config.xml in package u.v.w
>> > > > >
>> > > > > The JUNIT/spec test should be updated to catch this situation if
>> > > > possible.
>> > > > > I guess I could change the package names so that IF is in a.b.c
>> too
>> > but
>> > > > it
>> > > > > worked in 3.1 and I see no reason why this is "complex"
>> > > > > The u.v.w.IF features are implemented in a plugin that registers
>> the
>> > > > intent
>> > > > > filter in the app's AndroidManifest. It is developed by my
>> colleagues
>> > > who
>> > > > > chose the u.v.w package name. I am using this plugin in my app
>> that
>> > has
>> > > > the
>> > > > > a.b.c pac

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Jesse
Is this a JIRA issue and reproducable steps yet?
Does the project that is failing actually have a config.xml file?
How are you creating the project? Platform script, cordova-cli? What
version of the cli? ...

3.3.0 is already tagged and ready to go, so this needs reproducible steps
to get attention.


@purplecabbage
risingj.com


On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker wrote:

> Sorry no. It does not work on 3.3-rc1
> Axel
>
> ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
> 3.3.0-rc.1
> ignisvulpis@ubuntu:/host/20131210/phonegap$
>
> same error:
> D/CordovaActivity( 4871): CordovaActivity.init()
> D/CordovaWebView( 4871): >>>
>
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
> D/PluginManager( 4871): init()
> E/PluginManager( 4871):
>
> =
> E/PluginManager( 4871): ERROR: config.xml is missing.  Add
> res/xml/config.xml to your project.
> E/PluginManager( 4871):
>
> https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
> E/PluginManager( 4871):
>
> =
> D/CordovaWebView( 4871): >>> loadUrlNow()
> D/ItemImport( 4871): onResume() loadURL started:
> D/CordovaActivity( 4871):
>
> onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>
>
>
>
> 2013/12/11 Andrew Grieve 
>
> > If it worked in 3.1 and broke in 3.2, then it's very likely
> > CB-5481 and
> > is fixed in 3.3
> >
> >
> > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser  wrote:
> >
> > > No. As far as I'm concerned this problem doesn't exist until it is a
> JIRA
> > > issue with some sample code that I can use to reproduce the error. I
> > doubt
> > > that this was a regression since we never explicitly supported your use
> > > case to begin with.
> > >
> > > Of course, patches are always welcome.
> > >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
> > >
> > > > No. I intented this to be on the list. Gmail UI irks.
> > > > Could this be fixed in 3.3?
> > > > I know it is late but...
> > > >
> > > > -Axel
> > > >
> > > >
> > > > 2013/12/11 Joe Bowser 
> > > >
> > > > > Did you mean to to send this to me personally?
> > > > >
> > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker <
> ignisvul...@gmail.com
> > >
> > > > > wrote:
> > > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > > > > > The config.xml is where it belongs but PluginManager does not
> find
> > it
> > > > in
> > > > > a
> > > > > > certain (legal) setting.
> > > > > > When the app is started "normally" PluginManager finds config.xml
> > > > (which
> > > > > > proves that is at the correct location)
> > > > > > When the app is started because an IntentFilter triggers then
> > > > > > PluginInManager does not find it.
> > > > > >
> > > > > > The "main" activity is in one class like a.b.c.Main while the
> > > > > IntentFilter
> > > > > > activity is in another e.g. u.v.w.IF
> > > > > > IF starts Main successfully but - my guess is - that
> PluginManager
> > > > tries
> > > > > to
> > > > > > find config.xml in package u.v.w
> > > > > >
> > > > > > The JUNIT/spec test should be updated to catch this situation if
> > > > > possible.
> > > > > > I guess I could change the package names so that IF is in a.b.c
> too
> > > but
> > > > > it
> > > > > > worked in 3.1 and I see no reason why this is "complex"
> > > > > > The u.v.w.IF features are implemented in a plugin that registers
> > the
> > > > > intent
> > > > > > filter in the app's AndroidManifest. It is developed by my
> > colleagues
> > > > who
> > > > > > chose the u.v.w package name. I am using this plugin in my app
> that
> > > has
> > > > > the
> > > > > > a.b.c package namespace.
> > > > > > I do not agree that this a complex or irregular use of cordova or
> > > > > Intents.
> > > > > >
> > > > > > Please revert the code back to the 3.1 behaviour.
> > > > > >
> > > > > > Axel
> > > > > >
> > > > > >
> > > > > > 2013/12/11 Joe Bowser 
> > > > > >>
> > > > > >> I haven't noticed this during any of our testing, nor have I
> > noticed
> > > > > >> this when hacking with Google Glass, which uses the Google Glass
> > > voice
> > > > > >> trigger intent to launch Cordova, not the LAUNCHER intent like
> the
> > > > > >> launch menu.  Furthermore, our JUnit tests haven't caught the
> > tests
> > > > > >> either, and they depend on at least three plugins to be loaded
> by
> > > > > >> PluginManager from config.xml.  Honestly, this only breaks if
> you
> > > > > >> don't know what you're doing with Intents.
> > > > > >>
> > > > > >> Honestly, It sounds to me like your config.xml isn't in the
> > correct
> > > > > >> place in your project, especially if you're trying to do
> something
> > > > > >> complex with Android a

Re: Transitioning to a better File API implementation

2013-12-11 Thread Tommy Williams
+1
On 12/12/2013 6:26 am, "Ian Clelland"  wrote:

> Yeah, it would definitely require some kind of migration support. Not
> suggesting that this is something that we ever actually do, but we could
> give developers a bit of code that automatically looks in both places, and
> moves files to the new location on open. Or we do it under a flag that is
> off for existing apps (off-on-upgrade) and only enabled-by-default for new
> apps.
>
> I agree with you: this could cause worlds of pain, with developers, and
> worse with users. We shouldn't take any steps in that direction without a
> lot of very careful thought. For now, /Documents, sub-optimal though it may
> be, is where Cordova puts persistent files.
>
> Ian
>
> On Wed, Dec 11, 2013 at 2:18 PM, Tommy Williams 
> wrote:
>
> > Changing where PERSISTENT is located sounds like a very very bad idea.
> >
> > I know that would cause me personally a lot of grief with existing user
> > data.
> > On 11/12/2013 8:58 am, "Ian Clelland"  wrote:
> >
> > > On Tue, Dec 10, 2013 at 4:37 PM, Michael Gauthier <
> m...@silverorange.com
> > > >wrote:
> > >
> > > > On 10/12/13 05:02 PM, Ian Clelland wrote:
> > > >
> > > >> Yep.
> > > >>
> > > >> I think that Library is the more natural place for the HTML
> persistent
> > > >> filesystem, since it's used by an app for whatever persistent
> storage
> > it
> > > >> needs, without exposing all of it's implementation details to the
> > user.
> > > >> (There's lots of room for debate on this, I'm sure)
> > > >>
> > > >> The trouble is that we've historically used /Documents for
> persistent
> > > >> storage, and changing that will break apps.
> > > >>
> > > >>  I'm fine with a BC break, but I don't have to maintain any legacy
> > > > applications. /Library does make more sense as the default for
> > > PERSISTENT.
> > > > The big problem with BC is for installed apps with existing data on
> the
> > > > filesystem, right?
> > >
> > >
> > > Exactly. I'd hate for developers to update their plugins, and push a
> new
> > > version of their app that seems fine -- everything works great when you
> > > start from scratch -- but users with existing data find themselves
> > > completely locked out of it.
> > >
> > >
> > > >
> > > >  One idea is to allow something like requestFilesystem(DOCUMENT), in
> > > >> addition to PERSISTENT and TEMPORARY. Another suggestion has been to
> > add
> > > >> extra arguments -- hints -- such as "{sync: true}", or maybe in this
> > > case
> > > >> "{purpose: documents}" to specify the attributes of the filesystem
> > that
> > > is
> > > >> returned.
> > > >>
> > > > >
> > > > {sync: true} is a bit tricky because /Library/Cache is not synced but
> > > > /Library/Application Data is synced. Having a DOCUMENT type would
> match
> > > > /tmp and /Library as the top-level dirs mapping to file-system
> > constants.
> > > >
> > > > All my feedback is only from the iOS perspective though. Not sure if
> > > these
> > > > ideas make other platforms more or less complicated.
> > > >
> > > > Cheers,
> > > > Mike
> > > >
> > > >
> > > >>
> > > >>
> > > >> On Tue, Dec 10, 2013 at 2:39 PM, Michael Gauthier <
> > > m...@silverorange.com
> > > >> >wrote:
> > > >>
> > > >>  Hmm.. The two directories have different defined roles on iOS and
> > both
> > > >>> are
> > > >>> normal targets for applications to read/write files. Library is for
> > > cache
> > > >>> data or app-specific resources. Documents is for saving/loading
> > actual
> > > >>> things created by users within apps.
> > > >>>
> > > >>> See https://developer.apple.com/library/ios/documentation/
> > > >>>
> > >
> FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/
> > > >>> FileSystemOverview.html#//apple_ref/doc/uid/TP40010672-CH2-SW14
> > > >>>
> > > >>> Supporting both should be a goal.
> > > >>>
> > > >>> Cheers,
> > > >>> Mike
> > > >>>
> > > >>>
> > > >>>
> > > >>> On 2013-12-10 14:39, Ian Clelland wrote:
> > > >>>
> > > >>>  If you could do that before, it was probably a bug :) (You
> shouldn't
> > > be
> > >  able to use getParent on the filesystem root, and the native code
> > was
> > >  supposed to be checking for the correct path prefix before
> allowing
> > >  access
> > >  to any files on the device)
> > > 
> > >  At the moment, it's not possible to do that -- the filesystems
> > should
> > > be
> > >  properly isolated, and only allow access to correctly formed urls,
> > > like
> > >  filesystem://localhost/persistent/some-file-here.txt.
> > > 
> > >  What we *can* do easily, though, is allow a new URL scheme for
> > library
> > >  assets; something like filesystem://localhost/
> > >  library/some-file-here.txt,
> > >  and we can have a filesystem handler for those URLs. That'll work
> if
> > >  your
> > >  use case is just "I need access to files in /Library", rather than
> > "I
> > >  need
> > >  to get to them via string manipulation".
> > > 
> > >  I've a

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
Sorry no. It does not work on 3.3-rc1
Axel

ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
3.3.0-rc.1
ignisvulpis@ubuntu:/host/20131210/phonegap$

same error:
D/CordovaActivity( 4871): CordovaActivity.init()
D/CordovaWebView( 4871): >>>
loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
D/PluginManager( 4871): init()
E/PluginManager( 4871):
=
E/PluginManager( 4871): ERROR: config.xml is missing.  Add
res/xml/config.xml to your project.
E/PluginManager( 4871):
https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
E/PluginManager( 4871):
=
D/CordovaWebView( 4871): >>> loadUrlNow()
D/ItemImport( 4871): onResume() loadURL started:
D/CordovaActivity( 4871):
onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)




2013/12/11 Andrew Grieve 

> If it worked in 3.1 and broke in 3.2, then it's very likely
> CB-5481 and
> is fixed in 3.3
>
>
> On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser  wrote:
>
> > No. As far as I'm concerned this problem doesn't exist until it is a JIRA
> > issue with some sample code that I can use to reproduce the error. I
> doubt
> > that this was a regression since we never explicitly supported your use
> > case to begin with.
> >
> > Of course, patches are always welcome.
> >  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
> >
> > > No. I intented this to be on the list. Gmail UI irks.
> > > Could this be fixed in 3.3?
> > > I know it is late but...
> > >
> > > -Axel
> > >
> > >
> > > 2013/12/11 Joe Bowser 
> > >
> > > > Did you mean to to send this to me personally?
> > > >
> > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker  >
> > > > wrote:
> > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > > > > The config.xml is where it belongs but PluginManager does not find
> it
> > > in
> > > > a
> > > > > certain (legal) setting.
> > > > > When the app is started "normally" PluginManager finds config.xml
> > > (which
> > > > > proves that is at the correct location)
> > > > > When the app is started because an IntentFilter triggers then
> > > > > PluginInManager does not find it.
> > > > >
> > > > > The "main" activity is in one class like a.b.c.Main while the
> > > > IntentFilter
> > > > > activity is in another e.g. u.v.w.IF
> > > > > IF starts Main successfully but - my guess is - that PluginManager
> > > tries
> > > > to
> > > > > find config.xml in package u.v.w
> > > > >
> > > > > The JUNIT/spec test should be updated to catch this situation if
> > > > possible.
> > > > > I guess I could change the package names so that IF is in a.b.c too
> > but
> > > > it
> > > > > worked in 3.1 and I see no reason why this is "complex"
> > > > > The u.v.w.IF features are implemented in a plugin that registers
> the
> > > > intent
> > > > > filter in the app's AndroidManifest. It is developed by my
> colleagues
> > > who
> > > > > chose the u.v.w package name. I am using this plugin in my app that
> > has
> > > > the
> > > > > a.b.c package namespace.
> > > > > I do not agree that this a complex or irregular use of cordova or
> > > > Intents.
> > > > >
> > > > > Please revert the code back to the 3.1 behaviour.
> > > > >
> > > > > Axel
> > > > >
> > > > >
> > > > > 2013/12/11 Joe Bowser 
> > > > >>
> > > > >> I haven't noticed this during any of our testing, nor have I
> noticed
> > > > >> this when hacking with Google Glass, which uses the Google Glass
> > voice
> > > > >> trigger intent to launch Cordova, not the LAUNCHER intent like the
> > > > >> launch menu.  Furthermore, our JUnit tests haven't caught the
> tests
> > > > >> either, and they depend on at least three plugins to be loaded by
> > > > >> PluginManager from config.xml.  Honestly, this only breaks if you
> > > > >> don't know what you're doing with Intents.
> > > > >>
> > > > >> Honestly, It sounds to me like your config.xml isn't in the
> correct
> > > > >> place in your project, especially if you're trying to do something
> > > > >> complex with Android and intents.  I would check that making these
> > > > >> sorts of statements.  If config.xml wasn't being read, it would
> have
> > > > >> been caught right away, since everything is a plugin now.
> > > > >>
> > > > >>
> > > > >>
> > > > >> On Wed, Dec 11, 2013 at 4:33 AM,  
> wrote:
> > > > >> > Hi,
> > > > >> >
> > > > >> > We are seeing some strange effect on Android when we move from
> > > > >> > 3.1.0-0.2.0 to a newer version.
> > > > >> > We tried with 3.2.0-0.4.0 and 3.3-rc1
> > > > >> >
> > > > >> > I think that PluginManager changed and now fails to load
> > config.xml
> > > > >> >
> > > > >> > This happens in my project when the app is opened by the
> operatin

Re: Transitioning to a better File API implementation

2013-12-11 Thread Ian Clelland
Yeah, it would definitely require some kind of migration support. Not
suggesting that this is something that we ever actually do, but we could
give developers a bit of code that automatically looks in both places, and
moves files to the new location on open. Or we do it under a flag that is
off for existing apps (off-on-upgrade) and only enabled-by-default for new
apps.

I agree with you: this could cause worlds of pain, with developers, and
worse with users. We shouldn't take any steps in that direction without a
lot of very careful thought. For now, /Documents, sub-optimal though it may
be, is where Cordova puts persistent files.

Ian

On Wed, Dec 11, 2013 at 2:18 PM, Tommy Williams  wrote:

> Changing where PERSISTENT is located sounds like a very very bad idea.
>
> I know that would cause me personally a lot of grief with existing user
> data.
> On 11/12/2013 8:58 am, "Ian Clelland"  wrote:
>
> > On Tue, Dec 10, 2013 at 4:37 PM, Michael Gauthier  > >wrote:
> >
> > > On 10/12/13 05:02 PM, Ian Clelland wrote:
> > >
> > >> Yep.
> > >>
> > >> I think that Library is the more natural place for the HTML persistent
> > >> filesystem, since it's used by an app for whatever persistent storage
> it
> > >> needs, without exposing all of it's implementation details to the
> user.
> > >> (There's lots of room for debate on this, I'm sure)
> > >>
> > >> The trouble is that we've historically used /Documents for persistent
> > >> storage, and changing that will break apps.
> > >>
> > >>  I'm fine with a BC break, but I don't have to maintain any legacy
> > > applications. /Library does make more sense as the default for
> > PERSISTENT.
> > > The big problem with BC is for installed apps with existing data on the
> > > filesystem, right?
> >
> >
> > Exactly. I'd hate for developers to update their plugins, and push a new
> > version of their app that seems fine -- everything works great when you
> > start from scratch -- but users with existing data find themselves
> > completely locked out of it.
> >
> >
> > >
> > >  One idea is to allow something like requestFilesystem(DOCUMENT), in
> > >> addition to PERSISTENT and TEMPORARY. Another suggestion has been to
> add
> > >> extra arguments -- hints -- such as "{sync: true}", or maybe in this
> > case
> > >> "{purpose: documents}" to specify the attributes of the filesystem
> that
> > is
> > >> returned.
> > >>
> > > >
> > > {sync: true} is a bit tricky because /Library/Cache is not synced but
> > > /Library/Application Data is synced. Having a DOCUMENT type would match
> > > /tmp and /Library as the top-level dirs mapping to file-system
> constants.
> > >
> > > All my feedback is only from the iOS perspective though. Not sure if
> > these
> > > ideas make other platforms more or less complicated.
> > >
> > > Cheers,
> > > Mike
> > >
> > >
> > >>
> > >>
> > >> On Tue, Dec 10, 2013 at 2:39 PM, Michael Gauthier <
> > m...@silverorange.com
> > >> >wrote:
> > >>
> > >>  Hmm.. The two directories have different defined roles on iOS and
> both
> > >>> are
> > >>> normal targets for applications to read/write files. Library is for
> > cache
> > >>> data or app-specific resources. Documents is for saving/loading
> actual
> > >>> things created by users within apps.
> > >>>
> > >>> See https://developer.apple.com/library/ios/documentation/
> > >>>
> > FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/
> > >>> FileSystemOverview.html#//apple_ref/doc/uid/TP40010672-CH2-SW14
> > >>>
> > >>> Supporting both should be a goal.
> > >>>
> > >>> Cheers,
> > >>> Mike
> > >>>
> > >>>
> > >>>
> > >>> On 2013-12-10 14:39, Ian Clelland wrote:
> > >>>
> > >>>  If you could do that before, it was probably a bug :) (You shouldn't
> > be
> >  able to use getParent on the filesystem root, and the native code
> was
> >  supposed to be checking for the correct path prefix before allowing
> >  access
> >  to any files on the device)
> > 
> >  At the moment, it's not possible to do that -- the filesystems
> should
> > be
> >  properly isolated, and only allow access to correctly formed urls,
> > like
> >  filesystem://localhost/persistent/some-file-here.txt.
> > 
> >  What we *can* do easily, though, is allow a new URL scheme for
> library
> >  assets; something like filesystem://localhost/
> >  library/some-file-here.txt,
> >  and we can have a filesystem handler for those URLs. That'll work if
> >  your
> >  use case is just "I need access to files in /Library", rather than
> "I
> >  need
> >  to get to them via string manipulation".
> > 
> >  I've also had some discussions about making /Library the default
> place
> >  for
> >  the persistent filesystem, and leaving /Documents either just for
> > legacy
> >  apps, or making *that* the target of the special URL scheme. That's
> a
> >  proposal for a different day, though. There are some pretty big
> >  backwards-compatibility issues t

Re: [Android] The READ_PHONE_STATE permission

2013-12-11 Thread Tommy Williams
+1 for on by default in media, optional in others and documented.

Media really should pause on an incoming phone call... I can't think of a
use case where it would be desirable for it to keep playing.
On 11/12/2013 1:33 pm, "Andrew Grieve"  wrote:

> I think on-by-default on media, but also documented in the media plugin's
> readme why it has it.
>
>
> On Tue, Dec 10, 2013 at 7:23 PM, Joe Bowser  wrote:
>
> > Hey
> >
> > After working on CB-5504 and moving the Telephony code to App, I think
> > that we should try and figure out how we should use this permission.
> > I personally want to add this permission to the Media plugin since
> > it's the one that actually needs to know the state, but I'm guessing
> > that other plugins would want to know when you're receiving a call so
> > that they can stop what they're doing in the background.
> >
> > Should READ_PHONE_STATE be on by default on the Media plugin? Or
> > should we document this feature somewhere?
> >
> > Joe
> >
>


Re: Transitioning to a better File API implementation

2013-12-11 Thread Tommy Williams
Changing where PERSISTENT is located sounds like a very very bad idea.

I know that would cause me personally a lot of grief with existing user
data.
On 11/12/2013 8:58 am, "Ian Clelland"  wrote:

> On Tue, Dec 10, 2013 at 4:37 PM, Michael Gauthier  >wrote:
>
> > On 10/12/13 05:02 PM, Ian Clelland wrote:
> >
> >> Yep.
> >>
> >> I think that Library is the more natural place for the HTML persistent
> >> filesystem, since it's used by an app for whatever persistent storage it
> >> needs, without exposing all of it's implementation details to the user.
> >> (There's lots of room for debate on this, I'm sure)
> >>
> >> The trouble is that we've historically used /Documents for persistent
> >> storage, and changing that will break apps.
> >>
> >>  I'm fine with a BC break, but I don't have to maintain any legacy
> > applications. /Library does make more sense as the default for
> PERSISTENT.
> > The big problem with BC is for installed apps with existing data on the
> > filesystem, right?
>
>
> Exactly. I'd hate for developers to update their plugins, and push a new
> version of their app that seems fine -- everything works great when you
> start from scratch -- but users with existing data find themselves
> completely locked out of it.
>
>
> >
> >  One idea is to allow something like requestFilesystem(DOCUMENT), in
> >> addition to PERSISTENT and TEMPORARY. Another suggestion has been to add
> >> extra arguments -- hints -- such as "{sync: true}", or maybe in this
> case
> >> "{purpose: documents}" to specify the attributes of the filesystem that
> is
> >> returned.
> >>
> > >
> > {sync: true} is a bit tricky because /Library/Cache is not synced but
> > /Library/Application Data is synced. Having a DOCUMENT type would match
> > /tmp and /Library as the top-level dirs mapping to file-system constants.
> >
> > All my feedback is only from the iOS perspective though. Not sure if
> these
> > ideas make other platforms more or less complicated.
> >
> > Cheers,
> > Mike
> >
> >
> >>
> >>
> >> On Tue, Dec 10, 2013 at 2:39 PM, Michael Gauthier <
> m...@silverorange.com
> >> >wrote:
> >>
> >>  Hmm.. The two directories have different defined roles on iOS and both
> >>> are
> >>> normal targets for applications to read/write files. Library is for
> cache
> >>> data or app-specific resources. Documents is for saving/loading actual
> >>> things created by users within apps.
> >>>
> >>> See https://developer.apple.com/library/ios/documentation/
> >>>
> FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/
> >>> FileSystemOverview.html#//apple_ref/doc/uid/TP40010672-CH2-SW14
> >>>
> >>> Supporting both should be a goal.
> >>>
> >>> Cheers,
> >>> Mike
> >>>
> >>>
> >>>
> >>> On 2013-12-10 14:39, Ian Clelland wrote:
> >>>
> >>>  If you could do that before, it was probably a bug :) (You shouldn't
> be
>  able to use getParent on the filesystem root, and the native code was
>  supposed to be checking for the correct path prefix before allowing
>  access
>  to any files on the device)
> 
>  At the moment, it's not possible to do that -- the filesystems should
> be
>  properly isolated, and only allow access to correctly formed urls,
> like
>  filesystem://localhost/persistent/some-file-here.txt.
> 
>  What we *can* do easily, though, is allow a new URL scheme for library
>  assets; something like filesystem://localhost/
>  library/some-file-here.txt,
>  and we can have a filesystem handler for those URLs. That'll work if
>  your
>  use case is just "I need access to files in /Library", rather than "I
>  need
>  to get to them via string manipulation".
> 
>  I've also had some discussions about making /Library the default place
>  for
>  the persistent filesystem, and leaving /Documents either just for
> legacy
>  apps, or making *that* the target of the special URL scheme. That's a
>  proposal for a different day, though. There are some pretty big
>  backwards-compatibility issues there.
> 
> 
> 
>  On Tue, Dec 10, 2013 at 11:54 AM, Michael Gauthier <
>  m...@silverorange.com
> 
> > wrote:
> >
> 
>    Ian,
> 
> >
> > With the new URLs will it be possible to write to both /Documents and
> > /Library for iOS apps? In the old version, the filesystem root
> resolved
> > as
> > /Documents but it was possible to get to /Library by navigating the
> the
> > parent dir.
> >
> > Cheers,
> > Mike
> >
> >
> >
> > On 2013-11-15 15:19, Ian Clelland wrote:
> >
> >   On Fri, Nov 15, 2013 at 1:40 PM, purplecabbage <
> > purplecabb...@gmail.com
> >
> >>
> >>>  wrote:
> >>
> >>Considering the magnitude of the changes I would have expected
> that
> >> this
> >>
> >>  was just a new file plugin. The previous version was based on a
> spec,
> >>> and
> >>> if we are deviating from it we s

Re: cordova-amazon-fireos mirror on github?

2013-12-11 Thread Naik, Archana
Hi, Guys

Could someone please followup on this?
Thanks
Archana

On 12/2/13 12:09 PM, "Steven Gill"  wrote:

>https://issues.apache.org/jira/browse/INFRA-7054
>
>
>On Mon, Dec 2, 2013 at 11:54 AM, Naik, Archana  wrote:
>
>> Hello,
>>
>> Cordova-amazon-fireos doesn't have github mirror. What is the process of
>> creating one? I need to periodically up level it from cordova-android
>>and
>> need to work on its fork.
>> Please advise.
>>
>> Thanks
>> Archana
>>



Re: RELEASENOTES.md and coho

2013-12-11 Thread Michal Mocny
Thanks for chiming in Dan!


On Wed, Dec 11, 2013 at 1:48 PM, Dan Moore  wrote:

> Hi folks,
>
> As a user, having all the release information in one place would be
> fantastic.  I understand your desire to have a rapid release schedule
> (documented, as best as I can tell, here:
> http://phonegap.com/2012/04/12/rolling-releases-how-apache-cordova-becomes-phonegap-and-why/).
>
> But for normal app developers, having a clear understanding of what they
> get when upgrading (what features, bug fixes, improvements, etc have
> happened since the last release) is extremely helpful.  Doing this will
> probably increase adoption of the latest versions of cordova.
>

Absolutely Agree.


>
> Another option different from having everything on one blog would be to do
> something like what phonegap.com/blog does--a mix of content hosted there
> and content excerpts pulled in from other blogs.  (I don't know the exact
> mechanism nor how hard it would be to implement, though.)
>

Actually when we first discussed the cordova blog and how to post what,
this exact suggestion came up.  We all agreed it was a good idea (well, at
least those who decided to voice opinions at the time..).

At the time we mentioned that "release notes" are clearly the result of a
group effort on platforms/plugins, and thus a cordova centric blog post
should go out about it (but still posted from one specific Author with name
attribution).

For other types of posts that come from community members' personal blogs,
like user guides, or tips and tricks, we should absolutely syndicate.  I
don't think we've been doing this, though.  I'm not sure that we even
discussed the way to submit something for syndication.  Maybe we should
start on that, or maybe we should just leave it up to the phonegap blog,
I'm not sure.


>
> Cheers,
> Dan
>
>
>
>
>
> On Wednesday, December 11, 2013 11:39 AM, Andrew Grieve <
> agri...@chromium.org> wrote:
>
> Joe - would you be willing to write the blog post on Cordova's blog instead
> of a personal blog? Each cordova blog post does have an author with an
> optional link.
>
> I think having things on Cordova's blog rather than personal / downstream
> ones makes things more trusted & discoverable.
>
>
>
> On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:
>
> > On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve 
> > wrote:
> > > Yep, my main concern is communicating what's changed to our users for
> > > releases. Whether this file actually exists, or when it's updated, I
> care
> > > less about.
> > >
> > > Joe - if you don't think a single blog post is a good way to
> > communicating
> > > this, what's a good alternative? Should we have each platform write a
> > blog
> > > post as a part of the release instead of release notes?
> > >
> >
> > Yes, because until recently that's what we did.  Shaz wrote the iOS
> > one, and either Simon or I wrote a blog post about Android.  These
> > would then be syndicated and put on the phonegap.com blog.  We used to
> > have a perfectly good solution to this problem which went away roughly
> > around when 3.0.0 came out.
>  This mostly went away because of time
> > constraints and the fact that my own blog sucks ass and needs to be
> > migrated to a real server.  I think we need to go back to this.
> >
> > Also, this is a good way for people to get exposure and get their name
> > out there, so there's way more reward for doing this than just writing
> > RELEASENOTES.md which will be buried in the release and may or may not
> > be read.
> >
> > >
> > > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref 
> > wrote:
> > >
> > >> Michal wrote:
> > >> > when doing a release, you usually have to make
>  a
> > >> > mental note of what is worth testing, which usually means going
> > through
> > >> the
> > >> > changelog anyway, which means it isn't really adding serious time to
> > the
> > >> > release process.
> > >>
> > >> > However, this shouldn't be codified into our processes,
> > >> > and should be the responsibility of whoever is doing the blog post,
> > not
> > >> > whoever is doing the release, and those two aren't always the same.
> > >>
> > >> +1
> > >>
> > >> One problem is that the release blog seems to be pro forma and
> hurried.
> > >>
> > >> I've written
>  release notes with blog entries. Doing them well is
> > >> worthwhile.
> > >>
> > >> A few things that can help:
> > >> 1. Tagging issues at filing / analysis / resolution with a release
> note
> > >> indicator (yes, no)
> > >> 2. Working on the release notes before the release process finishes -
> > you
> > >> probably already have 90% of the release fixes known a few days
> before d
> > >> day. The last fixes can be yes/no as they're committed.
> > >> 3. It's important not to have "Fixed x; backed out fix for x". People
> > >> reading release notes don't care about the process between the
> previous
> > >> release and now, they want a clear indication of what has actually
> >
>  changed.
> > >>
> > >>
> > >> > So lets remove the req

Re: Plugman update on npm?

2013-12-11 Thread Steven Gill
I did do a small update last week with support for fireos & ubuntu so the
3.3.0-rc.1 didn't flail. I see there are some new changes now. I could push
out version 0.17.0 today.


On Wed, Dec 11, 2013 at 10:39 AM, Max Woghiren  wrote:

> Hi everyone—do we know when we plan to update npm with a newer version of
> plugman?  My understanding is that we have an "at most" weekly schedule for
> this, but have we done it recently?
>
> Thanks!
> -Max
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Dan Moore
Hi folks,

As a user, having all the release information in one place would be fantastic.  
I understand your desire to have a rapid release schedule (documented, as best 
as I can tell, here: 
http://phonegap.com/2012/04/12/rolling-releases-how-apache-cordova-becomes-phonegap-and-why/
 ).

But for normal app developers, having a clear understanding of what they get 
when upgrading (what features, bug fixes, improvements, etc have happened since 
the last release) is extremely helpful.  Doing this will probably increase 
adoption of the latest versions of cordova.

Another option different from having everything on one blog would be to do 
something like what phonegap.com/blog does--a mix of content hosted there and 
content excerpts pulled in from other blogs.  (I don't know the exact mechanism 
nor how hard it would be to implement, though.)

Cheers,
Dan





On Wednesday, December 11, 2013 11:39 AM, Andrew Grieve  
wrote:
 
Joe - would you be willing to write the blog post on Cordova's blog instead
of a personal blog? Each cordova blog post does have an author with an
optional link.

I think having things on Cordova's blog rather than personal / downstream
ones makes things more trusted & discoverable.



On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:

> On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve 
> wrote:
> > Yep, my main concern is communicating what's changed to our users for
> > releases. Whether this file actually exists, or when it's updated, I care
> > less about.
> >
> > Joe - if you don't think a single blog post is a good way to
> communicating
> > this, what's a good alternative? Should we have each platform write a
> blog
> > post as a part of the release instead of release notes?
> >
>
> Yes, because until recently that's what we did.  Shaz wrote the iOS
> one, and either Simon or I wrote a blog post about Android.  These
> would then be syndicated and put on the phonegap.com blog.  We used to
> have a perfectly good solution to this problem which went away roughly
> around when 3.0.0 came out. 
 This mostly went away because of time
> constraints and the fact that my own blog sucks ass and needs to be
> migrated to a real server.  I think we need to go back to this.
>
> Also, this is a good way for people to get exposure and get their name
> out there, so there's way more reward for doing this than just writing
> RELEASENOTES.md which will be buried in the release and may or may not
> be read.
>
> >
> > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref 
> wrote:
> >
> >> Michal wrote:
> >> > when doing a release, you usually have to make
 a
> >> > mental note of what is worth testing, which usually means going
> through
> >> the
> >> > changelog anyway, which means it isn't really adding serious time to
> the
> >> > release process.
> >>
> >> > However, this shouldn't be codified into our processes,
> >> > and should be the responsibility of whoever is doing the blog post,
> not
> >> > whoever is doing the release, and those two aren't always the same.
> >>
> >> +1
> >>
> >> One problem is that the release blog seems to be pro forma and hurried.
> >>
> >> I've written
 release notes with blog entries. Doing them well is
> >> worthwhile.
> >>
> >> A few things that can help:
> >> 1. Tagging issues at filing / analysis / resolution with a release note
> >> indicator (yes, no)
> >> 2. Working on the release notes before the release process finishes -
> you
> >> probably already have 90% of the release fixes known a few days before d
> >> day. The last fixes can be yes/no as they're committed.
> >> 3. It's important not to have "Fixed x; backed out fix for x". People
> >> reading release notes don't care about the process between the previous
> >> release and now, they want a clear indication of what has actually
>
 changed.
> >>
> >>
> >> > So lets remove the requirement, and I guess the RELEASNOTES.md file
> from
> >> the repos?
> >>
> >> +1
> >> -
> >> This transmission (including any attachments) may contain confidential
> >> information, privileged material (including material protected by the
> >> solicitor-client or other applicable privileges), or constitute
> non-public
> >> information. Any use of this information by anyone other than the
> intended
> >> recipient is prohibited. If you have received this transmission in
> error,
> >> please immediately reply to the sender and delete this information from
> >> your system. Use, dissemination, distribution, or reproduction of this
> >> transmission by unintended recipients is not authorized and may be
> unlawful.
> >>
>

Re: 3.3.0 Release

2013-12-11 Thread David Barth

Le 11/12/2013 16:39, David Barth a écrit :

Le 10/12/2013 19:37, Steven Gill a écrit :

Lets start the final round of tagging today!
https://issues.apache.org/jira/browse/CB-5538
+1, but we're still chasing down some dependency and chmod +x issues 
on the Ubuntu part. Will update this thread shortly.
Maxim added a workaround in those pull request to ensure that the 
dependencies for the ubuntu commands are satisfied:


https://github.com/apache/cordova-ubuntu/pull/2
https://github.com/apache/cordova-ubuntu/pull/1

Somehow, cordova-cli is not honoring the dependencies expressed in the 
package.json file here:


https://github.com/apache/cordova-ubuntu/blob/master/bin/package.json

maybe they should move one directory above, but I'm not so keen on 
changing that this late.


Please note also that the rc1 cordova-cli in the NPM registry still 
points to an old plugman version that does not support ubuntu.


Let me know if i'm missing something here.

I'll send an update when the remaining command permission issue is 
fixed. https://issues.apache.org/jira/browse/CB-5629


David


Re: RELEASENOTES.md and coho

2013-12-11 Thread Joe Bowser
On Wed, Dec 11, 2013 at 10:38 AM, Andrew Grieve  wrote:
> Joe - would you be willing to write the blog post on Cordova's blog instead
> of a personal blog? Each cordova blog post does have an author with an
> optional link.

If I have to, I think we should syndicate instead so we don't
copy/paste content.

> I think having things on Cordova's blog rather than personal / downstream
> ones makes things more trusted & discoverable.

I disagree.  We should have Cordova's blog syndicate other known
blogs.  I don't think anyone actually reads the Cordova blog.  I
haven't until today.

Joe


>
>
> On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:
>
>> On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve 
>> wrote:
>> > Yep, my main concern is communicating what's changed to our users for
>> > releases. Whether this file actually exists, or when it's updated, I care
>> > less about.
>> >
>> > Joe - if you don't think a single blog post is a good way to
>> communicating
>> > this, what's a good alternative? Should we have each platform write a
>> blog
>> > post as a part of the release instead of release notes?
>> >
>>
>> Yes, because until recently that's what we did.  Shaz wrote the iOS
>> one, and either Simon or I wrote a blog post about Android.  These
>> would then be syndicated and put on the phonegap.com blog.  We used to
>> have a perfectly good solution to this problem which went away roughly
>> around when 3.0.0 came out.  This mostly went away because of time
>> constraints and the fact that my own blog sucks ass and needs to be
>> migrated to a real server.  I think we need to go back to this.
>>
>> Also, this is a good way for people to get exposure and get their name
>> out there, so there's way more reward for doing this than just writing
>> RELEASENOTES.md which will be buried in the release and may or may not
>> be read.
>>
>> >
>> > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref 
>> wrote:
>> >
>> >> Michal wrote:
>> >> > when doing a release, you usually have to make a
>> >> > mental note of what is worth testing, which usually means going
>> through
>> >> the
>> >> > changelog anyway, which means it isn't really adding serious time to
>> the
>> >> > release process.
>> >>
>> >> > However, this shouldn't be codified into our processes,
>> >> > and should be the responsibility of whoever is doing the blog post,
>> not
>> >> > whoever is doing the release, and those two aren't always the same.
>> >>
>> >> +1
>> >>
>> >> One problem is that the release blog seems to be pro forma and hurried.
>> >>
>> >> I've written release notes with blog entries. Doing them well is
>> >> worthwhile.
>> >>
>> >> A few things that can help:
>> >> 1. Tagging issues at filing / analysis / resolution with a release note
>> >> indicator (yes, no)
>> >> 2. Working on the release notes before the release process finishes -
>> you
>> >> probably already have 90% of the release fixes known a few days before d
>> >> day. The last fixes can be yes/no as they're committed.
>> >> 3. It's important not to have "Fixed x; backed out fix for x". People
>> >> reading release notes don't care about the process between the previous
>> >> release and now, they want a clear indication of what has actually
>> changed.
>> >>
>> >>
>> >> > So lets remove the requirement, and I guess the RELEASNOTES.md file
>> from
>> >> the repos?
>> >>
>> >> +1
>> >> -
>> >> This transmission (including any attachments) may contain confidential
>> >> information, privileged material (including material protected by the
>> >> solicitor-client or other applicable privileges), or constitute
>> non-public
>> >> information. Any use of this information by anyone other than the
>> intended
>> >> recipient is prohibited. If you have received this transmission in
>> error,
>> >> please immediately reply to the sender and delete this information from
>> >> your system. Use, dissemination, distribution, or reproduction of this
>> >> transmission by unintended recipients is not authorized and may be
>> unlawful.
>> >>
>>


Plugman update on npm?

2013-12-11 Thread Max Woghiren
Hi everyone—do we know when we plan to update npm with a newer version of
plugman?  My understanding is that we have an "at most" weekly schedule for
this, but have we done it recently?

Thanks!
-Max


Re: RELEASENOTES.md and coho

2013-12-11 Thread Andrew Grieve
Joe - would you be willing to write the blog post on Cordova's blog instead
of a personal blog? Each cordova blog post does have an author with an
optional link.

I think having things on Cordova's blog rather than personal / downstream
ones makes things more trusted & discoverable.


On Wed, Dec 11, 2013 at 1:27 PM, Joe Bowser  wrote:

> On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve 
> wrote:
> > Yep, my main concern is communicating what's changed to our users for
> > releases. Whether this file actually exists, or when it's updated, I care
> > less about.
> >
> > Joe - if you don't think a single blog post is a good way to
> communicating
> > this, what's a good alternative? Should we have each platform write a
> blog
> > post as a part of the release instead of release notes?
> >
>
> Yes, because until recently that's what we did.  Shaz wrote the iOS
> one, and either Simon or I wrote a blog post about Android.  These
> would then be syndicated and put on the phonegap.com blog.  We used to
> have a perfectly good solution to this problem which went away roughly
> around when 3.0.0 came out.  This mostly went away because of time
> constraints and the fact that my own blog sucks ass and needs to be
> migrated to a real server.  I think we need to go back to this.
>
> Also, this is a good way for people to get exposure and get their name
> out there, so there's way more reward for doing this than just writing
> RELEASENOTES.md which will be buried in the release and may or may not
> be read.
>
> >
> > On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref 
> wrote:
> >
> >> Michal wrote:
> >> > when doing a release, you usually have to make a
> >> > mental note of what is worth testing, which usually means going
> through
> >> the
> >> > changelog anyway, which means it isn't really adding serious time to
> the
> >> > release process.
> >>
> >> > However, this shouldn't be codified into our processes,
> >> > and should be the responsibility of whoever is doing the blog post,
> not
> >> > whoever is doing the release, and those two aren't always the same.
> >>
> >> +1
> >>
> >> One problem is that the release blog seems to be pro forma and hurried.
> >>
> >> I've written release notes with blog entries. Doing them well is
> >> worthwhile.
> >>
> >> A few things that can help:
> >> 1. Tagging issues at filing / analysis / resolution with a release note
> >> indicator (yes, no)
> >> 2. Working on the release notes before the release process finishes -
> you
> >> probably already have 90% of the release fixes known a few days before d
> >> day. The last fixes can be yes/no as they're committed.
> >> 3. It's important not to have "Fixed x; backed out fix for x". People
> >> reading release notes don't care about the process between the previous
> >> release and now, they want a clear indication of what has actually
> changed.
> >>
> >>
> >> > So lets remove the requirement, and I guess the RELEASNOTES.md file
> from
> >> the repos?
> >>
> >> +1
> >> -
> >> This transmission (including any attachments) may contain confidential
> >> information, privileged material (including material protected by the
> >> solicitor-client or other applicable privileges), or constitute
> non-public
> >> information. Any use of this information by anyone other than the
> intended
> >> recipient is prohibited. If you have received this transmission in
> error,
> >> please immediately reply to the sender and delete this information from
> >> your system. Use, dissemination, distribution, or reproduction of this
> >> transmission by unintended recipients is not authorized and may be
> unlawful.
> >>
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Joe Bowser
On Wed, Dec 11, 2013 at 10:23 AM, Andrew Grieve  wrote:
> Yep, my main concern is communicating what's changed to our users for
> releases. Whether this file actually exists, or when it's updated, I care
> less about.
>
> Joe - if you don't think a single blog post is a good way to communicating
> this, what's a good alternative? Should we have each platform write a blog
> post as a part of the release instead of release notes?
>

Yes, because until recently that's what we did.  Shaz wrote the iOS
one, and either Simon or I wrote a blog post about Android.  These
would then be syndicated and put on the phonegap.com blog.  We used to
have a perfectly good solution to this problem which went away roughly
around when 3.0.0 came out.  This mostly went away because of time
constraints and the fact that my own blog sucks ass and needs to be
migrated to a real server.  I think we need to go back to this.

Also, this is a good way for people to get exposure and get their name
out there, so there's way more reward for doing this than just writing
RELEASENOTES.md which will be buried in the release and may or may not
be read.

>
> On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref  wrote:
>
>> Michal wrote:
>> > when doing a release, you usually have to make a
>> > mental note of what is worth testing, which usually means going through
>> the
>> > changelog anyway, which means it isn't really adding serious time to the
>> > release process.
>>
>> > However, this shouldn't be codified into our processes,
>> > and should be the responsibility of whoever is doing the blog post, not
>> > whoever is doing the release, and those two aren't always the same.
>>
>> +1
>>
>> One problem is that the release blog seems to be pro forma and hurried.
>>
>> I've written release notes with blog entries. Doing them well is
>> worthwhile.
>>
>> A few things that can help:
>> 1. Tagging issues at filing / analysis / resolution with a release note
>> indicator (yes, no)
>> 2. Working on the release notes before the release process finishes - you
>> probably already have 90% of the release fixes known a few days before d
>> day. The last fixes can be yes/no as they're committed.
>> 3. It's important not to have "Fixed x; backed out fix for x". People
>> reading release notes don't care about the process between the previous
>> release and now, they want a clear indication of what has actually changed.
>>
>>
>> > So lets remove the requirement, and I guess the RELEASNOTES.md file from
>> the repos?
>>
>> +1
>> -
>> This transmission (including any attachments) may contain confidential
>> information, privileged material (including material protected by the
>> solicitor-client or other applicable privileges), or constitute non-public
>> information. Any use of this information by anyone other than the intended
>> recipient is prohibited. If you have received this transmission in error,
>> please immediately reply to the sender and delete this information from
>> your system. Use, dissemination, distribution, or reproduction of this
>> transmission by unintended recipients is not authorized and may be unlawful.
>>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Andrew Grieve
Yep, my main concern is communicating what's changed to our users for
releases. Whether this file actually exists, or when it's updated, I care
less about.

Joe - if you don't think a single blog post is a good way to communicating
this, what's a good alternative? Should we have each platform write a blog
post as a part of the release instead of release notes?


On Wed, Dec 11, 2013 at 11:05 AM, Josh Soref  wrote:

> Michal wrote:
> > when doing a release, you usually have to make a
> > mental note of what is worth testing, which usually means going through
> the
> > changelog anyway, which means it isn't really adding serious time to the
> > release process.
>
> > However, this shouldn't be codified into our processes,
> > and should be the responsibility of whoever is doing the blog post, not
> > whoever is doing the release, and those two aren't always the same.
>
> +1
>
> One problem is that the release blog seems to be pro forma and hurried.
>
> I've written release notes with blog entries. Doing them well is
> worthwhile.
>
> A few things that can help:
> 1. Tagging issues at filing / analysis / resolution with a release note
> indicator (yes, no)
> 2. Working on the release notes before the release process finishes - you
> probably already have 90% of the release fixes known a few days before d
> day. The last fixes can be yes/no as they're committed.
> 3. It's important not to have "Fixed x; backed out fix for x". People
> reading release notes don't care about the process between the previous
> release and now, they want a clear indication of what has actually changed.
>
>
> > So lets remove the requirement, and I guess the RELEASNOTES.md file from
> the repos?
>
> +1
> -
> This transmission (including any attachments) may contain confidential
> information, privileged material (including material protected by the
> solicitor-client or other applicable privileges), or constitute non-public
> information. Any use of this information by anyone other than the intended
> recipient is prohibited. If you have received this transmission in error,
> please immediately reply to the sender and delete this information from
> your system. Use, dissemination, distribution, or reproduction of this
> transmission by unintended recipients is not authorized and may be unlawful.
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Andrew Grieve
If it worked in 3.1 and broke in 3.2, then it's very likely
CB-5481 and
is fixed in 3.3


On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser  wrote:

> No. As far as I'm concerned this problem doesn't exist until it is a JIRA
> issue with some sample code that I can use to reproduce the error. I doubt
> that this was a regression since we never explicitly supported your use
> case to begin with.
>
> Of course, patches are always welcome.
>  On 11 Dec 2013 07:58, "Axel Nennker"  wrote:
>
> > No. I intented this to be on the list. Gmail UI irks.
> > Could this be fixed in 3.3?
> > I know it is late but...
> >
> > -Axel
> >
> >
> > 2013/12/11 Joe Bowser 
> >
> > > Did you mean to to send this to me personally?
> > >
> > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker 
> > > wrote:
> > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > > > The config.xml is where it belongs but PluginManager does not find it
> > in
> > > a
> > > > certain (legal) setting.
> > > > When the app is started "normally" PluginManager finds config.xml
> > (which
> > > > proves that is at the correct location)
> > > > When the app is started because an IntentFilter triggers then
> > > > PluginInManager does not find it.
> > > >
> > > > The "main" activity is in one class like a.b.c.Main while the
> > > IntentFilter
> > > > activity is in another e.g. u.v.w.IF
> > > > IF starts Main successfully but - my guess is - that PluginManager
> > tries
> > > to
> > > > find config.xml in package u.v.w
> > > >
> > > > The JUNIT/spec test should be updated to catch this situation if
> > > possible.
> > > > I guess I could change the package names so that IF is in a.b.c too
> but
> > > it
> > > > worked in 3.1 and I see no reason why this is "complex"
> > > > The u.v.w.IF features are implemented in a plugin that registers the
> > > intent
> > > > filter in the app's AndroidManifest. It is developed by my colleagues
> > who
> > > > chose the u.v.w package name. I am using this plugin in my app that
> has
> > > the
> > > > a.b.c package namespace.
> > > > I do not agree that this a complex or irregular use of cordova or
> > > Intents.
> > > >
> > > > Please revert the code back to the 3.1 behaviour.
> > > >
> > > > Axel
> > > >
> > > >
> > > > 2013/12/11 Joe Bowser 
> > > >>
> > > >> I haven't noticed this during any of our testing, nor have I noticed
> > > >> this when hacking with Google Glass, which uses the Google Glass
> voice
> > > >> trigger intent to launch Cordova, not the LAUNCHER intent like the
> > > >> launch menu.  Furthermore, our JUnit tests haven't caught the tests
> > > >> either, and they depend on at least three plugins to be loaded by
> > > >> PluginManager from config.xml.  Honestly, this only breaks if you
> > > >> don't know what you're doing with Intents.
> > > >>
> > > >> Honestly, It sounds to me like your config.xml isn't in the correct
> > > >> place in your project, especially if you're trying to do something
> > > >> complex with Android and intents.  I would check that making these
> > > >> sorts of statements.  If config.xml wasn't being read, it would have
> > > >> been caught right away, since everything is a plugin now.
> > > >>
> > > >>
> > > >>
> > > >> On Wed, Dec 11, 2013 at 4:33 AM,   wrote:
> > > >> > Hi,
> > > >> >
> > > >> > We are seeing some strange effect on Android when we move from
> > > >> > 3.1.0-0.2.0 to a newer version.
> > > >> > We tried with 3.2.0-0.4.0 and 3.3-rc1
> > > >> >
> > > >> > I think that PluginManager changed and now fails to load
> config.xml
> > > >> >
> > > >> > This happens in my project when the app is opened by the operating
> > > >> > system because it is configured so through an intent filter.
> > > >> > The original activity is not a child of DroidGap but starts the
> > "main"
> > > >> > activity using startActivity.
> > > >> >
> > > >> > The code to get the id of the config.xml file is not working in
> this
> > > >> > case.
> > > >> >
> > > >> > Why was the 3.1.0 code changed?
> > > >> >
> > > >> > Cheers
> > > >> > Axel
> > > >> >
> > > >> > D/CordovaActivity(12320): CordovaActivity.init()
> > > >> > D/CordovaWebView(12320): >>>
> > > >> >
> > >
> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
> > > >> > D/PluginManager(12320): init()
> > > >> > E/PluginManager(12320):
> > > >> >
> > >
> >
> =
> > > >> > E/PluginManager(12320): ERROR: config.xml is missing.  Add
> > > >> > res/xml/config.xml to your project.
> > > >> > E/PluginManager(12320):
> > > >> >
> > >
> >
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
> > > >> > E/PluginManager(12320):
> > > >> >
> > >
> >
> =
> > > >> > D/CordovaWebView(12320): >>> loadU

RE: Windows Phone binary bridge

2013-12-11 Thread Sergey Grebnov (Akvelon)
I've just compared ArrayBuffer to string conversion time via currently used 
base64 method and via blob reading and the second one runs faster (2-4 times on 
WP8 HTC 8S and 7-10 times faster on iPhone5; don't have Android device to 
test). 

So in case browser natively support FileReader and Blob api we may want to use 
that method.

Here is test page I use
http://goo.gl/5mpjYu
http://sgrebnov.github.io/tmp/binPerfTest/www/js/index.js 

testBase64Str: function(data) {
 var defer = Q.defer();
defer.resolve(base64.fromArrayBuffer(data));

return defer.promise;
},

testBlobReader: function(data) {
var defer = Q.defer(),
blob = new Blob([data]),
reader = new FileReader();

reader.onloadend = function(e) {
defer.resolve(e.target.result);
}
reader.readAsText(blob, "UTF-8");
return defer.promise;
},


Thx!
Sergey
-Original Message-
From: Sergey Grebnov (Akvelon) 
Sent: Monday, December 9, 2013 9:36 PM
To: dev@cordova.apache.org
Subject: RE: Windows Phone binary bridge

I see.. I've added similar logic to windows phone exec bridge
https://github.com/apache/cordova-js/pull/59

Tested on File Api, below  is updated version after patching wp bridge
https://github.com/apache/cordova-plugin-file/pull/18 

PS. As showed at [1] manual array manipulation is not the fastest solution but 
it is the most convenient synchronous one 
[1]http://updates.html5rocks.com/2012/06/How-to-convert-ArrayBuffer-to-and-from-String
 

Thx!
Sergey
-Original Message-
From: iclell...@google.com [mailto:iclell...@google.com] On Behalf Of Ian 
Clelland
Sent: Monday, December 9, 2013 9:06 PM
To: dev@cordova.apache.org
Subject: Re: Windows Phone binary bridge

Yes, we used to use that; I had to switch it to use a base64-encoding method; 
Using Array.prototype.apply() literally pushes every element of the array 
individually onto the stack as a separate argument. The exact limit depends on 
the device, but eventually they all fail with a "Stack 
exploded"-sort-of-exception.

Ian


On Mon, Dec 9, 2013 at 10:32 AM, Michal Mocny  wrote:

> Where do you see an implementation using Array.apply(null, new
> Uint8Array(...)) ?
>
> We used to have this bug in ios/android binary bridge but that was 
> patched many releases ago (unless there is a stray one somewhere that 
> still needs fixing).
>
>
> On Mon, Dec 9, 2013 at 10:15 AM, Sergey Grebnov (Akvelon) < 
> v-seg...@microsoft.com> wrote:
>
> > Working on this...I just found out that current implementation via 
> > Array.apply(null, new Uint8Array(...)) doesn't work for big buffers
> (bigger
> > than 150Kb) and could raise Maximum call stack size exceeded error. 
> > So
> I'm
> > looking on better/right way to do typed arrays conversion... will 
> > keep
> you
> > updated.
> >
> > Thx!
> > Sergey
> > -Original Message-
> > From: Parashuram Narasimhan (MS OPEN TECH) [mailto:
> panar...@microsoft.com]
> > Sent: Saturday, December 7, 2013 1:52 AM
> > To: dev@cordova.apache.org
> > Subject: RE: Windows Phone binary bridge
> >
> > I think Sergey should be able to squeeze this into his schedule. We 
> > were looking at fixing plugin support and this could be a good way 
> > to start
> it.
> >
> > -Original Message-
> > From: Jesse [mailto:purplecabb...@gmail.com]
> > Sent: Friday, December 6, 2013 1:30 PM
> > To: dev@cordova.apache.org
> > Subject: Re: Windows Phone binary bridge
> >
> > FYI, you can buy a no contract Nokia 520 WP8 device for $120, and 
> > developing in the emulator is more than sufficient for most tasks.
> >
> > Sergey, do you got this?
> >
> >
> > @purplecabbage
> > risingj.com
> >
> >
> > On Fri, Dec 6, 2013 at 11:21 AM, Ian Clelland 
> >  > >wrote:
> >
> > > On Fri, Dec 6, 2013 at 2:15 PM, Sergey Grebnov (Akvelon) < 
> > > v-seg...@microsoft.com> wrote:
> > >
> > > > Agree and  like this idea. I don't think there are many other 
> > > > components (if any) which use ArrayBuffer but must be tested 
> > > > well anyway since this changes core. Ian do you want to complete 
> > > > this yourself or you are ok if
> > > I
> > > > handle this?
> > > >
> > >
> > > I'm totally okay with you doing it -- I don't have a real windows 
> > > phone device to test it on; I'd be committing with my eyes closed 
> > > and hoping for the best :)
> > >
> > > I think you're right that there aren't any other core plugins 
> > > using the binary bridge, but that could change, and there are 
> > > probably already third-party plugins using it.
> > >
> > > If we do this right, then other developers won't have to roll 
> > > their own solution in each case, and it will be easier to port the 
> > > third-party plugins to WP.
> > >
> > > Let me know if you need any support; I'll help if I can.
> > >
> > > Ian
> > >
> > >
> > > >
> > > > -Sergey
> > > > -Original Message-
> > > > From: iclell...@google.com [mailto:iclell...@google.com] On 
> > > > Behalf Of
> > > Ian
> > >

Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
No. As far as I'm concerned this problem doesn't exist until it is a JIRA
issue with some sample code that I can use to reproduce the error. I doubt
that this was a regression since we never explicitly supported your use
case to begin with.

Of course, patches are always welcome.
 On 11 Dec 2013 07:58, "Axel Nennker"  wrote:

> No. I intented this to be on the list. Gmail UI irks.
> Could this be fixed in 3.3?
> I know it is late but...
>
> -Axel
>
>
> 2013/12/11 Joe Bowser 
>
> > Did you mean to to send this to me personally?
> >
> > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker 
> > wrote:
> > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > > The config.xml is where it belongs but PluginManager does not find it
> in
> > a
> > > certain (legal) setting.
> > > When the app is started "normally" PluginManager finds config.xml
> (which
> > > proves that is at the correct location)
> > > When the app is started because an IntentFilter triggers then
> > > PluginInManager does not find it.
> > >
> > > The "main" activity is in one class like a.b.c.Main while the
> > IntentFilter
> > > activity is in another e.g. u.v.w.IF
> > > IF starts Main successfully but - my guess is - that PluginManager
> tries
> > to
> > > find config.xml in package u.v.w
> > >
> > > The JUNIT/spec test should be updated to catch this situation if
> > possible.
> > > I guess I could change the package names so that IF is in a.b.c too but
> > it
> > > worked in 3.1 and I see no reason why this is "complex"
> > > The u.v.w.IF features are implemented in a plugin that registers the
> > intent
> > > filter in the app's AndroidManifest. It is developed by my colleagues
> who
> > > chose the u.v.w package name. I am using this plugin in my app that has
> > the
> > > a.b.c package namespace.
> > > I do not agree that this a complex or irregular use of cordova or
> > Intents.
> > >
> > > Please revert the code back to the 3.1 behaviour.
> > >
> > > Axel
> > >
> > >
> > > 2013/12/11 Joe Bowser 
> > >>
> > >> I haven't noticed this during any of our testing, nor have I noticed
> > >> this when hacking with Google Glass, which uses the Google Glass voice
> > >> trigger intent to launch Cordova, not the LAUNCHER intent like the
> > >> launch menu.  Furthermore, our JUnit tests haven't caught the tests
> > >> either, and they depend on at least three plugins to be loaded by
> > >> PluginManager from config.xml.  Honestly, this only breaks if you
> > >> don't know what you're doing with Intents.
> > >>
> > >> Honestly, It sounds to me like your config.xml isn't in the correct
> > >> place in your project, especially if you're trying to do something
> > >> complex with Android and intents.  I would check that making these
> > >> sorts of statements.  If config.xml wasn't being read, it would have
> > >> been caught right away, since everything is a plugin now.
> > >>
> > >>
> > >>
> > >> On Wed, Dec 11, 2013 at 4:33 AM,   wrote:
> > >> > Hi,
> > >> >
> > >> > We are seeing some strange effect on Android when we move from
> > >> > 3.1.0-0.2.0 to a newer version.
> > >> > We tried with 3.2.0-0.4.0 and 3.3-rc1
> > >> >
> > >> > I think that PluginManager changed and now fails to load config.xml
> > >> >
> > >> > This happens in my project when the app is opened by the operating
> > >> > system because it is configured so through an intent filter.
> > >> > The original activity is not a child of DroidGap but starts the
> "main"
> > >> > activity using startActivity.
> > >> >
> > >> > The code to get the id of the config.xml file is not working in this
> > >> > case.
> > >> >
> > >> > Why was the 3.1.0 code changed?
> > >> >
> > >> > Cheers
> > >> > Axel
> > >> >
> > >> > D/CordovaActivity(12320): CordovaActivity.init()
> > >> > D/CordovaWebView(12320): >>>
> > >> >
> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
> > >> > D/PluginManager(12320): init()
> > >> > E/PluginManager(12320):
> > >> >
> >
> =
> > >> > E/PluginManager(12320): ERROR: config.xml is missing.  Add
> > >> > res/xml/config.xml to your project.
> > >> > E/PluginManager(12320):
> > >> >
> >
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
> > >> > E/PluginManager(12320):
> > >> >
> >
> =
> > >> > D/CordovaWebView(12320): >>> loadUrlNow()
> > >> > D/ItemImport(12320): onResume() loadURL started:
> > >
> > >
> >
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Josh Soref
Michal‎ wrote:
> when doing a release, you usually have to make a
> mental note of what is worth testing, which usually means going through the
> changelog anyway, which means it isn't really adding serious time to the
> release process.

> However, this shouldn't be codified into our processes,
> and should be the responsibility of whoever is doing the blog post, not
> whoever is doing the release, and those two aren't always the same.

+1

One problem is that the release blog seems to be pro forma and hurried. 

I've written release notes with blog entries. Doing them well is worthwhile. 

A few things that can help:
1. Tagging issues at filing / analysis / resolution with a release note 
indicator (yes, no) 
2. Working on the release notes before the release process finishes - you 
probably already have 90% of the release fixes known a few days before d day. 
The last fixes can be yes/no as they're committed.  
3. It's important not to have "Fixed x; backed out fix for x". People reading 
release notes don't care about the process between the previous release and 
now, they want a clear indication of what has actually changed.  
‎
> So lets remove the requirement, and I guess the RELEASNOTES.md file from the 
> repos?

+1
-
This transmission (including any attachments) may contain confidential 
information, privileged material (including material protected by the 
solicitor-client or other applicable privileges), or constitute non-public 
information. Any use of this information by anyone other than the intended 
recipient is prohibited. If you have received this transmission in error, 
please immediately reply to the sender and delete this information from your 
system. Use, dissemination, distribution, or reproduction of this transmission 
by unintended recipients is not authorized and may be unlawful.


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Axel Nennker
I think it is a good security practice to tie an app to the app's backend.


2013/12/11 Josh Soref 

> Ian wrote:
> > There was some talk on the list a couple months ago about this -- not for
> > file-transfer specifically,
> > but the general idea of supporting custom
> certificates, or CAs in Cordova.
>
> This came up yesterday in the office.
>
> > I think that, after a number of emails, we concluded that for users who
> > have legitimate custom certificate requirements, that there should be
> > os-policy-level mechanisms for adding custom certs, and that the
> individual
> > application was the wrong level to be managing them.
>
> I made the opposite argument. Users will not be able to do anything useful
> with global stores. The result is that unrelated applications will still /
> misappropriate certificates.
>
> Google is supporting zero trust:
>
> http://www.scmagazine.com.au/News/367057,googles-plan-to-kill-the-corporate-network.aspx
>
>
> http://www.darkreading.com/perimeter/forrester-pushes-zero-trust-model-for-se/227500145
>
> While you might be OK with a prompt to enter an RSA token, you could
> easily not recognize that the requesting party shouldn't be given it.
>
> Browser developers failed miserably the first time that client certificate
> UI was designed - Neither the "automatic selection" nor the "prompt user
> for certificate" choices work safely.
>
>
> -
> This transmission (including any attachments) may contain confidential
> information, privileged material (including material protected by the
> solicitor-client or other applicable privileges), or constitute non-public
> information. Any use of this information by anyone other than the intended
> recipient is prohibited. If you have received this transmission in error,
> please immediately reply to the sender and delete this information from
> your system. Use, dissemination, distribution, or reproduction of this
> transmission by unintended recipients is not authorized and may be unlawful.
>
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel Nennker
No. I intented this to be on the list. Gmail UI irks.
Could this be fixed in 3.3?
I know it is late but...

-Axel


2013/12/11 Joe Bowser 

> Did you mean to to send this to me personally?
>
> On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker 
> wrote:
> > Well, this repeatable. Use 3.2 fail, go back to 3.1 win.
> > The config.xml is where it belongs but PluginManager does not find it in
> a
> > certain (legal) setting.
> > When the app is started "normally" PluginManager finds config.xml (which
> > proves that is at the correct location)
> > When the app is started because an IntentFilter triggers then
> > PluginInManager does not find it.
> >
> > The "main" activity is in one class like a.b.c.Main while the
> IntentFilter
> > activity is in another e.g. u.v.w.IF
> > IF starts Main successfully but - my guess is - that PluginManager tries
> to
> > find config.xml in package u.v.w
> >
> > The JUNIT/spec test should be updated to catch this situation if
> possible.
> > I guess I could change the package names so that IF is in a.b.c too but
> it
> > worked in 3.1 and I see no reason why this is "complex"
> > The u.v.w.IF features are implemented in a plugin that registers the
> intent
> > filter in the app's AndroidManifest. It is developed by my colleagues who
> > chose the u.v.w package name. I am using this plugin in my app that has
> the
> > a.b.c package namespace.
> > I do not agree that this a complex or irregular use of cordova or
> Intents.
> >
> > Please revert the code back to the 3.1 behaviour.
> >
> > Axel
> >
> >
> > 2013/12/11 Joe Bowser 
> >>
> >> I haven't noticed this during any of our testing, nor have I noticed
> >> this when hacking with Google Glass, which uses the Google Glass voice
> >> trigger intent to launch Cordova, not the LAUNCHER intent like the
> >> launch menu.  Furthermore, our JUnit tests haven't caught the tests
> >> either, and they depend on at least three plugins to be loaded by
> >> PluginManager from config.xml.  Honestly, this only breaks if you
> >> don't know what you're doing with Intents.
> >>
> >> Honestly, It sounds to me like your config.xml isn't in the correct
> >> place in your project, especially if you're trying to do something
> >> complex with Android and intents.  I would check that making these
> >> sorts of statements.  If config.xml wasn't being read, it would have
> >> been caught right away, since everything is a plugin now.
> >>
> >>
> >>
> >> On Wed, Dec 11, 2013 at 4:33 AM,   wrote:
> >> > Hi,
> >> >
> >> > We are seeing some strange effect on Android when we move from
> >> > 3.1.0-0.2.0 to a newer version.
> >> > We tried with 3.2.0-0.4.0 and 3.3-rc1
> >> >
> >> > I think that PluginManager changed and now fails to load config.xml
> >> >
> >> > This happens in my project when the app is opened by the operating
> >> > system because it is configured so through an intent filter.
> >> > The original activity is not a child of DroidGap but starts the "main"
> >> > activity using startActivity.
> >> >
> >> > The code to get the id of the config.xml file is not working in this
> >> > case.
> >> >
> >> > Why was the 3.1.0 code changed?
> >> >
> >> > Cheers
> >> > Axel
> >> >
> >> > D/CordovaActivity(12320): CordovaActivity.init()
> >> > D/CordovaWebView(12320): >>>
> >> >
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
> >> > D/PluginManager(12320): init()
> >> > E/PluginManager(12320):
> >> >
> =
> >> > E/PluginManager(12320): ERROR: config.xml is missing.  Add
> >> > res/xml/config.xml to your project.
> >> > E/PluginManager(12320):
> >> >
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
> >> > E/PluginManager(12320):
> >> >
> =
> >> > D/CordovaWebView(12320): >>> loadUrlNow()
> >> > D/ItemImport(12320): onResume() loadURL started:
> >
> >
>


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Josh Soref
Ian wrote:
> There was some talk on the list a couple months ago about this -- not for
> file-transfer specifically,
> but the general idea of supporting custom
certificates, or CAs in Cordova.

This came up yesterday in the office. 

> I think that, after a number of emails, we concluded that for users who
> have legitimate custom certificate requirements, that there should be
> os-policy-level mechanisms for adding custom certs, and that the individual
> application was the wrong level to be managing them. 

I made the opposite argument. Users will not be able to do anything useful with 
global stores. The result is that unrelated applications will still / 
misappropriate certificates. 

Google is supporting zero trust:
http://www.scmagazine.com.au/News/367057,googles-plan-to-kill-the-corporate-network.aspx

http://www.darkreading.com/perimeter/forrester-pushes-zero-trust-model-for-se/227500145

While you might be OK with a prompt to enter an RSA token, you could easily not 
recognize that the requesting party shouldn't be given it. 

Browser developers failed miserably the first time that client certificate UI 
was designed - Neither the "automatic selection" nor the "prompt user for 
certificate" choices work safely. 


-
This transmission (including any attachments) may contain confidential 
information, privileged material (including material protected by the 
solicitor-client or other applicable privileges), or constitute non-public 
information. Any use of this information by anyone other than the intended 
recipient is prohibited. If you have received this transmission in error, 
please immediately reply to the sender and delete this information from your 
system. Use, dissemination, distribution, or reproduction of this transmission 
by unintended recipients is not authorized and may be unlawful.



Re: RELEASENOTES.md and coho

2013-12-11 Thread Michal Mocny
RE: where to download: http://cordova.apache.org/#download
(Theres a huge download button above-the-fold on cordova.io)

Personally, I like summarized release notes in blog posts, but I do agree
that committing a file that summarized git log into the same git repo is
really not useful.  And if that step is on the blocking path for releasing
a platform, perhaps that isn't right.

Andrew points out that when doing a release, you usually have to make a
mental note of what is worth testing, which usually means going through the
changelog anyway, which means it isn't really adding serious time to the
release process.  However, this shouldn't be codified into our processes,
and should be the responsibility of whoever is doing the blog post, not
whoever is doing the release, and those two aren't always the same.  So
lets remove the requirement, and I guess the RELEASNOTES.md file from the
repos?

-Michal

On Wed, Dec 11, 2013 at 10:20 AM, Joe Bowser  wrote:

> I don't think our platform release notes should be in a single blog post
> and have a single author.  I really want this out of the release process
> since I don't want to hold releases up fiddling with a markdown file for an
> hour or more.  That's the real issue.
> On 11 Dec 2013 07:11, "Andrew Grieve"  wrote:
>
> > I'll write them for Android if it's annoying for you. I really don't
> think
> > it's feasible for one person to make sense of all commit messages across
> > all platforms.
> >
> > On Wed, Dec 11, 2013 at 10:02 AM, Joe Bowser  wrote:
> >
> > > On Wed, Dec 11, 2013 at 6:53 AM, Andrew Grieve 
> > > wrote:
> > > > Release notes:
> > > > - Keep people engaged in reading about Cordova
> > > > - Make people want the new thing
> > > >
> > > > I think the best way to communicate release notes is to do it within
> > the
> > > > release blog posts.
> > > >
> > > > The release blog post is much much easier to write if there's
> already a
> > > > list of release notes to draw from.
> > > >
> > >
> > > I don't know why the person writing the blog post can't go off the
> > > commit logs.  This shouldn't be tied to the release, and I feel that
> > > this is unnecessary cruft that only serves a marketing purpose, and
> > > not a technical one.  Our average release has less than 50 commits, I
> > > don't really think it's appropriate to curate these commits,
> > > especially since I have a different opinion about which commits are
> > > important to curate than other people.  I really wish this was never
> > > added to our release process, and I really would like to see this file
> > > go away.
> > >
> > >
> > > >
> > > >
> > > > On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
> > > >
> > > >> I'm -1 on duplication of git log in a text file too. Unnecessary
> > cruft.
> > > >> On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
> > > >>
> > > >> > Honestly, I would rather have the log have all the commits but
> still
> > > >> > exist as opposed to not existing, which is what happened with
> 3.2.0.
> > > >> > I think that having this RELEASENOTES.md file slows down the
> release
> > > >> > process a LOT and there's very little gain except for people who
> > can't
> > > >> > read a git log.
> > > >> >
> > > >> > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill <
> > stevengil...@gmail.com>
> > > >> > wrote:
> > > >> > > It can be automated, but the output should be manually looked
> over
> > > to
> > > >> > remove
> > > >> > > commits that don't need to be in the release notes and blog
> posts
> > > >> > >
> > > >> > >
> > > >> > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser 
> > > wrote:
> > > >> > >>
> > > >> > >> OK, I'm filing a ticket because THIS should be automated.  This
> > is
> > > way
> > > >> > >> trickier than just tagging something.
> > > >> > >>
> > > >> > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill <
> > > stevengil...@gmail.com>
> > > >> > >> wrote:
> > > >> > >> > Hey Joe,
> > > >> > >> >
> > > >> > >> > Check out Documentation to Update section at
> > > >> > >> > http://wiki.apache.org/cordova/CuttingReleases
> > > >> > >> >
> > > >> > >> > Pretty much
> > > >> > >> >
> > > >> > >> > git log --pretty=format:'* %s' --topo-order --no-merges
> > > >> > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
> > > >> > >> >
> > > >> > >> >
> > > >> > >> >
> > > >> > >> >
> > > >> > >> >
> > > >> > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser <
> bows...@gmail.com
> > >
> > > >> > wrote:
> > > >> > >> >
> > > >> > >> >> Hey
> > > >> > >> >>
> > > >> > >> >> How are the RELEASENOTES.md get authored? Where do the
> author
> > > >> counts
> > > >> > >> >> come from?  I could do this by hand, but this doesn't look
> > like
> > > >> they
> > > >> > >> >> were.  Is this documented anywhere?
> > > >> > >> >>
> > > >> > >> >> Joe
> > > >> > >> >>
> > > >> > >
> > > >> > >
> > > >> >
> > > >>
> > >
> >
>


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Axel Nennker
Actually this is not only for enterprise apps.
The way I implemented the first version is so that _only_ this cert is
trusted but no other.
If you have an app talking to your backend you might want to pin the SSL
connection to certs shipped with the app.
If you renew the cert you would add the new cert to the list of trusted
certs.

Using self-signed certs is only one use case.

Sorry for the misleading subject line of my initial email.
Maybe this should be renamed to "Allow app to specify trusted certs for
FileTransfer".

To the resources discussion: A plugin is not the solution. I want to
provide parameters to a plugin. I think we can not assume that app
developers want to write plugins they want to use them (with parameters).
An ugly "solution" would be to put the cert(s) into config.xml

--- BEGIN CERTIFICATE -
MII


Axel


2013/12/11 Andrew Grieve 

> Yeah, plugins should be able to have resource files.  might be a
> better tag name.
>
> I actually like the idea of app-level self-signed certs. For many, putting
> certs on device is too much trouble to be feasible. If you're building an
> enterprise app, putting the cert in your app makes sense.
>
> Would be good if it could be done app-wide instead of on a per-plugin basis
> though.
>
>
> On Wed, Dec 11, 2013 at 9:39 AM, Ian Clelland  >wrote:
>
> > On Wed, Dec 11, 2013 at 9:36 AM, Ian Clelland  > >wrote:
> >
> > > As to the other question, for adding arbitrary resources, the best
> route
> > > would probably be to create a plugin that ships with the application,
> > that
> > > includes that file. Plugins should have the ability to place arbitrary
> > > files, while I think applications currently do not.
> > >
> >
> > I just looked through plugman, and at least the android handler doesn't
> do
> > this.  necessarily goes to /src/,  necessarily
> goes
> > to /libs/, and there are no other options.
> >
> > :(
> >
> > What do we think of adding something like  to the plugin
> spec?
> > Or to the top-level {config|app}.xml, so that devs can do this at the
> > application level?
> >
> > Ian
> >
> >
> > >
> > >
> > > On Wed, Dec 11, 2013 at 4:27 AM,  wrote:
> > >
> > >> Hi,
> > >>
> > >>
> > >>
> > >> I think that the trustAllHosts parameter in FileTransfer is convenient
> > >> but evil.
> > >>
> > >>
> > >>
> > >> I propose to add support for self-signed certs to FileTransfer.
> > >>
> > >> There does NOT seem to be an open issue for this on Jira:
> > >>
> > >>
> > >>
> >
> https://issues.apache.org/jira/browse/CB-3576?jql=project%20%3D%20CB%20AND%20text%20~%20self-signed
> > >>
> > >>
> > >>
> > >> Do you think that this is a good/needed/superfluous effort?
> > >>
> > >> Should I create an issue?
> > >>
> > >>
> > >>
> > >> I created an initial Android implementation here:
> > >>
> > >> https://github.com/AxelNennker/cordova-plugin-file-transfer
> > >>
> > >>
> > >>
> > >> Usage would be as follows:
> > >>
> > >> - Add a line like this to www/config.xml
> > >>
> > >>  > >> value="cacerts.pem" />
> > >>
> > >>
> > >>
> > >> Put the cert into a file and store it into the projects asset
> directory
> > >>
> > >> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ ls -l
> > >> platforms/android/assets/
> > >>
> > >> insgesamt 8
> > >>
> > >> -rwxrwxrwx 1 root root 1310 Dez 10 21:46 cacerts.pem
> > >>
> > >> drwxrwxrwx 1 root root 4096 Dez 10 22:51 www
> > >>
> > >> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ cat
> > >> platforms/android/assets/cacerts.pem
> > >>
> > >> -BEGIN CERTIFICATE-
> > >>
> > >> MIIDizCCAnOgAwIBAgIJAPh3RC7M50KHMA0GCSqGSIb3DQEBBQUAMFwxCzAJBgNV
> > >>
> > >> BAYTAkRFMQ8wDQYDVQQIDAZCZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UE
> > >>
> > >> CgwOVC1MYWJzIE5lbm5rZXIxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xMzEyMTAy
> > >>
> > >> MDQyNTFaFw0xNDEyMTAyMDQyNTFaMFwxCzAJBgNVBAYTAkRFMQ8wDQYDVQQIDAZC
> > >>
> > >> ZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UECgwOVC1MYWJzIE5lbm5rZXIx
> > >>
> > >> EjAQBgNVBAMMCWxvY2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoC
> > >>
> > >> ggEBANmBMf9r86HiavL10WzVpbBt03JY9JUppwY1BZKtjejw9b7tLypgueBAGBSB
> > >>
> > >> jpb5KoaIj01xkGMCySK15JNOgwf2U3zbgmt/IBg5Vk3tBxvGr9i2S3wqmJgeXpcb
> > >>
> > >> 8mAgkf/93l2/a/anftwIE6XfmWZqy4nxdX+AZ9hKnyfZs2bW1zxzydOlcFH3NB2e
> > >>
> > >> LXWFbYOtTt4jrP9VIM4FL1I+q9RVi5zjNLWayLsEpIRtD9h/kZ5A4S+r16XYvzqp
> > >>
> > >> vec5oJ+bOEs7kA2FtFzFeZXASxLd+5Zlx0j/uB0J+iY9y0TOWTcePRJV+d8nQB5p
> > >>
> > >> rpQ9zBxHiLJmnO+0sn6N1/FnomUCAwEAAaNQME4wHQYDVR0OBBYEFCXon1DQdFzw
> > >>
> > >> URRTzdQGWQ2I0abMMB8GA1UdIwQYMBaAFCXon1DQdFzwURRTzdQGWQ2I0abMMAwG
> > >>
> > >> A1UdEwQFMAMBAf8wDQYJKoZIhvcNAQEFBQADggEBAM17byCO4c/wiDgLYHywM6jd
> > >>
> > >> fsNP8FL+Bm6Itv1s6mNEQELR8XdQJHGB+TU3WAZk8avXNvFzPwpEN3f/QvAQjN7l
> > >>
> > >> noXT+rRkRHJKALSZe2AfBDP7WX9ibk5Vnm3PVnQa46f8vbvuxVEUu5e1YKcR1kXe
> > >>
> > >> booucDCp1g3vc9K6tbSyKS04CA+ys6pKHpqlxxVqSxe+1Kii3CeBLVcDXYrQvI22
> > >>
> > >> bi7xIMmV5njur+vkJKaHZtM6LuGe0k5Yw881/CBHCf7Vg3iZt8cqXC1OUo4qLV30
> > >>
> > >> 0x+a

Re: 3.3.0 Release

2013-12-11 Thread David Barth

Le 10/12/2013 19:37, Steven Gill a écrit :

Lets start the final round of tagging today!
https://issues.apache.org/jira/browse/CB-5538
+1, but we're still chasing down some dependency and chmod +x issues on 
the Ubuntu part. Will update this thread shortly.


David


Re: 3.3.0 Release

2013-12-11 Thread James Jong
+1 FYI, I have been testing Cordova with iOS 7.1 beta.  No new issues with it.
-James Jong

On Dec 10, 2013, at 1:40 PM, Joe Bowser  wrote:

> SGTM!
> 
> On Tue, Dec 10, 2013 at 10:37 AM, Steven Gill  wrote:
>> Lets start the final round of tagging today!
>> https://issues.apache.org/jira/browse/CB-5538



Re: Benchmark comparison of classic webview and Chromium webview

2013-12-11 Thread Joe Bowser
On Wed, Dec 11, 2013 at 7:03 AM, Michal Mocny  wrote:
> TLDR before we get too excited: Js perf seems a huge win, but at the moment
> html canvas is not gpu accelerated, so canvas based apps are actually not
> doing so well.  This is known and high priority for chrome-webview team,
> though I am not at all sure if it can be fixed in a minor patch of android
> 4.4.

Goodbye old broken webview, hello new slightly less broken webview.
We knew that things would be perfect, but there's some definite
improvements, which shouldn't be understated.  Also, I would consider
KitKat to be rushed, since there's a lot of other even more broken
things happening outside the webview (like gallery encoding
errors,etc).

> There are several crbugs going into specific details and user complaints if
> you want to read them (the comments section of the above blog post is a
> good start).
>

We've encountered bugs with JQueryMobile (which we can't submit back
to Chromium, because saying JQueryMobile broke your shit isn't
helpful), and bugs with text fields and the onchange event on text
fields not working due to some broken autocomplete code so far (which
I have submitted) .  These break any sort of input validation done in
JS.  This Chrome is better, but there's still some serious breakage,
and I hope that Chrome can get updated in a minor release like 4.4.3
and not have to wait for 4.5 or 5.0 to be brought up a few versions.

> If you want to have a canvas based app in 4.4 that doesn't completely suck,
> be very careful which what resolution your canvas is set to, perhaps opting
> to leave it in the default low-res css-pixel size mode.  This seems to be
> especially biting for cordova apps right now, since they by default run
> with targetSDK < 19, which leave chrome-webview in quirks mode where
> target-densityDpi is still respected, and apparently our cordova hello
> world templates set that wrong by default (or at least interact poorly with
> the state of 4.4 webview, I haven't finished investigating that part).
>

We don't run in quirks mode anymore, and we all knew that
target-densityDpi was going to be left by the wayside, since this was
a WebKit change that happened far before "The Great Forking".


Re: RELEASENOTES.md and coho

2013-12-11 Thread Joe Bowser
I don't think our platform release notes should be in a single blog post
and have a single author.  I really want this out of the release process
since I don't want to hold releases up fiddling with a markdown file for an
hour or more.  That's the real issue.
On 11 Dec 2013 07:11, "Andrew Grieve"  wrote:

> I'll write them for Android if it's annoying for you. I really don't think
> it's feasible for one person to make sense of all commit messages across
> all platforms.
>
> On Wed, Dec 11, 2013 at 10:02 AM, Joe Bowser  wrote:
>
> > On Wed, Dec 11, 2013 at 6:53 AM, Andrew Grieve 
> > wrote:
> > > Release notes:
> > > - Keep people engaged in reading about Cordova
> > > - Make people want the new thing
> > >
> > > I think the best way to communicate release notes is to do it within
> the
> > > release blog posts.
> > >
> > > The release blog post is much much easier to write if there's already a
> > > list of release notes to draw from.
> > >
> >
> > I don't know why the person writing the blog post can't go off the
> > commit logs.  This shouldn't be tied to the release, and I feel that
> > this is unnecessary cruft that only serves a marketing purpose, and
> > not a technical one.  Our average release has less than 50 commits, I
> > don't really think it's appropriate to curate these commits,
> > especially since I have a different opinion about which commits are
> > important to curate than other people.  I really wish this was never
> > added to our release process, and I really would like to see this file
> > go away.
> >
> >
> > >
> > >
> > > On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
> > >
> > >> I'm -1 on duplication of git log in a text file too. Unnecessary
> cruft.
> > >> On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
> > >>
> > >> > Honestly, I would rather have the log have all the commits but still
> > >> > exist as opposed to not existing, which is what happened with 3.2.0.
> > >> > I think that having this RELEASENOTES.md file slows down the release
> > >> > process a LOT and there's very little gain except for people who
> can't
> > >> > read a git log.
> > >> >
> > >> > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill <
> stevengil...@gmail.com>
> > >> > wrote:
> > >> > > It can be automated, but the output should be manually looked over
> > to
> > >> > remove
> > >> > > commits that don't need to be in the release notes and blog posts
> > >> > >
> > >> > >
> > >> > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser 
> > wrote:
> > >> > >>
> > >> > >> OK, I'm filing a ticket because THIS should be automated.  This
> is
> > way
> > >> > >> trickier than just tagging something.
> > >> > >>
> > >> > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill <
> > stevengil...@gmail.com>
> > >> > >> wrote:
> > >> > >> > Hey Joe,
> > >> > >> >
> > >> > >> > Check out Documentation to Update section at
> > >> > >> > http://wiki.apache.org/cordova/CuttingReleases
> > >> > >> >
> > >> > >> > Pretty much
> > >> > >> >
> > >> > >> > git log --pretty=format:'* %s' --topo-order --no-merges
> > >> > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
> > >> > >> >
> > >> > >> >
> > >> > >> >
> > >> > >> >
> > >> > >> >
> > >> > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser  >
> > >> > wrote:
> > >> > >> >
> > >> > >> >> Hey
> > >> > >> >>
> > >> > >> >> How are the RELEASENOTES.md get authored? Where do the author
> > >> counts
> > >> > >> >> come from?  I could do this by hand, but this doesn't look
> like
> > >> they
> > >> > >> >> were.  Is this documented anywhere?
> > >> > >> >>
> > >> > >> >> Joe
> > >> > >> >>
> > >> > >
> > >> > >
> > >> >
> > >>
> >
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Andrew Grieve
I'll write them for Android if it's annoying for you. I really don't think
it's feasible for one person to make sense of all commit messages across
all platforms.

On Wed, Dec 11, 2013 at 10:02 AM, Joe Bowser  wrote:

> On Wed, Dec 11, 2013 at 6:53 AM, Andrew Grieve 
> wrote:
> > Release notes:
> > - Keep people engaged in reading about Cordova
> > - Make people want the new thing
> >
> > I think the best way to communicate release notes is to do it within the
> > release blog posts.
> >
> > The release blog post is much much easier to write if there's already a
> > list of release notes to draw from.
> >
>
> I don't know why the person writing the blog post can't go off the
> commit logs.  This shouldn't be tied to the release, and I feel that
> this is unnecessary cruft that only serves a marketing purpose, and
> not a technical one.  Our average release has less than 50 commits, I
> don't really think it's appropriate to curate these commits,
> especially since I have a different opinion about which commits are
> important to curate than other people.  I really wish this was never
> added to our release process, and I really would like to see this file
> go away.
>
>
> >
> >
> > On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
> >
> >> I'm -1 on duplication of git log in a text file too. Unnecessary cruft.
> >> On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
> >>
> >> > Honestly, I would rather have the log have all the commits but still
> >> > exist as opposed to not existing, which is what happened with 3.2.0.
> >> > I think that having this RELEASENOTES.md file slows down the release
> >> > process a LOT and there's very little gain except for people who can't
> >> > read a git log.
> >> >
> >> > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill 
> >> > wrote:
> >> > > It can be automated, but the output should be manually looked over
> to
> >> > remove
> >> > > commits that don't need to be in the release notes and blog posts
> >> > >
> >> > >
> >> > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser 
> wrote:
> >> > >>
> >> > >> OK, I'm filing a ticket because THIS should be automated.  This is
> way
> >> > >> trickier than just tagging something.
> >> > >>
> >> > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill <
> stevengil...@gmail.com>
> >> > >> wrote:
> >> > >> > Hey Joe,
> >> > >> >
> >> > >> > Check out Documentation to Update section at
> >> > >> > http://wiki.apache.org/cordova/CuttingReleases
> >> > >> >
> >> > >> > Pretty much
> >> > >> >
> >> > >> > git log --pretty=format:'* %s' --topo-order --no-merges
> >> > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
> >> > >> >
> >> > >> >
> >> > >> >
> >> > >> >
> >> > >> >
> >> > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser 
> >> > wrote:
> >> > >> >
> >> > >> >> Hey
> >> > >> >>
> >> > >> >> How are the RELEASENOTES.md get authored? Where do the author
> >> counts
> >> > >> >> come from?  I could do this by hand, but this doesn't look like
> >> they
> >> > >> >> were.  Is this documented anywhere?
> >> > >> >>
> >> > >> >> Joe
> >> > >> >>
> >> > >
> >> > >
> >> >
> >>
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Andrew Grieve
Could be you're hitting: https://issues.apache.org/jira/browse/CB-5481


On Wed, Dec 11, 2013 at 9:44 AM, Joe Bowser  wrote:

> I haven't noticed this during any of our testing, nor have I noticed
> this when hacking with Google Glass, which uses the Google Glass voice
> trigger intent to launch Cordova, not the LAUNCHER intent like the
> launch menu.  Furthermore, our JUnit tests haven't caught the tests
> either, and they depend on at least three plugins to be loaded by
> PluginManager from config.xml.  Honestly, this only breaks if you
> don't know what you're doing with Intents.
>
> Honestly, It sounds to me like your config.xml isn't in the correct
> place in your project, especially if you're trying to do something
> complex with Android and intents.  I would check that making these
> sorts of statements.  If config.xml wasn't being read, it would have
> been caught right away, since everything is a plugin now.
>
>
>
> On Wed, Dec 11, 2013 at 4:33 AM,   wrote:
> > Hi,
> >
> > We are seeing some strange effect on Android when we move from
> 3.1.0-0.2.0 to a newer version.
> > We tried with 3.2.0-0.4.0 and 3.3-rc1
> >
> > I think that PluginManager changed and now fails to load config.xml
> >
> > This happens in my project when the app is opened by the operating
> system because it is configured so through an intent filter.
> > The original activity is not a child of DroidGap but starts the "main"
> activity using startActivity.
> >
> > The code to get the id of the config.xml file is not working in this
> case.
> >
> > Why was the 3.1.0 code changed?
> >
> > Cheers
> > Axel
> >
> > D/CordovaActivity(12320): CordovaActivity.init()
> > D/CordovaWebView(12320): >>>
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
> > D/PluginManager(12320): init()
> > E/PluginManager(12320):
> =
> > E/PluginManager(12320): ERROR: config.xml is missing.  Add
> res/xml/config.xml to your project.
> > E/PluginManager(12320):
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
> > E/PluginManager(12320):
> =
> > D/CordovaWebView(12320): >>> loadUrlNow()
> > D/ItemImport(12320): onResume() loadURL started:
>


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Andrew Grieve
Yeah, plugins should be able to have resource files.  might be a
better tag name.

I actually like the idea of app-level self-signed certs. For many, putting
certs on device is too much trouble to be feasible. If you're building an
enterprise app, putting the cert in your app makes sense.

Would be good if it could be done app-wide instead of on a per-plugin basis
though.


On Wed, Dec 11, 2013 at 9:39 AM, Ian Clelland wrote:

> On Wed, Dec 11, 2013 at 9:36 AM, Ian Clelland  >wrote:
>
> > As to the other question, for adding arbitrary resources, the best route
> > would probably be to create a plugin that ships with the application,
> that
> > includes that file. Plugins should have the ability to place arbitrary
> > files, while I think applications currently do not.
> >
>
> I just looked through plugman, and at least the android handler doesn't do
> this.  necessarily goes to /src/,  necessarily goes
> to /libs/, and there are no other options.
>
> :(
>
> What do we think of adding something like  to the plugin spec?
> Or to the top-level {config|app}.xml, so that devs can do this at the
> application level?
>
> Ian
>
>
> >
> >
> > On Wed, Dec 11, 2013 at 4:27 AM,  wrote:
> >
> >> Hi,
> >>
> >>
> >>
> >> I think that the trustAllHosts parameter in FileTransfer is convenient
> >> but evil.
> >>
> >>
> >>
> >> I propose to add support for self-signed certs to FileTransfer.
> >>
> >> There does NOT seem to be an open issue for this on Jira:
> >>
> >>
> >>
> https://issues.apache.org/jira/browse/CB-3576?jql=project%20%3D%20CB%20AND%20text%20~%20self-signed
> >>
> >>
> >>
> >> Do you think that this is a good/needed/superfluous effort?
> >>
> >> Should I create an issue?
> >>
> >>
> >>
> >> I created an initial Android implementation here:
> >>
> >> https://github.com/AxelNennker/cordova-plugin-file-transfer
> >>
> >>
> >>
> >> Usage would be as follows:
> >>
> >> - Add a line like this to www/config.xml
> >>
> >>  >> value="cacerts.pem" />
> >>
> >>
> >>
> >> Put the cert into a file and store it into the projects asset directory
> >>
> >> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ ls -l
> >> platforms/android/assets/
> >>
> >> insgesamt 8
> >>
> >> -rwxrwxrwx 1 root root 1310 Dez 10 21:46 cacerts.pem
> >>
> >> drwxrwxrwx 1 root root 4096 Dez 10 22:51 www
> >>
> >> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ cat
> >> platforms/android/assets/cacerts.pem
> >>
> >> -BEGIN CERTIFICATE-
> >>
> >> MIIDizCCAnOgAwIBAgIJAPh3RC7M50KHMA0GCSqGSIb3DQEBBQUAMFwxCzAJBgNV
> >>
> >> BAYTAkRFMQ8wDQYDVQQIDAZCZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UE
> >>
> >> CgwOVC1MYWJzIE5lbm5rZXIxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xMzEyMTAy
> >>
> >> MDQyNTFaFw0xNDEyMTAyMDQyNTFaMFwxCzAJBgNVBAYTAkRFMQ8wDQYDVQQIDAZC
> >>
> >> ZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UECgwOVC1MYWJzIE5lbm5rZXIx
> >>
> >> EjAQBgNVBAMMCWxvY2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoC
> >>
> >> ggEBANmBMf9r86HiavL10WzVpbBt03JY9JUppwY1BZKtjejw9b7tLypgueBAGBSB
> >>
> >> jpb5KoaIj01xkGMCySK15JNOgwf2U3zbgmt/IBg5Vk3tBxvGr9i2S3wqmJgeXpcb
> >>
> >> 8mAgkf/93l2/a/anftwIE6XfmWZqy4nxdX+AZ9hKnyfZs2bW1zxzydOlcFH3NB2e
> >>
> >> LXWFbYOtTt4jrP9VIM4FL1I+q9RVi5zjNLWayLsEpIRtD9h/kZ5A4S+r16XYvzqp
> >>
> >> vec5oJ+bOEs7kA2FtFzFeZXASxLd+5Zlx0j/uB0J+iY9y0TOWTcePRJV+d8nQB5p
> >>
> >> rpQ9zBxHiLJmnO+0sn6N1/FnomUCAwEAAaNQME4wHQYDVR0OBBYEFCXon1DQdFzw
> >>
> >> URRTzdQGWQ2I0abMMB8GA1UdIwQYMBaAFCXon1DQdFzwURRTzdQGWQ2I0abMMAwG
> >>
> >> A1UdEwQFMAMBAf8wDQYJKoZIhvcNAQEFBQADggEBAM17byCO4c/wiDgLYHywM6jd
> >>
> >> fsNP8FL+Bm6Itv1s6mNEQELR8XdQJHGB+TU3WAZk8avXNvFzPwpEN3f/QvAQjN7l
> >>
> >> noXT+rRkRHJKALSZe2AfBDP7WX9ibk5Vnm3PVnQa46f8vbvuxVEUu5e1YKcR1kXe
> >>
> >> booucDCp1g3vc9K6tbSyKS04CA+ys6pKHpqlxxVqSxe+1Kii3CeBLVcDXYrQvI22
> >>
> >> bi7xIMmV5njur+vkJKaHZtM6LuGe0k5Yw881/CBHCf7Vg3iZt8cqXC1OUo4qLV30
> >>
> >> 0x+alWlFoz9xtiKcfiaQwQL8O3Uip9VoK52XFGbi9yiTqTDuc1zYpHX8V/cbS3U=
> >>
> >> -END CERTIFICATE-
> >>
> >> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$
> >>
> >>
> >>
> >> The implementation is here:
> >>
> >>
> >>
> https://github.com/AxelNennker/cordova-plugin-file-transfer/blob/master/src/android/FileTransfer.java#L100
> >>
> >>
> >>
> >> I have a general question: How does an application programmer (cordova
> >> user) specify resources to be added to the project.
> >>
> >> I think that merges could be a way to do this but this always merges
> >> files into the platforms www directory.
> >>
> >> On Android this is platforms/android/assets/www but I don't want the
> file
> >> there.
> >>
> >>
> >>
> >> Cheers
> >>
> >> Axel
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >
>


Re: Benchmark comparison of classic webview and Chromium webview

2013-12-11 Thread Michal Mocny
TLDR before we get too excited: Js perf seems a huge win, but at the moment
html canvas is not gpu accelerated, so canvas based apps are actually not
doing so well.  This is known and high priority for chrome-webview team,
though I am not at all sure if it can be fixed in a minor patch of android
4.4.

There are several crbugs going into specific details and user complaints if
you want to read them (the comments section of the above blog post is a
good start).

If you want to have a canvas based app in 4.4 that doesn't completely suck,
be very careful which what resolution your canvas is set to, perhaps opting
to leave it in the default low-res css-pixel size mode.  This seems to be
especially biting for cordova apps right now, since they by default run
with targetSDK < 19, which leave chrome-webview in quirks mode where
target-densityDpi is still respected, and apparently our cordova hello
world templates set that wrong by default (or at least interact poorly with
the state of 4.4 webview, I haven't finished investigating that part).


On Tue, Dec 10, 2013 at 11:11 PM, Brian LeRoux  wrote:

> such good news =)
>
>
> On Wed, Dec 11, 2013 at 1:17 AM, Marcel Kinard  wrote:
>
> > I thought this would be of interest to the Android folks here.
> >
> > https://www.timroes.de/2013/11/23/old-webview-vs-chromium-webview/
> >
> >
> >
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Joe Bowser
On Wed, Dec 11, 2013 at 6:53 AM, Andrew Grieve  wrote:
> Release notes:
> - Keep people engaged in reading about Cordova
> - Make people want the new thing
>
> I think the best way to communicate release notes is to do it within the
> release blog posts.
>
> The release blog post is much much easier to write if there's already a
> list of release notes to draw from.
>

I don't know why the person writing the blog post can't go off the
commit logs.  This shouldn't be tied to the release, and I feel that
this is unnecessary cruft that only serves a marketing purpose, and
not a technical one.  Our average release has less than 50 commits, I
don't really think it's appropriate to curate these commits,
especially since I have a different opinion about which commits are
important to curate than other people.  I really wish this was never
added to our release process, and I really would like to see this file
go away.


>
>
> On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
>
>> I'm -1 on duplication of git log in a text file too. Unnecessary cruft.
>> On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
>>
>> > Honestly, I would rather have the log have all the commits but still
>> > exist as opposed to not existing, which is what happened with 3.2.0.
>> > I think that having this RELEASENOTES.md file slows down the release
>> > process a LOT and there's very little gain except for people who can't
>> > read a git log.
>> >
>> > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill 
>> > wrote:
>> > > It can be automated, but the output should be manually looked over to
>> > remove
>> > > commits that don't need to be in the release notes and blog posts
>> > >
>> > >
>> > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser  wrote:
>> > >>
>> > >> OK, I'm filing a ticket because THIS should be automated.  This is way
>> > >> trickier than just tagging something.
>> > >>
>> > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill 
>> > >> wrote:
>> > >> > Hey Joe,
>> > >> >
>> > >> > Check out Documentation to Update section at
>> > >> > http://wiki.apache.org/cordova/CuttingReleases
>> > >> >
>> > >> > Pretty much
>> > >> >
>> > >> > git log --pretty=format:'* %s' --topo-order --no-merges
>> > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
>> > >> >
>> > >> >
>> > >> >
>> > >> >
>> > >> >
>> > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser 
>> > wrote:
>> > >> >
>> > >> >> Hey
>> > >> >>
>> > >> >> How are the RELEASENOTES.md get authored? Where do the author
>> counts
>> > >> >> come from?  I could do this by hand, but this doesn't look like
>> they
>> > >> >> were.  Is this documented anywhere?
>> > >> >>
>> > >> >> Joe
>> > >> >>
>> > >
>> > >
>> >
>>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Joe Bowser
On Wed, Dec 11, 2013 at 6:54 AM, Michal Mocny  wrote:
> Also, for those downloading the zip, they cannot git log, right?

Anyone can visit this:
https://git-wip-us.apache.org/repos/asf?p=cordova-android.git

If they can't do so, then how are they downloading the zip in the first place?

>
>
> On Wed, Dec 11, 2013 at 9:53 AM, Andrew Grieve  wrote:
>
>> Release notes:
>> - Keep people engaged in reading about Cordova
>> - Make people want the new thing
>>
>> I think the best way to communicate release notes is to do it within the
>> release blog posts.
>>
>> The release blog post is much much easier to write if there's already a
>> list of release notes to draw from.
>>
>>
>>
>> On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
>>
>> > I'm -1 on duplication of git log in a text file too. Unnecessary cruft.
>> > On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
>> >
>> > > Honestly, I would rather have the log have all the commits but still
>> > > exist as opposed to not existing, which is what happened with 3.2.0.
>> > > I think that having this RELEASENOTES.md file slows down the release
>> > > process a LOT and there's very little gain except for people who can't
>> > > read a git log.
>> > >
>> > > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill 
>> > > wrote:
>> > > > It can be automated, but the output should be manually looked over to
>> > > remove
>> > > > commits that don't need to be in the release notes and blog posts
>> > > >
>> > > >
>> > > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser 
>> wrote:
>> > > >>
>> > > >> OK, I'm filing a ticket because THIS should be automated.  This is
>> way
>> > > >> trickier than just tagging something.
>> > > >>
>> > > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill <
>> stevengil...@gmail.com>
>> > > >> wrote:
>> > > >> > Hey Joe,
>> > > >> >
>> > > >> > Check out Documentation to Update section at
>> > > >> > http://wiki.apache.org/cordova/CuttingReleases
>> > > >> >
>> > > >> > Pretty much
>> > > >> >
>> > > >> > git log --pretty=format:'* %s' --topo-order --no-merges
>> > > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
>> > > >> >
>> > > >> >
>> > > >> >
>> > > >> >
>> > > >> >
>> > > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser 
>> > > wrote:
>> > > >> >
>> > > >> >> Hey
>> > > >> >>
>> > > >> >> How are the RELEASENOTES.md get authored? Where do the author
>> > counts
>> > > >> >> come from?  I could do this by hand, but this doesn't look like
>> > they
>> > > >> >> were.  Is this documented anywhere?
>> > > >> >>
>> > > >> >> Joe
>> > > >> >>
>> > > >
>> > > >
>> > >
>> >
>>


Re: review request for cordova-js gruntfile refactor

2013-12-11 Thread Andrew Grieve
Brian - the 8 commits change is pretty overwhelming to look at all at once.
Could you split them up into those that you want looked at, and those that
you don't (e.g. lib->src & splitting gruntfile into separate files don't
really need to be looked at).


On Tue, Dec 10, 2013 at 11:04 PM, Brian LeRoux  wrote:

> https://github.com/apache/cordova-js/pull/61
>
> In my quest to add the browser platform I got a little sidetracked cleaning
> up the gruntfile. Please have a look. The _btest fails currently but I
> think I want to move to phantom anyhow. Otherwise everything is identical.
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Michal Mocny
Also, for those downloading the zip, they cannot git log, right?


On Wed, Dec 11, 2013 at 9:53 AM, Andrew Grieve  wrote:

> Release notes:
> - Keep people engaged in reading about Cordova
> - Make people want the new thing
>
> I think the best way to communicate release notes is to do it within the
> release blog posts.
>
> The release blog post is much much easier to write if there's already a
> list of release notes to draw from.
>
>
>
> On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:
>
> > I'm -1 on duplication of git log in a text file too. Unnecessary cruft.
> > On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
> >
> > > Honestly, I would rather have the log have all the commits but still
> > > exist as opposed to not existing, which is what happened with 3.2.0.
> > > I think that having this RELEASENOTES.md file slows down the release
> > > process a LOT and there's very little gain except for people who can't
> > > read a git log.
> > >
> > > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill 
> > > wrote:
> > > > It can be automated, but the output should be manually looked over to
> > > remove
> > > > commits that don't need to be in the release notes and blog posts
> > > >
> > > >
> > > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser 
> wrote:
> > > >>
> > > >> OK, I'm filing a ticket because THIS should be automated.  This is
> way
> > > >> trickier than just tagging something.
> > > >>
> > > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill <
> stevengil...@gmail.com>
> > > >> wrote:
> > > >> > Hey Joe,
> > > >> >
> > > >> > Check out Documentation to Update section at
> > > >> > http://wiki.apache.org/cordova/CuttingReleases
> > > >> >
> > > >> > Pretty much
> > > >> >
> > > >> > git log --pretty=format:'* %s' --topo-order --no-merges
> > > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
> > > >> >
> > > >> >
> > > >> >
> > > >> >
> > > >> >
> > > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser 
> > > wrote:
> > > >> >
> > > >> >> Hey
> > > >> >>
> > > >> >> How are the RELEASENOTES.md get authored? Where do the author
> > counts
> > > >> >> come from?  I could do this by hand, but this doesn't look like
> > they
> > > >> >> were.  Is this documented anywhere?
> > > >> >>
> > > >> >> Joe
> > > >> >>
> > > >
> > > >
> > >
> >
>


Re: RELEASENOTES.md and coho

2013-12-11 Thread Andrew Grieve
Release notes:
- Keep people engaged in reading about Cordova
- Make people want the new thing

I think the best way to communicate release notes is to do it within the
release blog posts.

The release blog post is much much easier to write if there's already a
list of release notes to draw from.



On Tue, Dec 10, 2013 at 7:35 PM, Brian LeRoux  wrote:

> I'm -1 on duplication of git log in a text file too. Unnecessary cruft.
> On Dec 11, 2013 9:03 AM, "Joe Bowser"  wrote:
>
> > Honestly, I would rather have the log have all the commits but still
> > exist as opposed to not existing, which is what happened with 3.2.0.
> > I think that having this RELEASENOTES.md file slows down the release
> > process a LOT and there's very little gain except for people who can't
> > read a git log.
> >
> > On Tue, Dec 10, 2013 at 2:49 PM, Steven Gill 
> > wrote:
> > > It can be automated, but the output should be manually looked over to
> > remove
> > > commits that don't need to be in the release notes and blog posts
> > >
> > >
> > > On Tue, Dec 10, 2013 at 2:45 PM, Joe Bowser  wrote:
> > >>
> > >> OK, I'm filing a ticket because THIS should be automated.  This is way
> > >> trickier than just tagging something.
> > >>
> > >> On Tue, Dec 10, 2013 at 2:31 PM, Steven Gill 
> > >> wrote:
> > >> > Hey Joe,
> > >> >
> > >> > Check out Documentation to Update section at
> > >> > http://wiki.apache.org/cordova/CuttingReleases
> > >> >
> > >> > Pretty much
> > >> >
> > >> > git log --pretty=format:'* %s' --topo-order --no-merges
> > >> > origin/RELEASE_BRANCH..master >> RELEASENOTES.md
> > >> >
> > >> >
> > >> >
> > >> >
> > >> >
> > >> > On Tue, Dec 10, 2013 at 2:15 PM, Joe Bowser 
> > wrote:
> > >> >
> > >> >> Hey
> > >> >>
> > >> >> How are the RELEASENOTES.md get authored? Where do the author
> counts
> > >> >> come from?  I could do this by hand, but this doesn't look like
> they
> > >> >> were.  Is this documented anywhere?
> > >> >>
> > >> >> Joe
> > >> >>
> > >
> > >
> >
>


Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Joe Bowser
I haven't noticed this during any of our testing, nor have I noticed
this when hacking with Google Glass, which uses the Google Glass voice
trigger intent to launch Cordova, not the LAUNCHER intent like the
launch menu.  Furthermore, our JUnit tests haven't caught the tests
either, and they depend on at least three plugins to be loaded by
PluginManager from config.xml.  Honestly, this only breaks if you
don't know what you're doing with Intents.

Honestly, It sounds to me like your config.xml isn't in the correct
place in your project, especially if you're trying to do something
complex with Android and intents.  I would check that making these
sorts of statements.  If config.xml wasn't being read, it would have
been caught right away, since everything is a plugin now.



On Wed, Dec 11, 2013 at 4:33 AM,   wrote:
> Hi,
>
> We are seeing some strange effect on Android when we move from 3.1.0-0.2.0 to 
> a newer version.
> We tried with 3.2.0-0.4.0 and 3.3-rc1
>
> I think that PluginManager changed and now fails to load config.xml
>
> This happens in my project when the app is opened by the operating system 
> because it is configured so through an intent filter.
> The original activity is not a child of DroidGap but starts the "main" 
> activity using startActivity.
>
> The code to get the id of the config.xml file is not working in this case.
>
> Why was the 3.1.0 code changed?
>
> Cheers
> Axel
>
> D/CordovaActivity(12320): CordovaActivity.init()
> D/CordovaWebView(12320): >>> 
> loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
> D/PluginManager(12320): init()
> E/PluginManager(12320): 
> =
> E/PluginManager(12320): ERROR: config.xml is missing.  Add res/xml/config.xml 
> to your project.
> E/PluginManager(12320): 
> https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
> E/PluginManager(12320): 
> =
> D/CordovaWebView(12320): >>> loadUrlNow()
> D/ItemImport(12320): onResume() loadURL started:


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Ian Clelland
On Wed, Dec 11, 2013 at 9:36 AM, Ian Clelland wrote:

> As to the other question, for adding arbitrary resources, the best route
> would probably be to create a plugin that ships with the application, that
> includes that file. Plugins should have the ability to place arbitrary
> files, while I think applications currently do not.
>

I just looked through plugman, and at least the android handler doesn't do
this.  necessarily goes to /src/,  necessarily goes
to /libs/, and there are no other options.

:(

What do we think of adding something like  to the plugin spec?
Or to the top-level {config|app}.xml, so that devs can do this at the
application level?

Ian


>
>
> On Wed, Dec 11, 2013 at 4:27 AM,  wrote:
>
>> Hi,
>>
>>
>>
>> I think that the trustAllHosts parameter in FileTransfer is convenient
>> but evil.
>>
>>
>>
>> I propose to add support for self-signed certs to FileTransfer.
>>
>> There does NOT seem to be an open issue for this on Jira:
>>
>>
>> https://issues.apache.org/jira/browse/CB-3576?jql=project%20%3D%20CB%20AND%20text%20~%20self-signed
>>
>>
>>
>> Do you think that this is a good/needed/superfluous effort?
>>
>> Should I create an issue?
>>
>>
>>
>> I created an initial Android implementation here:
>>
>> https://github.com/AxelNennker/cordova-plugin-file-transfer
>>
>>
>>
>> Usage would be as follows:
>>
>> - Add a line like this to www/config.xml
>>
>> > value="cacerts.pem" />
>>
>>
>>
>> Put the cert into a file and store it into the projects asset directory
>>
>> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ ls -l
>> platforms/android/assets/
>>
>> insgesamt 8
>>
>> -rwxrwxrwx 1 root root 1310 Dez 10 21:46 cacerts.pem
>>
>> drwxrwxrwx 1 root root 4096 Dez 10 22:51 www
>>
>> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ cat
>> platforms/android/assets/cacerts.pem
>>
>> -BEGIN CERTIFICATE-
>>
>> MIIDizCCAnOgAwIBAgIJAPh3RC7M50KHMA0GCSqGSIb3DQEBBQUAMFwxCzAJBgNV
>>
>> BAYTAkRFMQ8wDQYDVQQIDAZCZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UE
>>
>> CgwOVC1MYWJzIE5lbm5rZXIxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xMzEyMTAy
>>
>> MDQyNTFaFw0xNDEyMTAyMDQyNTFaMFwxCzAJBgNVBAYTAkRFMQ8wDQYDVQQIDAZC
>>
>> ZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UECgwOVC1MYWJzIE5lbm5rZXIx
>>
>> EjAQBgNVBAMMCWxvY2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoC
>>
>> ggEBANmBMf9r86HiavL10WzVpbBt03JY9JUppwY1BZKtjejw9b7tLypgueBAGBSB
>>
>> jpb5KoaIj01xkGMCySK15JNOgwf2U3zbgmt/IBg5Vk3tBxvGr9i2S3wqmJgeXpcb
>>
>> 8mAgkf/93l2/a/anftwIE6XfmWZqy4nxdX+AZ9hKnyfZs2bW1zxzydOlcFH3NB2e
>>
>> LXWFbYOtTt4jrP9VIM4FL1I+q9RVi5zjNLWayLsEpIRtD9h/kZ5A4S+r16XYvzqp
>>
>> vec5oJ+bOEs7kA2FtFzFeZXASxLd+5Zlx0j/uB0J+iY9y0TOWTcePRJV+d8nQB5p
>>
>> rpQ9zBxHiLJmnO+0sn6N1/FnomUCAwEAAaNQME4wHQYDVR0OBBYEFCXon1DQdFzw
>>
>> URRTzdQGWQ2I0abMMB8GA1UdIwQYMBaAFCXon1DQdFzwURRTzdQGWQ2I0abMMAwG
>>
>> A1UdEwQFMAMBAf8wDQYJKoZIhvcNAQEFBQADggEBAM17byCO4c/wiDgLYHywM6jd
>>
>> fsNP8FL+Bm6Itv1s6mNEQELR8XdQJHGB+TU3WAZk8avXNvFzPwpEN3f/QvAQjN7l
>>
>> noXT+rRkRHJKALSZe2AfBDP7WX9ibk5Vnm3PVnQa46f8vbvuxVEUu5e1YKcR1kXe
>>
>> booucDCp1g3vc9K6tbSyKS04CA+ys6pKHpqlxxVqSxe+1Kii3CeBLVcDXYrQvI22
>>
>> bi7xIMmV5njur+vkJKaHZtM6LuGe0k5Yw881/CBHCf7Vg3iZt8cqXC1OUo4qLV30
>>
>> 0x+alWlFoz9xtiKcfiaQwQL8O3Uip9VoK52XFGbi9yiTqTDuc1zYpHX8V/cbS3U=
>>
>> -END CERTIFICATE-
>>
>> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$
>>
>>
>>
>> The implementation is here:
>>
>>
>> https://github.com/AxelNennker/cordova-plugin-file-transfer/blob/master/src/android/FileTransfer.java#L100
>>
>>
>>
>> I have a general question: How does an application programmer (cordova
>> user) specify resources to be added to the project.
>>
>> I think that merges could be a way to do this but this always merges
>> files into the platforms www directory.
>>
>> On Android this is platforms/android/assets/www but I don't want the file
>> there.
>>
>>
>>
>> Cheers
>>
>> Axel
>>
>>
>>
>>
>>
>>
>>
>


Re: Support self-signed certs in FileTransfer

2013-12-11 Thread Ian Clelland
There was some talk on the list a couple months ago about this -- not  for
file-transfer specifically, but the general idea of supporting custom
certificates, or CAs in Cordova.

I think that, after a number of emails, we concluded that for users who
have legitimate custom certificate requirements, that there should be
os-policy-level mechanisms for adding custom certs, and that the individual
application was the wrong level to be managing them. (Somebody please step
in and tell me if I'm misremembering this)

As to the other question, for adding arbitrary resources, the best route
would probably be to create a plugin that ships with the application, that
includes that file. Plugins should have the ability to place arbitrary
files, while I think applications currently do not.


On Wed, Dec 11, 2013 at 4:27 AM,  wrote:

> Hi,
>
>
>
> I think that the trustAllHosts parameter in FileTransfer is convenient but
> evil.
>
>
>
> I propose to add support for self-signed certs to FileTransfer.
>
> There does NOT seem to be an open issue for this on Jira:
>
>
> https://issues.apache.org/jira/browse/CB-3576?jql=project%20%3D%20CB%20AND%20text%20~%20self-signed
>
>
>
> Do you think that this is a good/needed/superfluous effort?
>
> Should I create an issue?
>
>
>
> I created an initial Android implementation here:
>
> https://github.com/AxelNennker/cordova-plugin-file-transfer
>
>
>
> Usage would be as follows:
>
> - Add a line like this to www/config.xml
>
>  value="cacerts.pem" />
>
>
>
> Put the cert into a file and store it into the projects asset directory
>
> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ ls -l
> platforms/android/assets/
>
> insgesamt 8
>
> -rwxrwxrwx 1 root root 1310 Dez 10 21:46 cacerts.pem
>
> drwxrwxrwx 1 root root 4096 Dez 10 22:51 www
>
> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ cat
> platforms/android/assets/cacerts.pem
>
> -BEGIN CERTIFICATE-
>
> MIIDizCCAnOgAwIBAgIJAPh3RC7M50KHMA0GCSqGSIb3DQEBBQUAMFwxCzAJBgNV
>
> BAYTAkRFMQ8wDQYDVQQIDAZCZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UE
>
> CgwOVC1MYWJzIE5lbm5rZXIxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xMzEyMTAy
>
> MDQyNTFaFw0xNDEyMTAyMDQyNTFaMFwxCzAJBgNVBAYTAkRFMQ8wDQYDVQQIDAZC
>
> ZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UECgwOVC1MYWJzIE5lbm5rZXIx
>
> EjAQBgNVBAMMCWxvY2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoC
>
> ggEBANmBMf9r86HiavL10WzVpbBt03JY9JUppwY1BZKtjejw9b7tLypgueBAGBSB
>
> jpb5KoaIj01xkGMCySK15JNOgwf2U3zbgmt/IBg5Vk3tBxvGr9i2S3wqmJgeXpcb
>
> 8mAgkf/93l2/a/anftwIE6XfmWZqy4nxdX+AZ9hKnyfZs2bW1zxzydOlcFH3NB2e
>
> LXWFbYOtTt4jrP9VIM4FL1I+q9RVi5zjNLWayLsEpIRtD9h/kZ5A4S+r16XYvzqp
>
> vec5oJ+bOEs7kA2FtFzFeZXASxLd+5Zlx0j/uB0J+iY9y0TOWTcePRJV+d8nQB5p
>
> rpQ9zBxHiLJmnO+0sn6N1/FnomUCAwEAAaNQME4wHQYDVR0OBBYEFCXon1DQdFzw
>
> URRTzdQGWQ2I0abMMB8GA1UdIwQYMBaAFCXon1DQdFzwURRTzdQGWQ2I0abMMAwG
>
> A1UdEwQFMAMBAf8wDQYJKoZIhvcNAQEFBQADggEBAM17byCO4c/wiDgLYHywM6jd
>
> fsNP8FL+Bm6Itv1s6mNEQELR8XdQJHGB+TU3WAZk8avXNvFzPwpEN3f/QvAQjN7l
>
> noXT+rRkRHJKALSZe2AfBDP7WX9ibk5Vnm3PVnQa46f8vbvuxVEUu5e1YKcR1kXe
>
> booucDCp1g3vc9K6tbSyKS04CA+ys6pKHpqlxxVqSxe+1Kii3CeBLVcDXYrQvI22
>
> bi7xIMmV5njur+vkJKaHZtM6LuGe0k5Yw881/CBHCf7Vg3iZt8cqXC1OUo4qLV30
>
> 0x+alWlFoz9xtiKcfiaQwQL8O3Uip9VoK52XFGbi9yiTqTDuc1zYpHX8V/cbS3U=
>
> -END CERTIFICATE-
>
> ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$
>
>
>
> The implementation is here:
>
>
> https://github.com/AxelNennker/cordova-plugin-file-transfer/blob/master/src/android/FileTransfer.java#L100
>
>
>
> I have a general question: How does an application programmer (cordova
> user) specify resources to be added to the project.
>
> I think that merges could be a way to do this but this always merges files
> into the platforms www directory.
>
> On Android this is platforms/android/assets/www but I don't want the file
> there.
>
>
>
> Cheers
>
> Axel
>
>
>
>
>
>
>


Re: File plugin threading on iOS

2013-12-11 Thread Ian Clelland
I'm hoping to have it all merged by the end of this week, if all of the
tests still pass.

Thanks for checking it out! :)

Ian


On Wed, Dec 11, 2013 at 8:59 AM, Michael Gauthier wrote:

>
> On 2013-12-10 18:10, Ian Clelland wrote:
>
>> On Tue, Dec 10, 2013 at 4:40 PM, Michael Gauthier > >wrote:
>>
>>  I quickly updated the File plugin to do the local file uri stuff in
>>> resolveLocalFileSystemURI in the background using [self.commandDelegate
>>> runInBackground:^{}].
>>>
>>>
>> Dev or master?
>>
>>  I made the change in master. Dev looks way awesomer. When do you think
> you will release?
>
>
>
>
>>
>>> It works and allows my app interface to respond while the FS calls are
>>> resolving. Is this approach crazy or obviously broken in some way?
>>>
>>>
>> Probably not crazy -- depending on how you've implemented it :) It's
>> basically what I'm doing in my branch as well.
>>
>> Currently, I have CDVFile using that pattern, [self.commandDelegate
>> runInBackground:], to call the actual implementation method in
>> CDVLocalFilesystem or CDVAssetLibraryFilesystem. CDVFile passes the
>> implementation methods a callback function that sends the actual plugin
>> response when the operation is finished. (Keeping all of the
>> "Cordova-plugin-y" bits in CDVFile, and just having the raw file
>> operations
>> in the file system implementation classes.
>>
>> You can see an example of what I've done in the implementation of [CDVFile
>> readAsText:] and in [CDVLocalFilesystem readFileAtURL:start:end:
>> callback:]
>> in the dev branch.
>>
>> Ian
>>
>>
>


Re: File plugin threading on iOS

2013-12-11 Thread Michael Gauthier


On 2013-12-10 18:10, Ian Clelland wrote:

On Tue, Dec 10, 2013 at 4:40 PM, Michael Gauthier wrote:


I quickly updated the File plugin to do the local file uri stuff in
resolveLocalFileSystemURI in the background using [self.commandDelegate
runInBackground:^{}].



Dev or master?

I made the change in master. Dev looks way awesomer. When do you think 
you will release?







It works and allows my app interface to respond while the FS calls are
resolving. Is this approach crazy or obviously broken in some way?



Probably not crazy -- depending on how you've implemented it :) It's
basically what I'm doing in my branch as well.

Currently, I have CDVFile using that pattern, [self.commandDelegate
runInBackground:], to call the actual implementation method in
CDVLocalFilesystem or CDVAssetLibraryFilesystem. CDVFile passes the
implementation methods a callback function that sends the actual plugin
response when the operation is finished. (Keeping all of the
"Cordova-plugin-y" bits in CDVFile, and just having the raw file operations
in the file system implementation classes.

You can see an example of what I've done in the implementation of [CDVFile
readAsText:] and in [CDVLocalFilesystem readFileAtURL:start:end:callback:]
in the dev branch.

Ian





PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project

2013-12-11 Thread Axel.Nennker
Hi,

We are seeing some strange effect on Android when we move from 3.1.0-0.2.0 to a 
newer version.
We tried with 3.2.0-0.4.0 and 3.3-rc1

I think that PluginManager changed and now fails to load config.xml

This happens in my project when the app is opened by the operating system 
because it is configured so through an intent filter.
The original activity is not a child of DroidGap but starts the "main" activity 
using startActivity.

The code to get the id of the config.xml file is not working in this case.

Why was the 3.1.0 code changed?

Cheers
Axel

D/CordovaActivity(12320): CordovaActivity.init()
D/CordovaWebView(12320): >>> 
loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
D/PluginManager(12320): init()
E/PluginManager(12320): 
=
E/PluginManager(12320): ERROR: config.xml is missing.  Add res/xml/config.xml 
to your project.
E/PluginManager(12320): 
https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
E/PluginManager(12320): 
=
D/CordovaWebView(12320): >>> loadUrlNow()
D/ItemImport(12320): onResume() loadURL started:


Support self-signed certs in FileTransfer

2013-12-11 Thread Axel.Nennker
Hi,



I think that the trustAllHosts parameter in FileTransfer is convenient but evil.



I propose to add support for self-signed certs to FileTransfer.

There does NOT seem to be an open issue for this on Jira:

https://issues.apache.org/jira/browse/CB-3576?jql=project%20%3D%20CB%20AND%20text%20~%20self-signed



Do you think that this is a good/needed/superfluous effort?

Should I create an issue?



I created an initial Android implementation here:

https://github.com/AxelNennker/cordova-plugin-file-transfer



Usage would be as follows:

- Add a line like this to www/config.xml





Put the cert into a file and store it into the projects asset directory

ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ ls -l 
platforms/android/assets/

insgesamt 8

-rwxrwxrwx 1 root root 1310 Dez 10 21:46 cacerts.pem

drwxrwxrwx 1 root root 4096 Dez 10 22:51 www

ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$ cat 
platforms/android/assets/cacerts.pem

-BEGIN CERTIFICATE-

MIIDizCCAnOgAwIBAgIJAPh3RC7M50KHMA0GCSqGSIb3DQEBBQUAMFwxCzAJBgNV

BAYTAkRFMQ8wDQYDVQQIDAZCZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UE

CgwOVC1MYWJzIE5lbm5rZXIxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xMzEyMTAy

MDQyNTFaFw0xNDEyMTAyMDQyNTFaMFwxCzAJBgNVBAYTAkRFMQ8wDQYDVQQIDAZC

ZXJsaW4xDzANBgNVBAcMBkJlcmxpbjEXMBUGA1UECgwOVC1MYWJzIE5lbm5rZXIx

EjAQBgNVBAMMCWxvY2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoC

ggEBANmBMf9r86HiavL10WzVpbBt03JY9JUppwY1BZKtjejw9b7tLypgueBAGBSB

jpb5KoaIj01xkGMCySK15JNOgwf2U3zbgmt/IBg5Vk3tBxvGr9i2S3wqmJgeXpcb

8mAgkf/93l2/a/anftwIE6XfmWZqy4nxdX+AZ9hKnyfZs2bW1zxzydOlcFH3NB2e

LXWFbYOtTt4jrP9VIM4FL1I+q9RVi5zjNLWayLsEpIRtD9h/kZ5A4S+r16XYvzqp

vec5oJ+bOEs7kA2FtFzFeZXASxLd+5Zlx0j/uB0J+iY9y0TOWTcePRJV+d8nQB5p

rpQ9zBxHiLJmnO+0sn6N1/FnomUCAwEAAaNQME4wHQYDVR0OBBYEFCXon1DQdFzw

URRTzdQGWQ2I0abMMB8GA1UdIwQYMBaAFCXon1DQdFzwURRTzdQGWQ2I0abMMAwG

A1UdEwQFMAMBAf8wDQYJKoZIhvcNAQEFBQADggEBAM17byCO4c/wiDgLYHywM6jd

fsNP8FL+Bm6Itv1s6mNEQELR8XdQJHGB+TU3WAZk8avXNvFzPwpEN3f/QvAQjN7l

noXT+rRkRHJKALSZe2AfBDP7WX9ibk5Vnm3PVnQa46f8vbvuxVEUu5e1YKcR1kXe

booucDCp1g3vc9K6tbSyKS04CA+ys6pKHpqlxxVqSxe+1Kii3CeBLVcDXYrQvI22

bi7xIMmV5njur+vkJKaHZtM6LuGe0k5Yw881/CBHCf7Vg3iZt8cqXC1OUo4qLV30

0x+alWlFoz9xtiKcfiaQwQL8O3Uip9VoK52XFGbi9yiTqTDuc1zYpHX8V/cbS3U=

-END CERTIFICATE-

ignisvulpis@ubuntu:/host/cordova-plugin-file-transfer/phonegap$



The implementation is here:

https://github.com/AxelNennker/cordova-plugin-file-transfer/blob/master/src/android/FileTransfer.java#L100



I have a general question: How does an application programmer (cordova user) 
specify resources to be added to the project.

I think that merges could be a way to do this but this always merges files into 
the platforms www directory.

On Android this is platforms/android/assets/www but I don't want the file there.



Cheers

Axel