Make sure these platforms get added to cordova platform ls, with appropriate logic for where Ubuntu can be built.
Braden On Thu, Dec 5, 2013 at 6:02 PM, Andrew Grieve <agri...@chromium.org> wrote: > On Thu, Dec 5, 2013 at 8:53 PM, Steven Gill <stevengil...@gmail.com> > wrote: > > > Just wanted to give a update > > > > Support for both of these now exists in plugman + CLI on master. It seems > > to be working well enough. > > > > Docs: This is a pretty important piece. Once we release 3.3.0, our users > > are going to want to know what plugins are supported, what the getting > > started requirements are, etc. I know FireOS has already sent some doc > > updates. I would suggest installing all of the plugins you support and > run > > mobile-spec. Once you have confirmed a plugin has full support, add your > > platform to the supported platform list. > > > > Docs can be found at https://github.com/apache/cordova-docs/ > > > > *FireOS* > > Plugman Tests: I commented out the tests because they need work. They > rely > > to heavily on a copy paste job of the android tests. I had to update the > > plugin.xml files for the three plugins in the tests to add support for > > amazon-fireos just to get the tests to run. > > > > Check-reqs: right now the requirements are being checked in the cli. We > > will want to change this to use the platform level chec_reqs script > > instead. Android made this change recently. Hopefully someone can point > you > > to the specific commits that made the change. This is suffering from the > > same problem Android just had, in that it is setting the base api level > at > > 17. We should update that number for now and after 3.3.0 work on > converting > > this. > > > To add to this - I made a check afterwards to just make CLI's > check_requirements return true immediately. It makes more sense to have the > bin/create script run check_reqs itself. The original reason for having > check_requirements in CLI was to be able to check requirements before > running lazy_load, but that never came to fruition (and is a pretty minor > optimization anyways). > > > > > > *Ubuntu* > > Tests! We need them! :) > > Need to make clear that ubuntu option isn't available on Mac or Windows. > > The error messages aren't clear here. Documentation around this would go > a > > long way! > > > > > > I will continue to give suggestions as I play around more with both of > > these platforms. Please file issues for any of the work you do and use > > those in the commit messages. > > > > The 3.3.0-rc.1 will be tagged & released tomorrow. I will tag both of > these > > platforms leading up to the release. I'm aiming to release 3.3.0 next Wed > > or Thursday. So we will still have some time to improve the experience > for > > users who will be trying out these platforms for the first time. > > >