Not that the things you note below aren't probably real bugs and all,
but, in a 1.5.x r30348 wc,
$ svn merge --reintegrate $SR/branches/1.5.x-r30215_at_30348
appears to DTRT for me.
Doing a 'svn diff -r30349' in the resulting wc shows me changes to
STATUS, as well as the following mergeinfo changes:
Property changes on: CHANGES
___________________________________________________________________
Modified: svn:mergeinfo
Reverse-merged /branches/1.5.x-r30215/CHANGES:r30236,30238,30245,30288
Property changes on: .
___________________________________________________________________
Modified: svn:mergeinfo
Merged /trunk:r29085-29089,29091,29094-29107,29111,29114,29117,29126-29127,29129-29133,29135-29150,29153-29164,29166,29174,29176-29186,29188-29189,29193-29194,29198-29206,29208-29251,29254-29256,29261,29267-29273,29277,29280-29281,29284,29287-29303,29305-29307,29309-29325,29327-29343,29345-29348,29358-29379,29381-29392,29397,29399,29401,29409,29412,29414-29415,29417-29423,29425-29426,29429,29433-29434,29436-29447,29449-29466,29468-29478,29482,29484,29486-29487,29489,29491,29493,29496,29498,29508,29527-29528,29531,29533,29539-29540,29542,29544,29546,29551,29553,29556,29559,29565,29567-29569,29571-29578,29581,29583,29591,29594,29600,29603,29607,29611,29613-29614,29619,29623,29625-29626,29630-29631,29634,29642,29648,29650,29656,29659-29660,29663-29664,29671-29672,29677-29680,29692,29738-29739,29742-29744,29746,29751,29763,29769-29770,29784,29787,29797,29801,29821,29824,29828,29835,29855,29858-29859,29868-29869,29876,29878,29883-29884,29895,29898,29900,29914,29920,29925,29930,29939-29940,29942,29950,29958,299
62,29968,29980,29986,29994-29997,30004,30009,30020,30030,30050,30054,30061-30062,30067,30070,30074,30086,30098,30117,30124,30130,30137,30145,30151,30159,30162,30185,30215,30221,30225-30226,30230,30243
Merged /branches/1.5.x-r30215:r30222-30235,30237,30239-30244,30246-30287,30289-30348
Merged /branches/svn-mergeinfo-enhancements:r30045-30214
I don't know if the reintegrate-produce mergeinfo or the
r30349-committed mergeinfo is better.
--dave
On Wed, Apr 9, 2008 at 7:53 AM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> Hyrum K. Wright wrote:
>
> >
> > > Hyrum - any thoughts as to what happened? I recall in IRC you were
> > > had some questions about how exactly to merge the 1.5.x-r30215 branch
> > > to 1.5.x. What did you ultimately end up doing?
> > >
> >
> > After a couple of failed attempts, the command I ultimately ended up
> running was:
> > $ svn merge -c 30236,30238,30245,30288
> https://svn.collab.net/repos/svn/branches/1.5.x-r30215
> >
>
> I cut the 1.5.x-r30215 branch from the 1.5.x branch. You should have
> needed only to do:
>
> svn merge https://svn.collab.net/repos/svn/branches/1.5.x-r30215 \
> /path/to/subversion-1.5.x
>
> I tried to replay this scenario (which ought to be time-safe, right?), but
> I got an error:
>
> $ cd projects/subversion-1.5.x
> $ svn merge http://svn.collab.net/repos/svn/branches/1.5.x-r30215@30348 .
> subversion/libsvn_ra_neon/util.c:711: (apr_err=160013)
> svn: File not found: revision 30447, path '/branches/1.5.x-r30215'
> $
>
> 'svn help merge' contains this:
>
> [...]
> 3. merge [-c M[,N...] | -r N:M ...] SOURCE[@REV] [WCPATH]
> [...]
> 3. In the third form, SOURCE can be either a URL or a working copy
> path (in which case its corresponding URL is used). SOURCE (in
> revision REV) is compared as it existed between revisions N and M
> for each revision range provided. If REV is not specified, HEAD
> is assumed. '-c M' is equivalent to '-r <M-1>:M', and '-c -M'
> does the reverse: '-r M:<M-1>'. If no revision ranges are
> specified, the default range of 0:HEAD is used. Multiple '-c'
> and/or '-r' instances may be specified, and mixing of forward
> and reverse ranges is allowed.
>
> Maybe I got bit by the default range of 0:HEAD? That's a stupid default,
> by the way, and inconsistent with our policy of allowing specified peg
> revisions dictate the upper bound on the operative revision range. But
> anyway, I tried with the obvious workaround:
>
> $ svn merge http://svn.collab.net/repos/svn/branches/1.5.x-r30215@30348 . \
> -r0:30348
> subversion/libsvn_ra_neon/util.c:711: (apr_err=160013)
> svn: Working copy path 'CHANGES' does not exist in repository
> $
>
> Ooookay. Well, I'll use 'svn log' to find a real range, then.
>
> $ svn log -q --stop-on-copy \
> http://svn.collab.net/repos/svn/branches/1.5.x-r30215@30348 \
> | grep '^r' | tail -1
> r30222 | cmpilato | 2008-04-03 13:38:30 -0400 (Thu, 03 Apr 2008)
> $
>
> (Sidebar: 'svn help log' indicates that HEAD:1 is the default revision
> range even when a peg revision to the URL is supplied. Clearly, that's a
> lie. How many bugs are we up to now?)
>
> Anyway, back to using a real range for this merge:
>
> $ svn merge http://svn.collab.net/repos/svn/branches/1.5.x-r30215@30348
> -r30222:30348 .
> --- Merging r30223 through r30348 into '.':
> U subversion/include/svn_client.h
> U subversion/libsvn_client/util.c
> U subversion/libsvn_client/mergeinfo.c
> U subversion/tests/cmdline/mergeinfo_tests.py
> U subversion/tests/cmdline/svntest/actions.py
> D subversion/tests/cmdline/svntest/parsers.py
> U subversion/svn/cl.h
> U subversion/svn/mergeinfo-cmd.c
> U subversion/svn/util.c
> U subversion/svn/main.c
> G .
> $ svn st
> M .
> M subversion/include/svn_client.h
> M subversion/libsvn_client/util.c
> M subversion/libsvn_client/mergeinfo.c
> M subversion/tests/cmdline/mergeinfo_tests.py
> D subversion/tests/cmdline/svntest/parsers.py
> M subversion/tests/cmdline/svntest/actions.py
> M subversion/svn/cl.h
> M subversion/svn/mergeinfo-cmd.c
> M subversion/svn/util.c
> M subversion/svn/main.c
> M CHANGES
> $ svn diff --depth empty
>
>
> Property changes on: .
> ___________________________________________________________________
> Modified: svn:mergeinfo
> Reverse-merged
> /trunk:r29085-29089,29091,29094-29107,29111,29114,29117,29126-29127,29129-29133,29135-29150,29153-29164,29166,29174,29176-29186,29188-29189,29193-29194,29198-29206,29208-29251,29254-29256,29261,29267-29273,29277,29280-29281,29284,29287-29303,29305-29307,29309-29325,29327-29343,29345-29348,29358-29379,29381-29392,29397,29399,29401,29409,29412,29414-29415,29417-29423,29425-29426,29429,29433-29434,29436-29447,29449-29466,29468-29478,29482,29484,29486-29487,29489,29491,29493,29496,29498,29508,29527-29528,29531,29533,29539-29540,29542,29544,29546,29551,29553,29556,29559,29565,29567-29569,29571-29578,29581,29583,29591,29594,29600,29603,29607,29611,29613-29614,29619,29623,29625-29626,29630-29631,29634,29642,29648,29650,29656,29659-29660,29663-29664,29671-29672,29677-29680,29692,29738-29739,29742-29744,29746,29751,29763,29769-29770,29784,29787,29797,29801,29821,29824,29828,29835,29855,29858-29859,29868-29869,29876,29878,29883-29884,29895,29898,29900,29914,29920,29925,29930,29939-2994
>
> 0,29942,29950,29958,29962,29968,29980,29986,29994-29997,30004,30009,30020,30030,30050,30054,30061-30062,30067,30070,30074,30086,30098,30117,30124,30130,30137,30145,30151,30159,30162,30185
> Reverse-merged /branches/svn-mergeinfo-enhancements:r30122
> Merged /trunk:r30215,30221,30225-30226,30230,30243,30286
> Merged /branches/1.5.x-r30215:r30223-30348
> Merged /branches/svn-mergeinfo-enhancements:r30045-30121,30123-30214
> $
>
> Oh boy. That's all so very not good. My guess? The svn:mergeinfo
> property actually stored on the backport branch first blew away the one
> stored on in the 1.5.x branch, and then was modified to include the current
> merge operation.
>
>
>
> --
> C. Michael Pilato <cmpilato_at_collab.net>
> CollabNet <> www.collab.net <> Distributed Development On Demand
>
>
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-09 18:12:05 CEST