Git/Workflow/Topic: Difference between revisions
Line 269: | Line 269: | ||
| | | | ||
:<code>$ git push</code> | :<code>$ git push</code> | ||
| | |||
.......... | |||
. \ | |||
...o----o----o---o *'''master''', origin/master | |||
\ / | |||
o----o | |||
|align="center"| | |||
[[#Push_non-fast-forward|Push non-fast-forward?]] | |||
|} | |} | ||
Revision as of 15:00, 3 May 2010
Introduction
This workflow is based on the branchy development workflow documented by git help workflows
.
Motivation
The primary goal of this workflow is to make release preparation and maintenance easier. We set the following requirements, of which some are themselves worthwhile goals:
- Amortize the effort of release preparation throughout development
- Support granular selection of features for release
- Allow immature features to be published without delaying release
- Keep unrelated development paths (topics) independent of one another
Design
The design of this workflow is based on the observation that meeting the highlighted goal makes the other goals easy. It is based on branchy development in which each branch has a well-defined purpose.
We define two branch types:
- Topic Branch
- Commits represent changes (real work)
- Distinguished by feature (one topic per feature or fix)
- Named locally by each developer (describe purpose of work)
- Heads not published (no named branch on server)
- Integration Branch
- Commits represent merges (merge topics together)
- Distinguished by stability (release maintenance, release preparation, development edge)
- Named everywhere
- Heads published on server
Notation
This document uses ascii-art to depict commit history:
Branch name |
master |
Current branch |
*master |
Commit with parent in same branch |
----o |
Commit with parent in another branch |
\ o |
Commit with two parents (merge) |
----o / |
Commit with arbitrary ancestry |
...o |
Topic branches generally consist of a linear sequence of commits forked off an integration branch:
...o master \ o----o----o----o topic
Integration branches generally consist of a sequence of merge commits:
...o ...o \ \ ...o----o----o----o----o master / / ...o ...o
Integration Branches
We use an integration branch for each stage of development:
- maint: Release maintenance (high stability). Only bug fixes should be published here. Only the release manager can push here.
- master: Release preparation (medium stability). Only mature features and bug fixes should be published here.
- next: Development (low stability). New features should be published here.
Development
We cover below the steps to take during each phase of development.
New Topic
Create a new topic branch for each separate feature or bug fix. Always start the topic from a stable integration branch, usually master. If the topic fixes a bug in the current release, use maint. Never start a topic from next! In the following table we review the steps and commands to create, develop, and publish a topic branch based on master.
Actions | Results | Troubleshooting |
---|---|---|
Update master to base work on the most recently integrated features. | ||
|
...o *master | |
|
...o----o *master | |
Create the local topic branch. Use a meaningful name for "topic". Name topics like you might name functions: concise but precise. A reader should have a general idea of the change to be made given just the branch name. | ||
|
...o----o master ^ *topic | |
This is where the real work happens. Edit, stage, and commit files repeatedly as needed for your work. During this step, avoid the "urge to merge" from an integration branch. Your changes are safely committed; you can merge your work with others when you are finished. | ||
|
...o----o master \ o *topic | |
|
...o----o master \ o----o *topic | |
When the topic is ready for publication it must be merged into next. Do not merge from next! It should be the current local branch when you merge. Switch to next and update it.
Use " | ||
|
...o----o master . \ . o----o topic . ........o *next | |
Merge the topic and test it. | ||
|
...o----o master . \ . o----o topic . \ ........o----o *next | |
Finally, publish the change. | ||
|
...o----o master . \ . o----o topic . \ ........o----o *next (and origin/next) |
Mature Topic
When a topic is ready for inclusion in the next release, we merge it into master.
Actions | Results | Troubleshooting |
---|---|---|
Update master to get the latest work by others. We will merge the topic into it. | ||
|
...o----o *master . \ . o----o topic . \ ........o----o next | |
|
.......... . \ ...o----o----o *master . \ . o----o topic . \ ........o----o next | |
Merge the topic and test it. | ||
|
.......... . \ ...o----o----o---o *master . \ / . o----o topic . \ ........o----o next | |
Delete the local branch. | ||
|
.......... . \ ...o----o----o---o *master . \ / . o----o . \ ........o----o next | |
Finally, publish the change. | ||
|
.......... . \ ...o----o----o---o *master, origin/master \ / o----o |
Note that master sees only the topics that have been merged into it. It cannot reach any of the merges into next:
.......... . \ ...o----o----o---o *master \ / o----o (topic)
Old Topic
Sometimes we need to continue work on an old topic that has already been merged to an integration branch and for which we no longer have a local topic branch. To revive an old topic, we create a local branch based on the last commit from the topic (this is not one of the merges into an integration branch).
First we need to identify the commit by its hash.
It is an ancestor of the integration branch into which it was once merged, say next.
Run git log
with the --first-parent
option to view the integration history:
$ git log --first-parent next commit 9057863... Merge: 2948732 a348901 ... Merge branch topicA commit 2948732... Merge: 1094687 b235725 ... Merge branch topicB commit 1094687... Merge: 8267263 c715789 ... Merge branch topicC
Locate the merge commit for the topic of interest, say topicB.
It's second parent is the commit from which we will restart work (b235725
in this example).
Actions | Results | Troubleshooting |
---|---|---|
Create a local topic branch starting from the commit identified above. | ||
|
.......... . \ ...o----o----o---o master . \ / . o----o *topic (b235725) . \ ........o----o----o next / / ...o ...o (c715789) (a348901) | |
Continue development on the topic. | ||
|
.......... . \ ...o----o----o---o master . \ / . o----o----o *topic . \ ........o----o----o next / / ...o ...o | |
|
.......... . \ ...o----o----o---o master . \ / . o----o----o----o *topic . \ ........o----o----o next / / ...o ...o | |
When the new portion of the topic is ready, merge it into next and test. | ||
|
.......... . \ ...o----o----o---o master . \ / . o----o----o----o topic . \ \ ........o----o----o----o *next / / ...o ...o | |
Publish next. | ||
| ||
Optionally repeat the above, publishing to next, to continue development based on feedback from initial publication. Finally, when the new changes are mature, merge to master and publish. | ||
|
.......... . \ ...o----o----o---o---------o *master . \ / / . o----o----o----o topic . \ \ ........o----o----o----o next / / ...o ...o | |
|
.......... . \ ...o----o----o---o---------o *master . \ / / . o----o----o----o . \ \ ........o----o----o----o next / / ...o ...o |
Again, note that master sees only the topics that have been merged into it. It cannot reach any of the merges into next:
.......... . \ ...o----o----o---o---------o *master \ / / o----o----o----o (topic)
Discussion
History Shape
The history graphs produced by this workflow may look complex compared to the fully linear history produced by a rebase workflow (used by CVS and Subversion):
.......... . \ ...o----o----o---o---------o master . \ / / . o----o----o----o topic . \ \ ........o----o----o----o next / / ...o ...o
However, consider the shape of history along each branch.
We can view it using Git's --first-parent
option.
It traverses history by following only the first parent of each merge.
The first parent is the commit that was currently checked out when the git merge
command was invoked to create the merge commit.
By following only the first parent, we see commits that logically belong to a specific branch.
Command | View |
---|---|
|
... \ o----o----o----o topic |
|
\ ...o----o----o---o---------o master / / |
|
\ \ ...o----o----o----o next / / |
Each branch by itself looks linear and has only commits with a specific purpose. The history behind each commit is unique to that purpose. Topic branches are independent, containing only commits for their specific feature or fix. Integration branches consist of merge commits that integrate topics together.
Note that achieving the nice separation of branches requires understanding of the above development procedure and strict adherence to it.
Troubleshooting
Here we document problems one might encounter while following the workflow instructions above. This is not a general Git troubleshooting page.
Trouble Pushing
Push non-fast-forward
When trying to publish new merge commits on an integration branch, perhaps next, the final push may fail:
$ git push To ... ! [rejected] next -> next (non-fast-forward) error: failed to push some refs to '...' To prevent you from losing history, non-fast-forward updates were rejected Merge the remote changes before pushing again. See the 'Note about fast-forwards' section of 'git push --help' for details.
This means that the server's next refers to a commit that is not reachable from the next you are trying to push:
...o----o master . \ . o----o topic . \ . .----o *next . . ....o----o origin/next / ...o (other-topic)
This is the Git equivalent to when cvs commit
complains that your file is not up-to-date, but now it applies to the whole project and not just one file.
Git is telling you that it cannot update next on the server to point at your merge commit because that would throw away someone else's work (such as other-topic).
There are a few possible causes, all of which mean you have not yet integrated your work with the latest from upstream:
- You forgot to run
git pull
beforegit merge
so you didn't have everything from upstream - Someone else managed to merge and push something into next since you last ran
git pull
Some Git documentation may tell you to just git pull
again to merge upstream work into yours.
That approach is not compatible with the goals of this workflow.
We want to maintain the "linear" nature of the merge sequence so that --first-parent
traversal of the integration branches gives an overview of the development.
The solution is to throw away your previous merge and try again, but this time start from the latest upstream work:
Actions | Results |
---|---|
|
...o----o master . \ . o----o topic . ...o----o *next, origin/next / ...o (other-topic) |
|
...o----o master . \ . o----o topic . \ ...o----o----o *next / ...o (other-topic) |
Now your next can reach the upstream work as well as yours. Publish it. | |
|
...o----o master . \ . o----o topic . \ ...o----o----o *next, origin/next / ...o (other-topic) |
See git rerere to help avoid resolving the same conflicts on each merge attempt.