I think having the conversion script makes sense (and you've already
implemented it, thanks Andrew). Supporting both can cause confusion.

So.... what is the consensus if any here.


On Thu, Nov 29, 2012 at 10:40 AM, Andrew Grieve <agri...@chromium.org>wrote:

> Copying from the JIRA issue to this thread:
>
> I think supporting both was one of the things that upset users when Android
> > made the switch (at least, it upset me). What happened was that I ended
> up
> > having both files present, and the code was silently using one and not
> the
> > other, and I couldn't figure out why my whitelist changes were not being
> > picked up.
>
>
>
> >
> > I think a conversion script was also suggested. I think it would be best
> > to just fail loudly if config.xml is missing with a message saying "run
> > this script: cordova/bin/plist2xml.py path/to/cordova.plist
>
>
>
>
>
>
>
>
> On Thu, Nov 29, 2012 at 1:26 PM, Braden Shepherdson <bra...@chromium.org
> >wrote:
>
> > Oh, sweet. I'm glad I went to lunch instead of working on it further :P
> >
> > Braden
> >
> >
> > On Thu, Nov 29, 2012 at 12:47 PM, Anis KADRI <anis.ka...@gmail.com>
> wrote:
> >
> > > I've already started working on pluginstall…which btw is now called
> > > plugman<http://github.com/imhotep/plugman>as it has diverged
> > > significantly from the original tool. I am expecting to
> > > commit the code sometime today.
> > >
> > >
> > > On Thu, Nov 29, 2012 at 8:38 AM, Braden Shepherdson <
> bra...@chromium.org
> > > >wrote:
> > >
> > > > The code I checked in, which is now tagged, is expecting config.xml
> > only.
> > > > It wouldn't be terribly hard to support plists too, but I agree that
> a
> > > > clean change is less confusing.
> > > >
> > > > I think a conversion script is overkill, it only takes about three to
> > > > convert one to the other (30 seconds if you vim macros like a boss)
> and
> > > > plenty of people will be able to grab the example config.xml and
> tweak
> > > one
> > > > or two settings rather than converting their whole thing.
> > > >
> > > > The current state of error messages is less than ideal. Currently if
> > you
> > > > try to build but have no config.xml, you just get an Xcode error
> > message
> > > > about the missing file :( I'm not sure how we can improve that. Our
> > users
> > > > are used to looking at our release notes, featuring this prominently
> > and
> > > > pointing to a guide to converting should sort things out.
> > > >
> > > > I'm working on getting pluginstall to handle it properly. Are there
> > more
> > > > docs that need updating?
> > > >
> > > > Braden
> > > >
> > >
> >
>

Reply via email to