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 97C58DCBD for ; Wed, 20 Feb 2013 19:34:40 +0000 (UTC) Received: (qmail 73424 invoked by uid 500); 20 Feb 2013 19:34:40 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 73397 invoked by uid 500); 20 Feb 2013 19:34:40 -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 73387 invoked by uid 99); 20 Feb 2013 19:34:40 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 20 Feb 2013 19:34:40 +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 (nike.apache.org: domain of shazron@gmail.com designates 209.85.128.47 as permitted sender) Received: from [209.85.128.47] (HELO mail-qe0-f47.google.com) (209.85.128.47) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 20 Feb 2013 19:34:33 +0000 Received: by mail-qe0-f47.google.com with SMTP id 2so3830571qea.20 for ; Wed, 20 Feb 2013 11:34:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:content-type; bh=v9wURszKbPIXx2WPh60rxRgOY/IDDMXDpA3pt+ML7qw=; b=b2Urb1jsk0WveroOIeZ78GRTm+icOTcCFvMQiEjPloBSxbrL3DrAiQPol3C3I9PL8h EEBCbxEpw1yMdcn1hvp36TWbaUUYWSAQrZxKo2DBFwc8pxkIsS+YT9SUCoM90jGjLohd +yFXRL6ISvK2ATWkgiPW9K9Z6zoYjxhXHLj/EZz/yPEuzzxM9eBp9kUH4aTR9v5/r2mK QAwU09cNOyGZTU3n4FZUCEOdPwy6QFS3BaKqWva6oNId7nImKD2zu3YPpezj+l5JUufN 5EtFeDL0a/G326tkELcuxZeu/srIUNrxBCnqT3Px+a8Lgs+RT7CdRvn75x9vNPg0eZfp 9RLA== MIME-Version: 1.0 X-Received: by 10.224.33.208 with SMTP id i16mr5631573qad.45.1361388853126; Wed, 20 Feb 2013 11:34:13 -0800 (PST) Received: by 10.49.15.196 with HTTP; Wed, 20 Feb 2013 11:34:12 -0800 (PST) In-Reply-To: References: Date: Wed, 20 Feb 2013 11:34:12 -0800 Message-ID: Subject: Re: too long to package a release? From: Shazron To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=20cf3074d574fde71304d62d0a18 X-Virus-Checked: Checked by ClamAV on apache.org --20cf3074d574fde71304d62d0a18 Content-Type: text/plain; charset=ISO-8859-1 http://cl.ly/MOrD Master always has all the changes. Next will never have experimental changes/features after next is created, just bug fixes. On Wednesday, February 20, 2013, Joe Bowser wrote: > Honestly, this process is too complex and we should just go back to > what we were doing before. I don't think our git flow was what is > slowing us down. > > > > On Wed, Feb 20, 2013 at 11:22 AM, Filip Maj wrote: > > Ok so the flow is: if you are committing into next, always merge into > > master. Good. So the CI setup doesn't need to differentiate between > master > > and next. It can always pull from master. > > > > On 2/20/13 11:04 AM, "Andrew Grieve" wrote: > > > >>Step 5 here: http://wiki.apache.org/cordova/CommitterWorkflow > >> > >>Probably it should be "isFixingRegression" instead of "isBugFix". I'll > >>update it now. > >> > >> > >>On Wed, Feb 20, 2013 at 1:59 PM, Filip Maj wrote: > >> > >>> I noticed on iOS the commits going into next are mirrored on master. > >>> > >>> For Android that was not done. > >>> > >>> What is the correct process? > >>> > >>> On 2/20/13 10:12 AM, "Michal Mocny" wrote: > >>> > >>> >oooo I didn't know that. Thanks! > >>> > > >>> > > >>> >On Wed, Feb 20, 2013 at 1:10 PM, Becky Gibson > >>> >wrote: > >>> > > >>> >> Thank you, Michael! I do usually go a git push --dry-run to check > >>>that > >>> >>I > >>> >> am pushing what I expect but I'll try the diff as well. > >>> >> > >>> >> > >>> >> On Wed, Feb 20, 2013 at 1:07 PM, Michal Mocny > >>> >>wrote: > >>> >> > >>> >> > So there is also http://wiki.apache.org/cordova/CuttingReleases > >>>which > >>> >> may > >>> >> > be useful (esp Taggin section). > >>> >> > > >>> >> > As far as the confusion with the two branch names: "topic_branch" > >>>is > >>> >>your > >>> >> > local working branch for a bugfix/feature, and "to_be_merged" is > >>> >>really > >>> >> > "temporary_new_name_for_a_branch_to_do_rebase_in". I usually skip > >>> >>that > >>> >> > step and take the risk or rebasing in topic_branch (aside: this > may > >>> >> > negatively affect diffs if you push updates for a > >>> >>review-edit-re-review > >>> >> > cycle -- but this isn't an issue for cordova). > >>> >> > > >>> >> > Do not checkout 'next' into your master branch. Update your local > >>> >> branches > >>> >> > to include the remote next branch (with 'git pull apache' with no > >>> >>branch) > >>> >> > then you can switch to the next branch locally, apply your patch > >>> >>there, > >>> >> and > >>> >> > push to that remote branch directly. Later, merge that commit > into > >>> >> master > >>> >> > locally, and push that. > >>> >> > > >>> >> > Do not push to apache next from your local master, or else you > will > >>> >>push > >>> >> > all the changes. > >>> >> > > >>> >> > I agree, this is a little confusing, but after a few practice runs > >>>it > >>> >> > should be easy to figure out. You should probably also check what > >>> >>would > >>> >> be > >>> >> > pushed with 'git diff apache/[target-branch]' or tag on --stat to > >>> >>that to > >>> >> > just see that files that would signal a quick "uh-oh". > >>> >> > > >>> >> > I'll work to update the wiki later today, and likely others will > >>>have > --20cf3074d574fde71304d62d0a18--