+7  A: 

Merging only revisions 4,7, and 11-15 with svnmerge:

svnmerge.py merge -r4,7,11-15

And with regular svn:

svn merge -c4,7 -r10:15 http://.../branches/TRY-XX-Foo
orip
Running from command line still produced the same result as TortoiseSVN, causing conflict and pulling in unwanted changes.
eed3si9n
Sorry, I guess I don't understand your scenario. Cherry-picking revisions to merge works just like I show (and evidently, like you tried), I'm not sure what's the problem.
orip
Orips answer should be right and give you the result you want. Maybe you didn't leave revision 5 out in your command-line statement. Then obviously you would pull the change into the merge.
boutta
@boutta, have you tried reproducing the result?
eed3si9n
@eed3si9n: I had a already formed branch, added a (unwanted) change to a file, then added a wanted change to the same file and did an svn merge -c$revision URL into the trunk of the project. Here $revision was the revision number of the wanted change. After the merge I got only the wanted change.
boutta
I did everything on the command line only with a -c option because for the test I only added 2 further revisions as in your example.
boutta
A: 

If you don't want the unwanted change, do not merge revision 4:5, but just revision 5. It means you merge the change committed in revision 5.

Wimmel
4:5 means take the difference between 4 and 5 and apply it to the working directory. I've tried -c5 command line option, but the result is the same.
eed3si9n
+3  A: 

The problem is that both svn

A
<<<<<<< .working
=======
B (unwanted change)
C (important bug fix)
>>>>>>> .merge-right.r341

and TortoiseSVN is treating the situation as 2-way merge. I've heard of the term 3-way merge, so I gave Beyond Compare a shot. With quick set up with TortoiseSVN, Edit Conflict now bring up the following screen. This is not perfect, since it's still requiring human intervention, but at least I can tell which changes are coming from where.

See screenshot.

eed3si9n
Using the subclipse merge view you can enable the original view which will get you the three way merge view. Check the top right corner.
Eduard Wirch
Please put this information in your question, because this is an answer.
furtelwart
@guerda, I am posting the use of 3-way merge tool as an answer.
eed3si9n
+1  A: 

Another thing you could do would be to manually undo the bad commit on the branch, which would then allow you to merge the branch back into the trunk as you would normally.

TortoiseSVN

Using TortoiseSVN you open up the log view on a file, select the offending version, and choose "Revert changes from this revision" from the right click menu. Commit the changes it makes on your working copy and then you can merge the branch back in easily.

Command Line

To do this with the command line client you perform a reverse merge, (This is taken from the Pragmatic Source Control using Subversion book) where you merge the changes between the offending version and the previous version into the working copy of the file. Then as above you'd commit the changes and can then branch normally. In your example you would do something like:

svn merge -r 4:3 test.txt
Daemin
Undoing is not an option since the unwanted changes in branches/TRY-XX-Foo/ represent half-baked features, not ready to get merged into trunk.
eed3si9n
+2  A: 

I believe you are including the revisions you want correctly, but the merge algorithm is failing to find the place to insert the wanted change and so including the line above it also. Here are the same steps but with a different set of changes, and I believe it works as you expected originally:

$ svnadmin create repo
$ svn mkdir -m '' file://`pwd`/repo/trunk

Committed revision 1.
$ svn mkdir -m '' file://`pwd`/repo/branches

Committed revision 2.
$ svn co file://`pwd`/repo/trunk co.trunk
Checked out revision 2.
$ cat > co.trunk/test.txt << EOF
> A
> B
> C
> EOF
$ svn add co.trunk/test.txt
A         co.trunk/test.txt
$ svn commit -m '' co.trunk
Adding         co.trunk/test.txt
Transmitting file data .
Committed revision 3.
$ svn copy -m '' file://`pwd`/repo/trunk file://`pwd`/repo/branches/testbr

Committed revision 4.
$ svn co file://`pwd`/repo/branches/testbr co.testbr
A    co.testbr/test.txt
Checked out revision 4.
$ cat > co.testbr/test.txt << EOF
> A
> A1 unwanted
> B
> C
> EOF
$ svn commit -m '' co.testbr
Sending        co.testbr/test.txt
Transmitting file data .
Committed revision 5.
$ cat > co.testbr/test.txt << EOF
> A
> A1 unwanted
> B
> B1 wanted
> C
> EOF
$ svn commit -m '' co.testbr
Sending        co.testbr/test.txt
Transmitting file data .
Committed revision 6.
$ svn merge -r 5:6 file://`pwd`/repo/branches/testbr co.trunk
--- Merging r6 into 'co.trunk':
U    co.trunk/test.txt
$ cat co.trunk/test.txt
A
B
B1 wanted
C
Randy Proctor
A: 

As pointed out by other users (I won't take credit for noticing it because I didn't), it may be the trivial nature of this merge (i.e. lack of context around the change) that is confusing the tools.

I do a lot of merging and, as you discovered, the merge tool provided by Tortoise is awful. A three-way merge tool is an absolute must if you do this very often. Beyond Compare is my personal favorite, but there are other that are free (KDiff3) and ones that are not (Araxis).

You will notice that Beyond Compare did the right thing in the end, even if it makes you manually verify it correctness!

Jeremy
Lack of context is certainly is the key, thus "multiple consecutive changes".
eed3si9n
+2  A: 

Well to clarify a thing about merge is that it actually has 2 steps.

  1. Merge
  2. Commit

So that means that after your merge is done, you can do a manual diff against head and the other branch to make sure that the merge was correct. And if something was wrong with it, like in your case, you can manually fix it before the commit.

/Johan

Johan
+1  A: 

In TortoiseSVN, you must only specify the revisions you want to merge. Unlike the command line client where you have to specify e.g. -r4:5 to merge the changes between r4 and r5, you only have to specify '5' as the revision number to merge in the TortoiseSVN merge dialog. If you're not sure, always use log dialog from the merge dialog and select the revisions you want to merge in that log dialog (then click OK and the selected revisions will automatically get set in the merge dialog).

As for resolving your conflict in TortoiseMerge: According to the screenshot in your question, TortoiseMerge shows you two conflicted lines (the ones shown as '????' in the bottom view). What you want is to include the change 'C' but not 'B'?

  • left click on the first '???' line to select it, then right-click, choose 'use block from "mine"' from the context menu
  • left click on the second '???' line to select it, then right-click, choose 'use block from "theirs"' from the context menu
  • Click the save button (or File->Save)
  • Optionally click on the "Mark as resolved" button
Stefan
First of all, thank you for the great tool. I've been keeping up with TortoiseSVN updates, so I know that you need to specify 5, instead of 4-5.
eed3si9n
In this simplified example, it's easy to tell the "unwanted" part, but Subversion shouldn't have brought any changes from r4. In reality, it's hard to tell which part came from the unwanted revision that I did not specify. I frequently merge branch changes written by others.
eed3si9n
Well, Subversion didn't actually merge r4. The problem you're seeing here is that Subversion encountered a conflict, and to indicate that conflict it not only marks the conflicting lines but also 1 line around the conflicting lines (i.e., context lines). If you had a line D, that line would also
Stefan
be included in the conflict. So actually, it still is easy to spot those 'unwanted' lines.Also, it you had more context lines (this example has almost no context lines, almost all lines are include din the merge), Subversion would be much more accurate with the merge.
Stefan