Return-Path: X-Original-To: apmail-subversion-dev-archive@minotaur.apache.org Delivered-To: apmail-subversion-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B4FB01763B for ; Tue, 1 Sep 2015 15:14:43 +0000 (UTC) Received: (qmail 38111 invoked by uid 500); 1 Sep 2015 15:14:43 -0000 Delivered-To: apmail-subversion-dev-archive@subversion.apache.org Received: (qmail 38062 invoked by uid 500); 1 Sep 2015 15:14:43 -0000 Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@subversion.apache.org Received: (qmail 38046 invoked by uid 99); 1 Sep 2015 15:14:43 -0000 Received: from Unknown (HELO spamd4-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Sep 2015 15:14:43 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd4-us-west.apache.org (ASF Mail Server at spamd4-us-west.apache.org) with ESMTP id C8CF9C0FA1 for ; Tue, 1 Sep 2015 15:14:42 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd4-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -0.121 X-Spam-Level: X-Spam-Status: No, score=-0.121 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd4-us-west.apache.org (amavisd-new); dkim=pass (1024-bit key) header.d=btopenworld.com Received: from mx1-us-east.apache.org ([10.40.0.8]) by localhost (spamd4-us-west.apache.org [10.40.0.11]) (amavisd-new, port 10024) with ESMTP id 9uYUOqK4yqqL for ; Tue, 1 Sep 2015 15:14:41 +0000 (UTC) Received: from rgout01.bt.lon5.cpcloud.co.uk (rgout01.bt.lon5.cpcloud.co.uk [65.20.0.178]) by mx1-us-east.apache.org (ASF Mail Server at mx1-us-east.apache.org) with ESMTP id 4D2DD5070E for ; Tue, 1 Sep 2015 15:14:41 +0000 (UTC) X-OWM-Source-IP: 209.85.213.43 (US) X-OWM-Env-Sender: julianfoad@btinternet.com X-CTCH-RefID: str=0001.0A090201.55E5C0DA.0046,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 X-Junkmail-Premium-Raw: score=7/50,refid=2.7.2:2015.8.4.134216:17:7.944,ip=209.85.213.43,rules=__PHISH_SPEAR_HTTP_RECEIVED, __YOUTUBE_RCVD, __MIME_VERSION, __IN_REP_TO, __REFERENCES, __HAS_FROM, __HAS_MSGID, __SANE_MSGID, __BOUNCE_CHALLENGE_SUBJ, __BOUNCE_NDR_SUBJ_EXEMPT, __TO_MALFORMED_2, __CT, __CT_TEXT_PLAIN, CT_TEXT_PLAIN_UTF8_CAPS, __HELO_GMAIL, __STOCK_PHRASE_25, __SUBJ_ALPHA_NEGATE, __FORWARDED_MSG, BODYTEXTP_SIZE_3000_LESS, BODY_SIZE_2000_2999, __MIME_TEXT_ONLY, __RDNS_GMAIL, HTML_00_01, HTML_00_10, BODY_SIZE_5000_LESS, WEBMAIL_SOURCE, __PHISH_SPEAR_STRUCTURE_1, __PHISH_SPEAR_STRUCTURE_2, BODY_SIZE_7000_LESS, NO_URI_FOUND, REFERENCES, NO_URI_HTTPS, __DQ_NEG_IP, __DQ_NEG_HEUR X-CTCH-Spam: Unknown Received: from mail-vk0-f43.google.com (209.85.213.43) by rgout01.bt.lon5.cpcloud.co.uk (8.6.122.06) (authenticated as julianfoad@btopenworld.com) id 55AD674704979780 for dev@subversion.apache.org; Tue, 1 Sep 2015 16:14:34 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btopenworld.com; s=btcpcloud; t=1441120481; bh=D/T2luzg2r+9yfVorsdtE7efR8+tGuYDUoxz3tW8ACY=; h=MIME-Version:In-Reply-To:References:From:Date:Message-ID:Subject:To:Cc; b=T5wV3eDsomxFV5JZjSFhO4ap1rYfcWt7/961oJXj9l2DOpsVOIX0o2265AvU1xQqh7dS+FK6uy7QTRFtohrIaGzxW19FmesFsPJStZwpsoIXCSGlRoc1HERjnuA4Jt3LsjNRFqNoiyo73Oh9Bk/twxkO1VwteYlNB2jOcDmJx6U= Received: by vkbf67 with SMTP id f67so53098812vkb.0 for ; Tue, 01 Sep 2015 08:14:32 -0700 (PDT) X-Received: by 10.52.110.5 with SMTP id hw5mr10568445vdb.7.1441120472698; Tue, 01 Sep 2015 08:14:32 -0700 (PDT) MIME-Version: 1.0 Received: by 10.31.6.82 with HTTP; Tue, 1 Sep 2015 08:14:13 -0700 (PDT) In-Reply-To: <55E0B068.5030608@egosoft.com> References: <55E0B068.5030608@egosoft.com> From: Julian Foad Date: Tue, 1 Sep 2015 16:14:13 +0100 Message-ID: Subject: Re: E195016 on trying a normal merge / is_reintegrate_like determined correctly? To: Stefan Hett Cc: "dev@subversion.apache.org" Content-Type: text/plain; charset=UTF-8 Thanks for the query, Stefan. I ony have time for a quick look just now; I'll take a more detailed look at this later this week. But one thing I can say right away... (below) Stefan Hett wrote: > running the attached test script brings up the following error: > "E195016: Reintegrate can only be used if revisions 2 through 8 were > previously merged from [branchURL] to the reintegrate source, but this is > not the case: > trunk > Missing ranges: branches/A:2,5" > > The weird thing here (at least weird in my understanding) is that it tries > to do a reintegration merge even though I'm trying to perform a catach-up > merge with trunk on the branch. > > Looking at merge.c at line 12462 the following if-check determines it's a > "reintegrate-like" merge and therefore tries to do a reintegration merge, > instead of a "normal" merge: > if (base_on_source->rev >= base_on_target->rev) > > base_on_source->rev = 3 > base_on_target->rev = 4 > > I'm wondering whether this if-check suffices here to determine whether it is > a reintegration merge or not, given my test-scenario which has cherry-picked > revisions in both directions (trunk <-> branch). > To me it looks like the check relies on cherry-picking only being performed > in one or the other direction, because otherwise the > "youngest-complete-synced-point" couldn't be used to determine the direction > for reintegration merges... By "complete synced point" I would mean a branch-revision (such as r4 on target) at which *all* changes up to this point have been merged to the other branch. A cherry-picked revision after that (say r6 on target was merged to source) wouldn't make target@6 "completely" merged to source because target@5 is missing. Maybe that helps to explain it. Anyway I'll look in more detail, and look at your script, later. Regards, - Julian > Or am I missing something? > > Sorry if I'm completely off here --- not too familiar with all the details > of the automerge and it's twisiting my brain trying to follow/interpret the > design details from code.