Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7721DCF9D for ; Tue, 16 Jul 2013 19:17:44 +0000 (UTC) Received: (qmail 97131 invoked by uid 500); 16 Jul 2013 19:17:44 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 97101 invoked by uid 500); 16 Jul 2013 19:17:43 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 97093 invoked by uid 99); 16 Jul 2013 19:17:43 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 16 Jul 2013 19:17:43 +0000 X-ASF-Spam-Status: No, hits=2.5 required=5.0 tests=FRT_ADOBE2,HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of iclelland@google.com designates 209.85.214.175 as permitted sender) Received: from [209.85.214.175] (HELO mail-ob0-f175.google.com) (209.85.214.175) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 16 Jul 2013 19:17:39 +0000 Received: by mail-ob0-f175.google.com with SMTP id xn12so1220750obc.20 for ; Tue, 16 Jul 2013 12:17:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=J9umjMij2mKfA8OEK1svBxJ3ysDIW4OqiL/8Wh8i6P4=; b=muG1HOo0F8+y3LGvbnsEdKbFd992WrFsmog+5OzfP60r8f6M2hMsCZeVA0cEfyLpa+ LTg8cDqeAYhICsiOhkZ2Ec4K7TptTDSeGGsXrEgo9mGtxL2gpnRlEX+oZqkPJW98sGLg gNWtE7nCslrFC8PIT+qqoDpp4PvU1AvdkNd3+dOROY4ZPcAu7rXP9xofdYWI/RaO/VLi uwOESfa7IbYGykR0oQ6iU0scrjEt20GsSOPzAQOTrDtPX+LYBPNNQgWW+CJflV+sOLUK 7Qa2jHnysDW2c0LlSppLS36cJoBmczJ6Y84cbTcN2jNYslAiLTdobAHNXREzJAyhxED7 cr0g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type:x-gm-message-state; bh=J9umjMij2mKfA8OEK1svBxJ3ysDIW4OqiL/8Wh8i6P4=; b=Il4Bu79PiRBuNL4ifuR0ESoJXfPiABakCxyP4BQbx2uxFhkuXMpQUcjvzdoXpvPIBZ GoqiftexNSvs5xRgozkmylOw9CtxptXg52HtMd0L5zG1j8nFJdRP2RtH4YtXIayAnqiJ o0kS/oIYzL4tQp/OHK0D6rtB76PzdLq0KhZFJ2pdMNIN8Fg7kK3K5rD92M+K+ByhNqbc RHIygv+n/z1rfAPLpBm1f49y3LbDDPU0zHUcVPuLfmpn7U0/HYkZi0Puoo1KJPYOqUnT ZccH2Wiy2JUMSAhrX9HfIctELJLhGfJor1VcnaR00KLOM2jdJw0sI/5j+ni+ZKJCO3Mm cv1w== X-Received: by 10.60.117.233 with SMTP id kh9mr3708070oeb.58.1374002238331; Tue, 16 Jul 2013 12:17:18 -0700 (PDT) MIME-Version: 1.0 Received: by 10.182.106.196 with HTTP; Tue, 16 Jul 2013 12:16:58 -0700 (PDT) In-Reply-To: References: From: Ian Clelland Date: Tue, 16 Jul 2013 15:16:58 -0400 Message-ID: Subject: Re: 3.0.0 Testing thread To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=047d7b3a92a05631c104e1a5d3cb X-Gm-Message-State: ALoCoQkbrkMfaMMSHp/XKyUe2zKKcxPnnlEycpoFdxLyhfH54uqL9EPh4cooXscwVD5nmSmyeUh9ubRIJiohYnQXNv1s7i38nJDSqX8f3k6mIFawE3Gw75d/M0HZoHm81KVi5lU7Tm+HRAotz4WIVdQ5Ze6Ol9KnbGtF4zjmyx48xfQgDLGl4+Mpq+axx8ViGWUG30ognk5V X-Virus-Checked: Checked by ClamAV on apache.org --047d7b3a92a05631c104e1a5d3cb Content-Type: text/plain; charset=UTF-8 Agreed. It's not random, but there's nothing in the process that guarantees that. It should be tagged, and the tagged version should go out with the -android repo. Tag it (3.0.0rc2 -- or 3.0.0-rc2 to be semver-conformant), and you can re-build and package it with cordova-android. On Tue, Jul 16, 2013 at 3:14 PM, Joe Bowser wrote: > I'm going to re-tag the JS. Since the only commit is this fix, none of > the other platforms should need re-testing, but grabbing a random JS > isn't the right way to do things, IMO. > > On Tue, Jul 16, 2013 at 11:58 AM, Ian Clelland > wrote: > > I've moved the latest cordova-js into cordova android, and pushed to > > master. All of the mobilespec auto tests are passing for me. (With > > mobile-spec-dependencies and all of its dependent plugins installed > through > > plugman) > > > > Ian > > > > > > On Tue, Jul 16, 2013 at 2:45 PM, Ian Clelland > wrote: > > > >> On Tue, Jul 16, 2013 at 2:42 PM, Joe Bowser wrote: > >> > >>> Has the issue been resolved, and is it plugin-related or is it part of > >>> cordova-android? > >>> > >> > >> Yes, no and maybe. > >> That was the android bridge issue; cordova-js/lib/android/exec.js. It > will > >> be apart of cordova-android as soon as I package up the patched > cordova-js > >> and move it to cordova-android/framework/assets/www. > >> > >> Ian > >> > >> > >>> On Tue, Jul 16, 2013 at 11:36 AM, Ian Clelland > > >>> wrote: > >>> > That's right -- the issue was android-specific. > >>> > > >>> > Ian > >>> > > >>> > > >>> > On Tue, Jul 16, 2013 at 2:32 PM, Shazron wrote: > >>> > > >>> >> No need to re-get and re-tag JS right for the other platforms? > >>> >> > >>> >> > >>> >> On Tue, Jul 16, 2013 at 10:46 AM, Andrew Grieve < > agrieve@chromium.org > >>> >> >wrote: > >>> >> > >>> >> > Okay, seems it was a bad rebase when Ian made the base64 change. > >>> Will be > >>> >> > fixed shortly. Weird that these would pass at all for anyone in > the > >>> past > >>> >> > week! > >>> >> > > >>> >> > > >>> >> > On Tue, Jul 16, 2013 at 1:27 PM, Andrew Grieve < > agrieve@chromium.org > >>> > > >>> >> > wrote: > >>> >> > > >>> >> > > Okay - on my N4 4.2.2 they are failing as well. I'll look into > it. > >>> >> > > > >>> >> > > > >>> >> > > On Tue, Jul 16, 2013 at 1:11 PM, Joe Bowser > >>> wrote: > >>> >> > > > >>> >> > >> I'm testing on the HTC One running stock 4.2.2. The Google one > >>> without > >>> >> > >> sense and the other crap. > >>> >> > >> On Jul 16, 2013 9:51 AM, "Andrew Grieve" < > agrieve@chromium.org> > >>> >> wrote: > >>> >> > >> > >>> >> > >> > Joe - what setup are you seeing the failures for? I'm running > >>> latest > >>> >> > >> > everything and on 4.1.1 emulator all file tests pass. > >>> >> > >> > > >>> >> > >> > Shouldn't be related to ResourceApi change, as the File > plugin > >>> >> doesn't > >>> >> > >> use > >>> >> > >> > it. > >>> >> > >> > > >>> >> > >> > > >>> >> > >> > On Tue, Jul 16, 2013 at 11:51 AM, Filip Maj > >>> wrote: > >>> >> > >> > > >>> >> > >> > > Yes, when you clone down either of the tools, ALWAYS run > `npm > >>> >> > >> install` in > >>> >> > >> > > its directory to reinitialize the dependencies. Even when > just > >>> >> > >> updating > >>> >> > >> > > the code for the tools, run `npm install` just in case in > >>> case the > >>> >> > >> deps > >>> >> > >> > > changed > >>> >> > >> > > > >>> >> > >> > > On 7/16/13 8:06 AM, "Shazron" wrote: > >>> >> > >> > > > >>> >> > >> > > >aha, cordova-cli specified plugman 0.9.3 -- and that > works. > >>> It's > >>> >> a > >>> >> > >> bug > >>> >> > >> > > >when > >>> >> > >> > > >cordova-cli uses the latest plugman > >>> >> > >> > > > > >>> >> > >> > > > > >>> >> > >> > > > > >>> >> > >> > > >On Tue, Jul 16, 2013 at 8:03 AM, David Kemp < > >>> drkemp@google.com> > >>> >> > >> wrote: > >>> >> > >> > > > > >>> >> > >> > > >> I had the same error that you got, but running npm > install > >>> in > >>> >> the > >>> >> > >> > > >> cordova-cli directory installed a fresh one (not sure > which > >>> >> > >> version ) > >>> >> > >> > > >>and > >>> >> > >> > > >> everything worked fine > >>> >> > >> > > >> > >>> >> > >> > > >> > >>> >> > >> > > >> On Tue, Jul 16, 2013 at 10:50 AM, Shazron < > >>> shazron@gmail.com> > >>> >> > >> wrote: > >>> >> > >> > > >> > >>> >> > >> > > >> > I installed plugman 0.9.6 before using cordova-cli > from > >>> >> master, > >>> >> > >> and > >>> >> > >> > > >>that > >>> >> > >> > > >> is > >>> >> > >> > > >> > the latest on npm - but I assume you mean the latest > >>> from the > >>> >> > >> > > >> > cordova-plugman repo > >>> >> > >> > > >> > > >>> >> > >> > > >> > > >>> >> > >> > > >> > On Tue, Jul 16, 2013 at 7:42 AM, David Kemp < > >>> >> drkemp@google.com > >>> >> > > > >>> >> > >> > > wrote: > >>> >> > >> > > >> > > >>> >> > >> > > >> > > the newest cli needs the newest plugman. > >>> >> > >> > > >> > > Also if you uninstall plugins with the older > version, > >>> the > >>> >> new > >>> >> > >> one > >>> >> > >> > > >>wont > >>> >> > >> > > >> > put > >>> >> > >> > > >> > > them in. > >>> >> > >> > > >> > > > >>> >> > >> > > >> > > > >>> >> > >> > > >> > > > >>> >> > >> > > >> > > On Tue, Jul 16, 2013 at 10:27 AM, Shazron < > >>> >> shazron@gmail.com > >>> >> > > > >>> >> > >> > > wrote: > >>> >> > >> > > >> > > > >>> >> > >> > > >> > > > I'm using the master version of the cordova-cli, > >>> >> > installing a > >>> >> > >> > > >>plugin > >>> >> > >> > > >> is > >>> >> > >> > > >> > > > fine, but uninstall throws this error: > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > $ ../cordova-cli/bin/cordova plugin remove > >>> >> > >> > > >> > > org.apache.cordova.core.console > >>> >> > >> > > >> > > > [TypeError: Object function > uninstallPlugin(platform, > >>> >> > >> > project_dir, > >>> >> > >> > > >> id, > >>> >> > >> > > >> > > > plugins_dir, options, callback) { > >>> >> > >> > > >> > > > if (!platform_modules[platform]) { > >>> >> > >> > > >> > > > var err = new Error(platform + " not > >>> >> supported."); > >>> >> > >> > > >> > > > if (callback) return callback(err); > >>> >> > >> > > >> > > > else throw err; > >>> >> > >> > > >> > > > } > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > var plugin_dir = path.join(plugins_dir, id); > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > if (!fs.existsSync(plugin_dir)) { > >>> >> > >> > > >> > > > var err = new Error('Plugin "' + id + '" > not > >>> >> found. > >>> >> > >> > > >>Already > >>> >> > >> > > >> > > > uninstalled?'); > >>> >> > >> > > >> > > > if (callback) return callback(err); > >>> >> > >> > > >> > > > else throw err; > >>> >> > >> > > >> > > > } > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > var current_stack = new action_stack(); > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > options.is_top_level = true; > >>> >> > >> > > >> > > > runUninstall(current_stack, platform, > >>> project_dir, > >>> >> > >> > plugin_dir, > >>> >> > >> > > >> > > > plugins_dir, options, callback); > >>> >> > >> > > >> > > > } has no method 'uninstallPlatform'] > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > On Tue, Jul 16, 2013 at 7:21 AM, Joe Bowser < > >>> >> > >> bowserj@gmail.com> > >>> >> > >> > > >> wrote: > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > > Has anyone managed to get plugman to uninstall a > >>> >> plugin? > >>> >> > >> The > >>> >> > >> > > >> > > > > dependencies plugin never cleanly installs or > >>> >> uninstalls. > >>> >> > >> > > >> > > > > > >>> >> > >> > > >> > > > > On Tue, Jul 16, 2013 at 6:37 AM, Shazron < > >>> >> > >> shazron@gmail.com> > >>> >> > >> > > >> wrote: > >>> >> > >> > > >> > > > > > https://issues.apache.org/jira/browse/CB-4264 > >>> >> > >> > > >> > > > > > > >>> >> > >> > > >> > > > > > Turns out it was a "false positive" failure, > the > >>> test > >>> >> > >> needs > >>> >> > >> > > >>to be > >>> >> > >> > > >> > > > > improved. > >>> >> > >> > > >> > > > > > But so far all systems go for iOS. > >>> >> > >> > > >> > > > > > > >>> >> > >> > > >> > > > > > > >>> >> > >> > > >> > > > > > On Mon, Jul 15, 2013 at 6:54 PM, Shazron < > >>> >> > >> shazron@gmail.com > >>> >> > >> > > > >>> >> > >> > > >> > wrote: > >>> >> > >> > > >> > > > > > > >>> >> > >> > > >> > > > > >> So far I went and tested with the plugins > >>> (specified > >>> >> > in > >>> >> > >> the > >>> >> > >> > > >> > > > > >> dependencies-plugin on cordova-mobile-spec) > on > >>> >> master > >>> >> > >> for > >>> >> > >> > > >>iOS, > >>> >> > >> > > >> > with > >>> >> > >> > > >> > > 1 > >>> >> > >> > > >> > > > > test > >>> >> > >> > > >> > > > > >> failing: > >>> >> > >> > > >> > > > > >> > >>> >> > >> > > >> > > > > >> File API DirectoryReader interface > readEntries > >>> >> > >> > file.spec.109 > >>> >> > >> > > >> > should > >>> >> > >> > > >> > > > > return > >>> >> > >> > > >> > > > > >> an empty entry list on the second call. > >>> >> > >> > > >> > > > > >> Expected 0 not to be 0. > >>> >> > >> > > >> > > > > >> > >>> >> > >> > > >> > > > > > >>> >> > >> > > >> > > > > >>> >> > >> > > >> > > > >>> >> > >> > > >> > > >>> >> > >> > > >> > >>> >> > >> > > > >>> >> > >> > > > >>> >> > >> > > >>> >> > >> > >>> >> > > > >>> >> > > > >>> >> > > >>> >> > >>> > >> > --047d7b3a92a05631c104e1a5d3cb--