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 9D50F107DF for ; Tue, 29 Oct 2013 15:48:53 +0000 (UTC) Received: (qmail 61708 invoked by uid 500); 29 Oct 2013 15:47:42 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 61514 invoked by uid 500); 29 Oct 2013 15:47:38 -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 61258 invoked by uid 99); 29 Oct 2013 15:47:30 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Oct 2013 15:47:30 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of brian.leroux@gmail.com designates 209.85.223.180 as permitted sender) Received: from [209.85.223.180] (HELO mail-ie0-f180.google.com) (209.85.223.180) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Oct 2013 15:47:26 +0000 Received: by mail-ie0-f180.google.com with SMTP id e14so52388iej.25 for ; Tue, 29 Oct 2013 08:47:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:content-type; bh=XKl2Lb57OKFktNCV9b7kwL+jHOxBtBIYxZJN4mM4DI0=; b=mmdT8wZ/SoBPU81tBN+2bNglB2sf6c49sQ+TkdJ7DVA1Z4OTmDIYOZX5I7e4M546CM sI3Yqp2B/TDkzWwBviCqb5Xve3g2NkgA2RFmEPgfY2pgMLzSr0RVU4crAZXCzb/XJU/T sQFrGocKG60yMwrmqUKg9FCTXxUhTunglPNprySsxKJp1HFYwTCl9dbzasbBEY6QRRf/ 3gyrwqv9bJwB2yatociDwQfO54btJqwrKTbz/PbG20Vvh4/HX2dQYUb9EbJN1LGyK8uP lMRG+htdp1loXmBjv1hvvJdMP+mYY/+dCIcULJB2cWDCF5XbSgTIIC/8qwxX5sGMjPDW cpMA== MIME-Version: 1.0 X-Received: by 10.50.1.78 with SMTP id 14mr13088830igk.37.1383061625057; Tue, 29 Oct 2013 08:47:05 -0700 (PDT) Sender: brian.leroux@gmail.com Received: by 10.50.114.132 with HTTP; Tue, 29 Oct 2013 08:47:04 -0700 (PDT) In-Reply-To: References: Date: Tue, 29 Oct 2013 08:47:04 -0700 X-Google-Sender-Auth: T5zFuRt8fHR9QpKpH340KnkgdWc Message-ID: Subject: Re: CLI build failure From: Brian LeRoux To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=047d7bdc119add3bac04e9e3209a X-Virus-Checked: Checked by ClamAV on apache.org --047d7bdc119add3bac04e9e3209a Content-Type: text/plain; charset=ISO-8859-1 There are a few ways to check if there is an update published to npm. `npm help info` is one. this is new http://npm-onupdate.info/ On Tuesday, October 29, 2013, Michal Mocny wrote: > I think you can also install plugman into node_modules *before* running > cli's npm install and it will not try to fetch from npm since there is > already a compatible version there. > > -Michal > > > On Tue, Oct 29, 2013 at 8:15 AM, Carlos Santana > >wrote: > > > David, > > One thing you can can try is after you clone cordova-cli > > edit cordova-cli package.json to point to a Git URL instead of npm for > > plugman, this is before running npm install on cli > > > > from this: > > "plugman": "0.14.x", > > > > pick one: base on [1] > > "plugman": "apache/cordova-plugman.git#0.14.x", > > "plugman": "git://git.apache.org/cordova-plugman.git#0.14.x", > > "plugman": " > > https://git-wip-us.apache.org/repos/asf/cordova-plugman.git#0.14.x", > > > > git clone cordova-cli > > edit package.json > > npm install > > > > no need to run "npm install" a second time. > > > > [1]: https://npmjs.org/doc/json.html#Git-URLs-as-Dependencies > > > > --Carlos > > > > > > On Tue, Oct 29, 2013 at 8:06 AM, David Kemp wrote: > > > > > There is an interesting issue with detecting change though. > > > The failure is detected easily because a commit to CLI triggered a > build. > > > The only reason I knew it was fixed was because a commit to JS caused a > > > rebuild on the Master branches (which worked). > > > > > > Ideally, updating the plugman NPM module would have been detected an > > > triggered a build. I am not sure I can track that... > > > > > > > > > On Tue, Oct 29, 2013 at 7:56 AM, David Kemp wrote: > > > > > > > > > > > The CI always uses master cli and master plugman using this > procedure: > > > > > > > > git clone CLI > > > > npm install (installs the 'wrong' plugman) > > > > delete the node-modules/plugman directory > > > > git clone plugman (into node-modules/plugman) > > > > npm install (plugman) > > > > > > > > When CLI refers to a plugman that does not exist, the first 'npm > > install' > > > > fails and aborts the test. > > > > If plugman was the last reference, I suppose that the error could > > > possibly > > > > be ignored and move on anyway, but I am not sure that the npm install > > > would > > > > really be complete even then (post install tasks?). > > > > > > > > If there is a better way to override the npm install behaviour, I > would > > > be > > > > happy to give it a try. > > > > > > > > The commit that fixed the problem was about midnight, about an hour > > > before > > > > Michal checked it. > > > > > > > > David > > > > > > > > > > > > On Tue, Oct 29, 2013 at 1:10 AM, Michal Mocny > > > wrote: > > > > > > > >> Just pulled latest cli/plugman to check that the version numbers & > > deps, > > > >> and seems that they are. So im guessing its a tooling version > > mismatch > > > on > > > >> the CI machine (using dev cli with released plugman). > > > >> > > > >> -Michal > > > >> > > > >> > > > >> On Tue, Oct 29, 2013 at 12:57 AM, Michal Mocny > > > > >> wrote: > > > >> > > > >> > If you are using both versions off master, why are you getting > that > > > >> error > > > >> > message? > > > >> > > > > >> > Seems it may happen if using master CLI and running npm install > > > without > > > >> > linking plugman first? > > > >> > > > > >> > > > > >> > On Mon, Oct 28, 2013 at 9:52 PM, Steven Gill < > > stevengill97@gmail.com > > > >> >wrote: > > > >> > > > > >> >> That is because I pushed plugman + cli to m --047d7bdc119add3bac04e9e3209a--