Skip to content

Bug fixes and patching

Wayne Witzel III edited this page Sep 29, 2015 · 3 revisions

juju and Bug Fixes

When fixing a bug that affects more than a single branch the prefered process for fixing the bug is to perform the work against the oldest branch (back porting) and then apply that patch up to each release branch and finally in to master.

For example, if a bug is reported to impact R1, R2, and master. You would perform the work against a branch of R1. When your fix is ready you would use the pull request and review process. See: [reviewboard] Then you would apply your fix to R2, and finally once your fix was merged in to R2, you would apply the fix to current master.

How To Backport

Once your inital fix has been merged by the bot (the back port) the process of applying your patch can be done with a few git commands. In this example I will show you applying the fix in to master.

You will want to locate the SHA for the merge commit that was generated by jujubot. This will be viewable on github or in your git log output. Copy the SHA since we will use it to cherry-pick the fix in to master.

git checkout master
git checkout -b <fixed-branch-name>
git cherry-pick -m 1 <merge-commit-sha>

You may have some minor merge conflicts with the cherry-pick that need to be fixed, this is rare when back porting the change first, but occasionally it does happen.

git push your-remote <fixed-branch-name>

Now your new branch is ready to follow the same pull request and review process as the original fix. Be sure to note that this is a patch from a backport and link to the previous review.

Clone this wiki locally