[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Tweaking 1.5.x's Mergeinfo (Was: svn commit: r30390 - branches/1.5.x)

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Wed, 09 Apr 2008 12:42:43 -0400

> '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.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2008-04-09 18:42:55 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.