Backporting

For general advice, see ​http://svnbook.red-bean.com/en/1.4/svn.branchmerge.copychanges.html

For GRASS, see Merge between branches (backporting).

Policies

Q: Should developers do it themselves? Or ask another dev to do it to keep some review sanity?

A: It depends: preferably a developer should be able to backport him/herself but

• do not break the release branch
• make your commits to the dev branch first
• make tests
• only fix bugs, do not introduce new features
• Only do refactoring in trunk
  • (large and complicated changes can introduce new hard to spot bugs, the idea of the stable branch is to asymptote to zero bugs)

If the above is unclear, it should be discussed on the developers list. If in doubt, ask! In this case, a path/diff is appreciated.

Hints

Three little scripts to make merging between branches easier:

• grass-addons/tools/svn64merge​
• grass-addons/tools/svn65merge​
• grass-addons/tools/svn7merge​

To use them, first make sure all branches are up to date (svn up) and clean (svn diff), then cd into the top of the source tree you want to pull in to. If you try to run it from the module's own subdir it will fail. Then run the script named for the branch you want to pull from, followed by the revision number of the change you wish to merge (no leading "r" or [square brackets] around the rev number please). Mind that you might want to run 'svn propdel svn:mergeinfo .' before doing an untargeted 'svn commit'.

Example:

 # make the original edit
 cd devbr6
 svn diff lib/gis/parser.c
 svn up lib/gis/parser.c
 vi lib/gis/parser.c
 svn diff lib/gis/parser.c
 # .. run make; test the change ..
 svn commit lib/gis/parser.c   # note the rev number of the commit

 # now backport it
 cd ../relbr64
 svn diff lib/gis/parser.c
 svn up lib/gis/parser.c
 svn65merge 54321
 svn diff lib/gis/parser.c
 # .. run make; test the change ..
 svn commit lib/gis/parser.c

Comparing branches

A diff tool like Kompare, Kdiff3, or Meld can help. These run the standard UNIX "diff -u" utility and show the results in an easy to navigate GUI.

You'll want to tell them to ignore the build files, svn paperwork files, compiled binaries, and "Last changed" timestamps on the help pages. This is typically done by going into the options and telling it to ignore a list of wildcard'ed path names, and regex filter for the document content.

For example, to compare devbr6 and relbr64 using kdiff3: (I used the QT version of Kdiff3, the other tools are similar)

  cd grass/svn/
  svn checkout https://svn.osgeo.org/grass/grass/branches/releasebranch_6_4 relbr64
  svn checkout https://svn.osgeo.org/grass/grass/branches/develbranch_6 devbr6

  kdiff3 relbr64/ devbr6/

• This will take a little while to scan the two directory trees
• Once loaded, go to Settings → Configure KDiff3...
• Go into the Directory tab
  • To "File-anti-pattern(s):" add ";*.tmp.html"
  • To "Dir-anti-pattern(s):" add ";OBJ.x86*;dist.x86*;bin.x86*"
    • (if not working with translations today you might add ";locale" to that too)
  • In "File Comparison Mode" select "Full analysis"
• Go to the Diff tab
  • In the "Line-matching preprocessor command:" line add:

    grep -v "Last changed: $Date: 2"
    

    • (this will ignore the $Date:$ svn keywords at the end of the help pages)
• Click "Ok" to go back to the main GUI window, and press Shift-F5 to rescan with the new filters.

You can now double click on a filename to review changes.