Return-Path: X-Original-To: apmail-commons-dev-archive@www.apache.org Delivered-To: apmail-commons-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 8BFE619680 for ; Mon, 28 Mar 2016 00:49:40 +0000 (UTC) Received: (qmail 31654 invoked by uid 500); 28 Mar 2016 00:49:40 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 31526 invoked by uid 500); 28 Mar 2016 00:49:40 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 31514 invoked by uid 99); 28 Mar 2016 00:49:39 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 28 Mar 2016 00:49:39 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 64CD31A0120 for ; Mon, 28 Mar 2016 00:49:39 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 1.178 X-Spam-Level: * X-Spam-Status: No, score=1.178 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=2, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd2-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=yahoo.com Received: from mx2-lw-us.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id MFmEuOXZWE3N for ; Mon, 28 Mar 2016 00:49:36 +0000 (UTC) Received: from nm22-vm0.bullet.mail.ne1.yahoo.com (nm22-vm0.bullet.mail.ne1.yahoo.com [98.138.91.60]) by mx2-lw-us.apache.org (ASF Mail Server at mx2-lw-us.apache.org) with ESMTPS id 823AB5F23C for ; Mon, 28 Mar 2016 00:49:36 +0000 (UTC) Received: from [98.138.101.128] by nm22.bullet.mail.ne1.yahoo.com with NNFMP; 28 Mar 2016 00:49:28 -0000 Received: from [98.138.89.240] by tm16.bullet.mail.ne1.yahoo.com with NNFMP; 28 Mar 2016 00:49:28 -0000 Received: from [127.0.0.1] by omp1013.mail.ne1.yahoo.com with NNFMP; 28 Mar 2016 00:49:28 -0000 X-Yahoo-Newman-Property: ymail-3 X-Yahoo-Newman-Id: 840335.87899.bm@omp1013.mail.ne1.yahoo.com X-YMail-OSG: 3YALIK8VM1m6wWdpgew7brPPuUvNdNJzDGJGK0BTnWhYTjYpCcvc2GZINB_bQM5 DO4kvrzGaUHkPYt7sPiO3jsWeU8aBpfw3t7unXRJYuln6X7dHmXZKcXfB5iyphDaMqa8vkGKi7aa SYzLd3c9d2QeHP7siGU0rEHcCgUE6xWnMx6.F.8_V3BfAN1d2oFPPhBkHToW3LJngvTmKKkQzfC0 BPPhiXhtdBVchaA.gk6ybqrWbC0dA1M3w9nrQbPIr6iTKq8JdL9AHX2YBiscSRylMQWu1zXvy1R6 HHmxX69XCX8cevKusbF2wxCbGPeXckP25FOAHMKqi.JcwGt8hRh.gHCduzUkML3q0xzomQ7GewCR WkSr8j07Jg5cNdB9Ao.cplZ2wGpbKOQDqj2h2yhxemxElD3AUYZdSxD6URt12kIPP1pBipImFG.u X47N36WAI7JLMJHhAXTMM6UgUoSdB__DAhJJXaYjU6bsr_zxQ4jc.72grT2zZnlmKG13VAa_hwko 4cTmr2y7FM8EmCTmd3_cYtPFk2FTE Received: by 98.138.105.240; Mon, 28 Mar 2016 00:49:28 +0000 Date: Mon, 28 Mar 2016 00:49:27 +0000 (UTC) From: Al Chou Reply-To: Al Chou To: Commons Developers List Message-ID: <258366476.1360521.1459126167994.JavaMail.yahoo@mail.yahoo.com> In-Reply-To: References: <77698a8bd514adeebd721fd1b682a5ec@scarlet.be> <56F55FAA.20405@nrl.navy.mil> Subject: Re: [Math] "rebase" vs "merge" (Was: RNG refactoring) MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_1360520_2105796411.1459126167989" ------=_Part_1360520_2105796411.1459126167989 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Agreed, changing the commit ancestry history of a branch that other people = have already (even merely potentially) fetched and checked out locally is g= enerally frowned upon. =C2=A0Interesting to hear that anyone ever has gotte= n consensus to do so via "pull --rebase"! =C2=A0On rare occasions at my job= we have needed to "push --force" a publicly shared branch to clear up a pr= oblem, and we do get consensus before doing so, but no one enjoys those sit= uations. Al=20 On Sunday, March 27, 2016 5:44 PM, Matt Sicker wrote= : =20 In general, don't use rebase on branches that other people are also using without first making sure everyone is willing to also rebase their own local copies. To do that, they can run "git pull --rebase" to update when you've force-pushed a rebased branch. This is generally looked down upon, but is sometimes necessary. On 26 March 2016 at 17:33, wrote: > Unlike a regular merge, rebase applies a branch's commits' changes onto > the specified commit (which is typically identified by specifying a branc= h > name), in the same order as they appear in the branch that is being > rebased. The name of the command says that you are redefining the "base", > in other words, the starting point, of the branch. Depending on the chang= es > made by those commits, you may see what appears to be the same > commit/change applied more than once. See http://ProGit.org/book for > great explanations of this and other Git features. > > Al > > On Mar 26, 2016, 12:01 -0700, Gilles, wrote= : > > On Fri, 25 Mar 2016 11:56:26 -0400, Evan Ward wrote: > > > I'm not sure if this is the problem, but a good rule of thumb is that > > > if > > > you have pushed a commit > > > > Did I? > > > > What I wanted is > > * publish code ("feature-MATH-1335") > > * publish code ("feature-MATH-1158") that requires the new code > > present in "feature-MATH-1335" > > > > So I "rebase"d "feature-MATH-1158" on "feature-MATH-1335" in order > > to incorporate everything from there. [What "git" tells during this > > operation looks like the right thing to do.] > > > > By the way, I did the same with your change: I "rebase"d all my local > > branches on "develop" after your merged your change to that branch. > > > > What is the difference with "merge" and if "merge" should have been > > used, then when does one use "rebase"? > > > > Maybe I did the right thing; and it's just normal that there is an > > email flood in such cases... > > > > Thanks, > > Gilles > > > > > you should merge it instead of rebasing it. It > > > looks to me like 6ddf476 and ce8c82f are the same, so I think when > > > you > > > ran rebase it put it on top of the bug fix I pushed up recently, > > > duplicating the commit. > > > > > > Best Regards, > > > Evan > > > > > > On 03/25/2016 11:34 AM, Gilles wrote: > > > > Hi. > > > > > > > > Last week, and just now, I've pushed local branches that handle > > > > the following issues (and others, either related, or set to > > > > "Won't fix [in current code]" in JIRA[1]): > > > > > > > > https://issues.apache.org/jira/browse/MATH-1335 > > > > https://issues.apache.org/jira/browse/MATH-1336 > > > > https://issues.apache.org/jira/browse/MATH-1337 > > > > https://issues.apache.org/jira/browse/MATH-1339 > > > > https://issues.apache.org/jira/browse/MATH-1158 > > > > https://issues.apache.org/jira/browse/MATH-1338 > > > > > > > > [I've just seen that for branch "feature-MATH-1158" which is > > > > "git rebase"d on "feature-MATH-1335", the push is recreating > > > > all the MATH-1335 commits (as guessed from the flood of emails). > > > > Something I was not expecting: sorry I misunderstood how this > > > > is supposed to work...] > > > > > > > > > > > > Regards, > > > > Gilles > > > > > > > > [1] See the "links" in the relevant JIRA reports. > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > > For additional commands, e-mail: dev-help@commons.apache.org > > > --=20 Matt Sicker ------=_Part_1360520_2105796411.1459126167989--