Login | Register
My pages Projects Community openCollabNet

Discussions > dev [DISABLED] > Re: 1.5.x currently doesn't build

subversion
Discussion topic

There will be a brief maintenance window every Friday at 17:00 Pacific.
For further details, see CollabNet's maintenance and upgrade policy.

Back to topic list

Re: 1.5.x currently doesn't build

Author "Paul Burba" <ptburba at gmail dot com>
Full name "Paul Burba" <ptburba at gmail dot com>
Date 2008-12-17 15:03:44 PST
Message On Wed, Dec 17, 2008 at 11:05 AM, Paul Burba <ptburba at gmail dot com> wrote:
> On Wed, Dec 17, 2008 at 11:00 AM, Hyrum K. Wright
> <hyrum_wright@mai​l.utexas.edu> wrote:
>> Paul Burba wrote:
>>> On Wed, Dec 17, 2008 at 10:06 AM, Hyrum K. Wright
>>> <hyrum_wright@mai​l.utexas.edu> wrote:
>>>> Julian Foad wrote:
>>>>> On Tue, 2008-12-16 at 13:50 -0600, Hyrum K. Wright wrote:
>>>>>> 1.5.x currently doesn't build due to the merge in r34734, with this log message:
>>>>>> --------------------​--------------------​--------------------​------------
>>>>>> r34734 | hwright | 2008-12-16 12:04:52 -0600 (Tue, 16 Dec 2008) | 15 lines
>>>>>>
>>>>>> Merge r34385, r34393 from trunk:
>>>>>>
>>>>>> * r34385, r34393
>>>>>> Fix broken merge when the merge target's natural history includes
>>>>>> resurrections.
>>>>>> Notes:
>>>>>> r34385 is a new test. r34393 is the fix.
>>>>>> Justification:
>>>>>> This problem was encountered 'in the wild' - see
>>>>>> http://svn.haxx.se/d​ev/archive-2008-11/0​618.shtml. When encountered
>>>>>> cherry harvest merges are completely broken.
>>>>>> Votes:
>>>>>> +1: pburba, stylesen, cmpilato
>>>>>>
>>>>>>
>>>>>> It turns out that r34393 introduced a call to
>>>>>> svn_mergeinfo__get_r​ange_endpoints(), which was introduced on trunk in r34306.
>>>>>> r34306 is currently awaiting review as part of the
>>>>>> 1.5.x-reintegrate-improvements branch.
>>>>>>
>>>>>> I created the 1.5.x-build-fixes branch to attempt to fix the build, but I think
>>>>>> that merging the reintegrate-improvements branch would be the better solution.
>>>>> On 1.5.x-build-fixes I get:
>>>>>
>>>>> subversion/libsvn_su​br/mergeinfo.c:1501:​ no previous prototype for
>>>>> 'svn_mergeinfo__cata​log_to_formatted_str​ing'
>>>>> subversion/libsvn_su​br/mergeinfo.c:1565:​ no previous prototype for
>>>>> 'svn_mergeinfo__to_f​ormatted_string'
>>>>>
>>>>> Those two functions appear to be redundant in the 1.5.x-build-fixes
>>>>> branch. That means in one sense that the warnings are harmless, but in
>>>>> another sense it's wrong to include them because they are not part of
>>>>> the fix. They were added to that branch as part of r34740 and then a
>>>>> further change r34741 was made to support them.
>>>>>
>>>>> Shouldn't we revert the bad merge straight away? All this proposing to
>>>>> fix it, and voting on the fix, while the branch is still broken, is
>>>>> getting in the way of reviewing further back-ports.
>>>> My preference (and the correct way to do it, IIUC) is to merge the reintegrate
>>>> improvements, which includes r34306 in its entirety, to 1.5.x. That branch is
>>>> still lacking a +1, so maybe we should focus review efforts there.
>>>
>>> Ok, if the reintegrate improvements are backported we can do that,
>>> making all of this moot. Assuming they are not, then we can revert
>>> the "bad" merge of r34385 and r34393 and then I can renominate those
>>> changes along with (the parts? of) r34306 that are required.
>>>
>>> The only real question is how long to wait to find out if the
>>> reintegrate improvements are going in or not. Is anyone looking at
>>> those right now?
>>
>> Paul,
>> I'm in the midst of merging the build-fixes branch to 1.5.x (running tests now)
>> so 1.5.x will be back to a stable state for further review. Given that there
>> may be conflicts between build-fixes and the reintegrate-improvements branch, do
>> you want to bring reintegrate-improvements up-to-date with 1.5.x after I merge?
>
> Will do

Done. I've locally reintegrated the 1.5.x-reintegrate-improvements to
1.5.x and ran the [FSFS]x[LOCAL] tests with no failures.

One oddity, when I did the merge I got an unexpected conflict on merge.c:

>svn info
Path: .
URL: http://svn.collab.ne​t/repos/svn/branches​/1.5.x
Repository Root: http://svn.collab.net/repos/svn
Repository UUID: 612f8ebc-c883-4be0-9​ee0-a4e9ef946e3a
Revision: 34802
Node Kind: directory
Schedule: normal
Last Changed Author: hwright
Last Changed Rev: 34799
Last Changed Date: 2008-12-17 15:20:41 -0500 (Wed, 17 Dec 2008)

>svn merge --reintegrate http://svn.collab.ne​t/repos/svn/branches​/1.5.x-reintegrate-i​mprovements .
--- Merging differences between repository URLs into '.':
U STATUS
 G www\images\subvers​ion-diagram.png
 G www\images\subvers​ion_logo-200x173.png​
 G www\images\subvers​ion_logo_hor-468x64.​png
 G www\images\subvers​ion_logo-384x332.png​
 G COMMITTERS
 G notes\tree-conflict​s\scratch-pad.txt
U subversion\libsvn_f​s_base\tree.c
U subversion\libsvn_fs_base\fs.c
U subversion\include​svn_mergeinfo.h
U subversion\include\svn_client.h
U subversion\include​private\svn_mergei​nfo_private.h
 G subversion\include\private
UG subversion\libsvn_wc\adm_ops.c
U subversion\libsvn_s​ubr\mergeinfo.c
 G subversion\libsvn_subr
Conflict discovered in 'subversion/libsvn_c​lient/merge.c'.
Select: (p) postpone, (df) diff-full, (e) edit,
        (mc) mine-conflict, (tc) theirs-conflict,
        (s) show all options: p
CG subversion\libsvn_c​lient\merge.c
U subversion\libsvn_client\ra.c
U subversion\libsvn_client\copy.c
 G subversion\bindings\swig
U subversion\mod_dav_svn\mirror.c
U subversion\tests\c​mdline\revert_tests​.py
U subversion\tests\c​mdline\copy_tests.p​y
U subversion\tests\c​mdline\update_tests​.py
UG subversion\tests\c​mdline\svntest\act​ions.py
U subversion\tests\c​mdline\merge_tests.​py
U subversion\libsvn_r​a_svn\cyrus_auth.c
U subversion\libsvn_r​a_svn\internal_auth​.c
U subversion\libsvn_r​a_svn\ra_svn.h
 G CHANGES
 G .
Summary of conflicts:
  Text conflicts: 1

The conflict was on the whole file:

>svn diff subversion\libsvn_c​lient\merge.c
Index: subversion/libsvn_cl​ient/merge.c
====================​====================​====================​=======
--- subversion/libsvn_cl​ient/merge.c (revision 34802)
+++ subversion/libsvn_cl​ient/merge.c (working copy)
@@ -1,3 +1,4 @@
+<<<<<<< .working
 /*
  * merge.c: merging
  *
@@ -7086,3 +7087,7473 @@
                                target_wcpath, recurse, ignore_ancestry, force,
                                dry_run, NULL, ctx, pool);
 }
+=======
+/*
+ * merge.c: merging

<SNIP the whole right side file added>

+ target_wcpath, recurse, ignore_ancestry, force,
+ dry_run, NULL, ctx, pool);
+}
+>>>>>>> .merge-right.r34802

Since there were no changes to merge.c on 1.5.x since I last synched
1.5.x-reintegrate-improvements with it I just ran svn resolve --accept
theirs-full subversion\libsvn_c​lient\merge.c and this gave what I
expected.

Anyone know what happened here? Am I missing something simple?

Paul

« Previous message in topic | 7 of 14 | Next message in topic »

Messages

Show all messages in topic

Subject
Author
Full name
Date posted
+
1.5.x currently doesn't build "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> 2008-12-16 11:50:35 PST
     Re: 1.5.x currently doesn't build julianfoad Julian Foad 2008-12-17 05:30:52 PST
         Re: 1.5.x currently doesn't build "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> 2008-12-17 07:06:50 PST
             Re: 1.5.x currently doesn't build "Paul Burba" <ptburba at gmail dot com> "Paul Burba" <ptburba at gmail dot com> 2008-12-17 07:49:44 PST
                 Re: 1.5.x currently doesn't build "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> "Hyrum K dot Wright" <hyrum_wright at mail dot utexas dot edu> 2008-12-17 08:00:43 PST
                     Re: 1.5.x currently doesn't build "Paul Burba" <ptburba at gmail dot com> "Paul Burba" <ptburba at gmail dot com> 2008-12-17 08:06:02 PST
                         Re: 1.5.x currently doesn't build "Paul Burba" <ptburba at gmail dot com> "Paul Burba" <ptburba at gmail dot com> 2008-12-17 15:03:44 PST
                             Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] julianfoad Julian Foad 2008-12-18 05:54:49 PST
                                 Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] "Paul Burba" <ptburba at gmail dot com> "Paul Burba" <ptburba at gmail dot com> 2008-12-18 06:42:30 PST
                                 Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] cmpilato C. Michael Pilato 2008-12-18 07:24:47 PST
                                     Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] cmpilato C. Michael Pilato 2008-12-22 06:35:46 PST
                                 Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] "Paul Burba" <ptburba at gmail dot com> "Paul Burba" <ptburba at gmail dot com> 2008-12-18 08:10:59 PST
                                     Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] markphip Mark Phippard 2008-12-18 08:26:44 PST
                                     Re: Problems merging the reintegrate-improvements branch [was: 1.5.x currently doesn't build] julianfoad Julian Foad 2008-12-18 11:05:03 PST
Messages per page: