Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-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 74F031098B for ; Tue, 27 May 2014 21:16:48 +0000 (UTC) Received: (qmail 35605 invoked by uid 500); 27 May 2014 21:16:47 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 35553 invoked by uid 500); 27 May 2014 21:16:47 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 35545 invoked by uid 99); 27 May 2014 21:16:47 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 May 2014 21:16:47 +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 shadowsor@gmail.com designates 209.85.128.182 as permitted sender) Received: from [209.85.128.182] (HELO mail-ve0-f182.google.com) (209.85.128.182) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 May 2014 21:16:43 +0000 Received: by mail-ve0-f182.google.com with SMTP id sa20so11661837veb.27 for ; Tue, 27 May 2014 14:16:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=irDGQDPbhBFHbCekHCr6mc2RmGJ8guCa6edFWETRLKs=; b=K0c33hyshbQ6P10H/ViwxKVdRcelZ9uueWBh9XSw7zaTJMFj8XktZKlLDRpW6/UNhb bvippYZw1+yktwfl+68kGkmbVQYiv1A3roerverkw2W+F0TEsgXP7cH7KF7hS4gGTrej MtzVody4utjEKvUm87N0iHsLWXSmvsCErrK8zf3qVN22mdj0c/jEkuIBdTJi+PCmFoht GyA3sUsDiNe6m0/8ltBEcHSWfAeHDFjT3nX4bPMqhTWxnGgmvpoPsYwz5O8dWsceA/lk +R3XPuz7ioL/+bMJt+eD/lUccfTUJ35Ka94RdoI4cIZnvXFNqrZhdb0kfI8xqib9/BUw 6csg== MIME-Version: 1.0 X-Received: by 10.52.138.14 with SMTP id qm14mr3709193vdb.49.1401225380053; Tue, 27 May 2014 14:16:20 -0700 (PDT) Received: by 10.52.190.170 with HTTP; Tue, 27 May 2014 14:16:20 -0700 (PDT) In-Reply-To: References: Date: Tue, 27 May 2014 15:16:20 -0600 Message-ID: Subject: Re: marvin master changes in 4.4-forward From: Marcus To: "dev@cloudstack.apache.org" Content-Type: multipart/alternative; boundary=bcaec51b1daf074b5a04fa683543 X-Virus-Checked: Checked by ClamAV on apache.org --bcaec51b1daf074b5a04fa683543 Content-Type: text/plain; charset=UTF-8 My impression was that we should change as little as possible once a release is cut. We fix bugs, but we don't change code just to make it more maintainable, for fear of introducing more bugs or regressing the known state of the release. That aside, this fix is fairly minor so I'll probably just drop it. The only question that remains is what people should do going forward when a change needs to be made to an RC branch that is incompatible with its "-forward" branch. On Tue, May 27, 2014 at 2:23 PM, Daan Hoogland wrote: > Marcus, > > I don't see why only fixes should go in 4.4. It should have been > announced before feature freeze but there might be good reasons to > refactor if it improves maintainability or removes bugs. You can > revert the related commit and apply yours. or mix them. > > regards > --bcaec51b1daf074b5a04fa683543--