Ah, indeed. CLI has hard coded req on 17 in module.exports.check_requirements in android_parser.js. That needs to change especially quickly now since the platform SDK dependency was bumped to 18 in our 3.2 release (so CLI and platform workflows have atm different requirements).
On Wed, Nov 27, 2013 at 2:49 PM, Joe Bowser <bows...@gmail.com> wrote: > The problem is that we should never be relying on the CLI for this > info, and that the platforms should be doing it. That's why I edited > the bug in the first place. This is also a problem with iOS, although > less of a problem due to >=. > > On Wed, Nov 27, 2013 at 11:43 AM, Michal Mocny <mmo...@chromium.org> > wrote: > > Would be nice to change scripts to check for >= target-level instead of > > exactly ==. I currently just re-installed my android SDK locally with > only > > 4.4 (API 19), and now fail to create a project with CLI since it isnt > > exactly the version we look for. > > > > Since there won't be another android release for a while I'm sure this > issue > > won't come up after the cordova 3.3 release version bump.. so maybe its > low > > priority. > > > > In the mean time, now that we've shipped 3.2, shall we bump the version > on > > master? > > > > > > On Thu, Nov 7, 2013 at 10:55 AM, Braden Shepherdson <bra...@chromium.org > > > > wrote: > >> > >> I think some people are confused about this number and backward > >> compatibility. > >> > >> There are *two* version constraints on Android. The one we're discussing > >> here is which API we're compiling against, which should generally be set > >> at > >> the latest stable version (+1 to 3.2 targeting API 18 (Android 4.3)). > >> > >> There's another, separate value for the *minimum* API version. Your > >> project > >> cannot use any methods or classes introduced after that API version. For > >> Cordova, we are currently setting that to API 10 (2.3.3). So we should > >> support devices running 10, and 14-18. We don't officially support > 11-13, > >> because no one cares about Honeycomb, but they mostly work anyway. > >> > >> Braden > >> > >> > >> On Thu, Nov 7, 2013 at 10:17 AM, Carlos Santana > >> <csantan...@gmail.com>wrote: > >> > >> > We are also in the works on getting a Nexus 5, probably on Friday we > >> > want > >> > to get unlocked > >> > > >> > > >> > On Thu, Nov 7, 2013 at 9:38 AM, Joe Bowser <bows...@gmail.com> wrote: > >> > > >> > > That's correct. I only have an old Nexus 7 with an AOSP build I made > >> > > yesterday to test things on 4.4 while I wait for the Nexus 5 or the > >> > > official updates, and since I have no camera drivers working. > >> > > > >> > > BTW: We still have a gallery, but I think the Camera plugin may be > >> > > broken > >> > > in 4.4. > >> > > On Nov 7, 2013 6:27 AM, "Carlos Santana" <csantan...@gmail.com> > >> > > wrote: > >> > > > >> > > > To be clear the goal for Cordova 3.2 is to set target 18 (Android > >> > > > 4.3) > >> > > and > >> > > > for Cordova 3.3 target 19 (Android 4.4) ? > >> > > > > >> > > > We are about to release 3.2 so I want to be sure. > >> > > > > >> > > > I vote to set target 19/4.4 (KitKat) for Cordova 3.3 (Allows time > to > >> > > > investigate, test, integrate, and document) > >> > > > > >> > > > > >> > > > > >> > > > On Thu, Nov 7, 2013 at 8:19 AM, Gorkem Ercan > >> > > > <gorkem.er...@gmail.com> > >> > > > wrote: > >> > > > > >> > > > > Joe is right on the money. If the target is older than 13 it > will > >> > fail > >> > > to > >> > > > > compile which was detected elsewhere [1]. > >> > > > > > >> > > > > Also as detected on the other thread CLI is not actually [2] [3] > >> > > > delegating > >> > > > > the requirement checks to platform scripts for > >> > > > > Android and iOS. After this issue is resolved it will depend on > >> > > > > the > >> > > value > >> > > > > on the project.properties. Hopefully that is more likely to be > >> > updated > >> > > > > and we will not end up forcing level 17 when 18 and 19 was out. > >> > > > > > >> > > > > I am not happy about the fixed target versions overall though. > On > >> > JBoss > >> > > > > tools we decided to look for the newest target that exists and > use > >> > that > >> > > > as > >> > > > > long as it > >> > > > > is newer that minimum (which is 17 at this time). This way we > >> > > > > would > >> > not > >> > > > > disappoint a developer who gets the latest and greatest SDK > >> > > > > update. > >> > > > > > >> > > > > [1] https://issues.jboss.org/browse/JBIDE-15885 > >> > > > > [2] https://issues.apache.org/jira/browse/CB-5298 > >> > > > > [3] https://issues.apache.org/jira/browse/CB-5297 > >> > > > > > >> > > > > -- > >> > > > > Gorkem > >> > > > > > >> > > > > > >> > > > > On Thu, Nov 7, 2013 at 7:51 AM, Brian LeRoux <b...@brian.io> > wrote: > >> > > > > > >> > > > > > Axel: you are correct that we want as many devices as > possible. > >> > This > >> > > is > >> > > > > why > >> > > > > > we've always set to the highest level. If you set to the > highest > >> > > level > >> > > > it > >> > > > > > is inclusive to all platforms before it (that we decide to > >> > support). > >> > > > > > > >> > > > > > > >> > > > > > On Thu, Nov 7, 2013 at 12:00 PM, Axel Nennker < > >> > ignisvul...@gmail.com > >> > > > > >> > > > > > wrote: > >> > > > > > > >> > > > > > > I think the highest level is not what developers need. > >> > > > > > > When you create a product/app you want your app to run on as > >> > > > > > > many > >> > > > > devices > >> > > > > > > as possible and not only the latest. > >> > > > > > > Am 07.11.2013 11:36 schrieb "Brian LeRoux" <b...@brian.io>: > >> > > > > > > > >> > > > > > > > Apologies I think there is another thread about this but > I'd > >> > like > >> > > > to > >> > > > > > > > understand more about what we're thinking here. There's > been > >> > > > > discussion > >> > > > > > > > that we should make this configurable. I disagree. I think > >> > > > > > > > we > >> > > need > >> > > > to > >> > > > > > > > target highest available level possible, as we always have > >> > > > > > > > in > >> > the > >> > > > > past, > >> > > > > > > and > >> > > > > > > > take backwards compat on. That means we need to update > [1]. > >> > > > > > > > > >> > > > > > > > Thoughts? > >> > > > > > > > > >> > > > > > > > [1] > >> > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > > https://github.com/apache/cordova-cli/blob/master/src/metadata/android_parser.js#L56 > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > >> > > > > >> > > > -- > >> > > > Carlos Santana > >> > > > <csantan...@gmail.com> > >> > > > > >> > > > >> > > >> > > >> > > >> > -- > >> > Carlos Santana > >> > <csantan...@gmail.com> > >> > > > > > >
