On Wed, Apr 9, 2008 at 12:42 PM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
>
> > '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.
> >
>
> I'm testing a patch for this now. (I suspect that I actually wrote the
> bogus code myself, too.)
>
>
> > 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
> > $
> >
>
> Not sure what's up here.
>
>
>
> > (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?)
> >
>
> This was fixed in r30449, and proposed for backport to 1.5.x.
>
>
>
>
> > 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-29
> >
> 94
>
> >
> >
> 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.
> >
>
> I haven't even begun to look into this. I should note, though, that my
> Subversion client was probably of 1.5.x-rc1 pedigree, so not quite the
> newest code.
I'm looking into this now.
Paul
---------------------------------------------------------------------
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 19:44:22 CEST