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 910FFE36C for ; Fri, 18 Jan 2013 14:46:18 +0000 (UTC) Received: (qmail 6675 invoked by uid 500); 18 Jan 2013 14:46:18 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 6378 invoked by uid 500); 18 Jan 2013 14:46:14 -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 6312 invoked by uid 99); 18 Jan 2013 14:46:11 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 18 Jan 2013 14:46:11 +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 (nike.apache.org: domain of mmocny@google.com designates 209.85.160.51 as permitted sender) Received: from [209.85.160.51] (HELO mail-pb0-f51.google.com) (209.85.160.51) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 18 Jan 2013 14:46:05 +0000 Received: by mail-pb0-f51.google.com with SMTP id ro12so2072640pbb.38 for ; Fri, 18 Jan 2013 06:45:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:x-received:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; bh=k4k34mNxqZ2fNIOB1UTZ1kz7PWfyZGLCLJuBiQMquKM=; b=glzvG6bUzzGhnw9NvAgjgcJ3YAuujC762HyA5iLzkEymJg/QA3wunL0CH9+Ad01e6K jHixW3jX2FIIKni7XsyLZ9L1hc56EUvuedHLh97wjyg3wlKJS/Bc6VcD+qgTxzm2jd3d uilGJl9orr6cOvonCb80iQB1f7g4IUx4dfJ8T+x5xgbOJuDMhAg1JH2+QOeNacogFj6j 9Mp2dRIPctE10W7koCk7VShGNZhZuUFgSsGF17zeDR8FQNb1hiqDMeqN4C91FJmaHSUR j4otnuBxynJsMA+7biTVrNUKvrpQjbCDlWLAPUqe6e/036MjAicSSipO1r0JDGzOCrIP Xj+g== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:x-received:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; bh=k4k34mNxqZ2fNIOB1UTZ1kz7PWfyZGLCLJuBiQMquKM=; b=IMy2Wp97IIqpaE8XyPlVVju/Oqy6z2Rgo0Ew0mVEsV7M3Tz9udwsn+77JFCb4QbEHZ rn0xFmOgrGSmQTrVh9u9v1weYvzjk7QZiDDIa1XCOnudXs11QE1WR5qzbUOh9vI97zyr i9QbhkWBnbR7vXhC7SiSZqDvSef9P5gkEkg9A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:x-received:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type :x-gm-message-state; bh=k4k34mNxqZ2fNIOB1UTZ1kz7PWfyZGLCLJuBiQMquKM=; b=JuyQAYafeRBOJGg/3hWvc2Pe/NTCRmR92Epaa527eUkCC4EK2Fm1OltyNnrjTSmjnq LPux92jtJnhsTXo9BPIcbOzA6sY25vg/C6hv22EVmKCa3fB44XrdTRhDPgC7vu8NtYC/ 5egcj11t2RFxiOq0JsHNuq7REgWogEjUzbHHHPHdE568r+5tNKdLfOrvwl+s0y+VxpP/ l6jI7LgrkvjJDhSNHdb/v2tGdFr+JqR8yRfGPopy+S7lck3CQA0c6QRUXvU/vPup3zwm 2W//ATZtJwiCq7C5ctogacu7Cex5/Hs2ZSGfeLgmRhAsoAfnJgZUE84r36GQqWyWighx EUJQ== MIME-Version: 1.0 X-Received: by 10.68.231.40 with SMTP id td8mr5965933pbc.48.1358520343582; Fri, 18 Jan 2013 06:45:43 -0800 (PST) Sender: mmocny@google.com Received: by 10.68.195.10 with HTTP; Fri, 18 Jan 2013 06:45:43 -0800 (PST) In-Reply-To: References: Date: Fri, 18 Jan 2013 09:45:43 -0500 X-Google-Sender-Auth: Pvcjj3a_7KekJ1SJqWe-krrfznM Message-ID: Subject: Re: Cordova-CLI: Version handling From: Michal Mocny To: dev@cordova.apache.org Content-Type: multipart/alternative; boundary=047d7b339b737fd6ba04d3912a18 X-Gm-Message-State: ALoCoQk5q+Br7o1KlONWi+HRkYWxqpPMWHenJ9vvpquxyBo2tltwNHZXi4uQSB7GDEPvspvI35POU1WmyaBYVFOyXsP+olkAMsTGeAHR5MyCkFjaM77bvZFEAFqTi/QCDaOSj6xTPwDtQtWOk/58e1I4iJfPeRPiHBVyKLdn4xtnDR4bMi+j4AjHHjUqWu+m1X7sOQI7Av10 X-Virus-Checked: Checked by ClamAV on apache.org --047d7b339b737fd6ba04d3912a18 Content-Type: text/plain; charset=ISO-8859-1 On Thu, Jan 17, 2013 at 8:05 PM, Andrew Grieve wrote: > On Thu, Jan 17, 2013 at 3:17 PM, Filip Maj wrote: > > > > > > > On 1/17/13 11:57 AM, "Andrew Grieve" wrote: > > > > >Looking at iOS as an example: > > > > > >To do an upgrade: > > >1. Close xcode > > >2. Delete CordovaLib & CordovaLib.xcodeproj > > >3. Copy in new CordovaLib & CordovaLib.xcodeproj > > > > > >Sometimes: > > >- Add extra frameworks to your App's target > > > > > > > > >The thing I'd be afraid of with a script, is if they've modified their > > >CordovaLib or their CordovaLib.xcodeproj in any way. Maybe we can just > > >mitigate this by showing them a warning and a confirm prompt before > going > > >ahead with it? > > > > If they've modified it then I think expecting a stock upgrade or > downgrade > > script to work is unrealistic. We can't solve all possible problems our > > users will encounter.. > > > Yeah, ideas along these lines would be to do store checksums of the files > when installing, and then using those to see if any have changed. If > they've changed, then do a silent update, if they *have* changed, then show > a warning prompt and blow away their changes. A warning every time would > likely be fine too, since the "use source control" argument applies here > too. What about just initializing a local git repo (I cannot recall if cli was to depend on git or not already?). This has the added advantage that we can checkout user changes into a branch, upgrade, and then give them the option to rebase. > > > > > > >I don't think we should put effort into downgrading. People should be > > >using > > >source-control for that. Again though, a warning message + confirm > prompt > > >should suffice here? > > > > Agree but wanted to open up the discussion anyways. > > > -1 to downgrade support. > > >Fil, what are your thoughts around versioning the cordova-cli tool? > > > > Yep. The tools are now following the point releases of the main project > as > > of 2.3. However, the patch revision for the CLI tools are on their own > > track. This way I can roll out fixes and updates quickly, on a > per-release > > basis. I think currently the tools are at 2.3.4 since I committed four > > patches since enabling support for cordova 2.3.0. Make sense? > > > > Makes sense! > > > > > > >I think > > >I'd like it to work so that cordova-cli *is* versioned along with the > rest > > >of it, but that it would work with older cordova projects without > forcing > > >them to upgrade. > > > > Not sure if this is doable. The maintenance necessary to maintain old > > cordova structures would be a lot of work. For example, plugins.xml vs. > > config.xml support. Not only did we change the name, but now the > structure > > is diverging greatly from what this file used to be. > > > > Good point. Might be too early to tell if this will be important or not, > but it might be worth making it so that multiple versions can be installed. > e.g. If a user has multiple projects running on different versions of > Cordova. Maybe if we allow this, we could even have the tools detect the > version and just pass through calls to the correct version of the CLI > tools? Either way not something we have to support from the get-go. > > > > > > > > > > > > > > > > > > > > > > > > > > > >On Thu, Jan 17, 2013 at 2:15 PM, Filip Maj wrote: > > > > > >> Questions: how to handle moving between cordova versions? Do we want > to > > >> support both upgrading and downgrading? How to land support for this > in > > >> cordova-cli? > > >> > > >> My answers: > > >> > > >> - how to handle moving between cordova versions? > > >> Suggestion: add an "upgrade" (and "downgrade") script to each > platform > > >> implementation, which handles moving from the previous version to the > > >> current and vice-versa. > > >> - Do we want to support both upgrading and downgrading? > > >> IMO: Yes. > > >> - How to land support for this in cordova-cli? > > >> > > >> CLI tools shell out to above-mentioned scripts. > > >> > > >> In my mind we would draw a line in the sand (let's say, 2.4, or 2.5) > on > > >> which Cordova version(s) support upgrade and downgrade scripts. From > > >>that > > >> point on we can tell our users "OK, if you run 2.4 (or 2.5) or newer, > > >>you > > >> can use these scripts. Or you can use cordova-cli if you want a > unified > > >> tool for this. If you are below this version, take a look at the > > >> "Upgrading" guides written for the platforms in the docs." > > >> > > >> Thoughts? Any other suggestions? > > >> > > >> > > > > > --047d7b339b737fd6ba04d3912a18--