ParaView/Git/Develop: Difference between revisions
No edit summary |
|||
Line 96: | Line 96: | ||
:<code>$ git pullall</code> | :<code>$ git pullall</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
<br/> | <br/> | ||
[http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.pullall</code>] | [http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.pullall</code>] | ||
<br/> | <br/> | ||
([http:// | ([http://schacon.github.com/git/git-pull.html <code>pull</code>] and | ||
[http:// | [http://schacon.github.com/git/git-submodule.html <code>submodule</code>] <code>update</code>) | ||
|} | |} | ||
Line 120: | Line 120: | ||
:''(If you are fixing a bug in the latest release then substitute'' <code>origin/release</code> ''for'' <code>origin/master</code>''.<br/>Then run'' <code>git submodule update</code>''.)'' | :''(If you are fixing a bug in the latest release then substitute'' <code>origin/release</code> ''for'' <code>origin/master</code>''.<br/>Then run'' <code>git submodule update</code>''.)'' | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-fetch.html <code>git help fetch</code>] | ||
<br/> | <br/> | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
<br/> | <br/> | ||
[http:// | [http://schacon.github.com/git/git-submodule.html <code>git help submodule</code>] | ||
<br/> | <br/> | ||
[http://progit.org/book/ch3-2.html Pro Git: Basic Branching] | [http://progit.org/book/ch3-2.html Pro Git: Basic Branching] | ||
Line 137: | Line 137: | ||
:<code>$ git commit</code> | :<code>$ git commit</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-add.html <code>git help add</code>] | ||
<br/> | <br/> | ||
[http:// | [http://schacon.github.com/git/git-commit.html <code>git help commit</code>] | ||
<br/> | <br/> | ||
[http://progit.org/book/ch2-2.html Pro Git: Recording Changes] | [http://progit.org/book/ch2-2.html Pro Git: Recording Changes] | ||
Line 157: | Line 157: | ||
:<code>$ git checkout ''my-topic''</code> | :<code>$ git checkout ''my-topic''</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
|- | |- | ||
| | | | ||
Line 167: | Line 167: | ||
[http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.prepush</code>] | [http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.prepush</code>] | ||
<br/> | <br/> | ||
([http:// | ([http://schacon.github.com/git/git-log.html <code>log</code>] <code>origin/master..</code>) | ||
|- | |- | ||
| | | | ||
Line 178: | Line 178: | ||
[http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.stage-push</code>] | [http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.stage-push</code>] | ||
<br/> | <br/> | ||
([http:// | ([http://schacon.github.com/git/git-push.html <code>push</code>] <code>stage HEAD</code>) | ||
|} | |} | ||
Line 195: | Line 195: | ||
:<code>$ git checkout ''my-topic''</code> | :<code>$ git checkout ''my-topic''</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
|- | |- | ||
| | | | ||
Line 229: | Line 229: | ||
:<code>$ git submodule update</code> | :<code>$ git submodule update</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
<br/> | <br/> | ||
[http:// | [http://schacon.github.com/git/git-submodule.html <code>git help submodule</code>] | ||
|- | |- | ||
| | | | ||
Line 242: | Line 242: | ||
:<code>$ git commit</code> | :<code>$ git commit</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-add.html <code>git help add</code>] | ||
<br/> | <br/> | ||
[http:// | [http://schacon.github.com/git/git-commit.html <code>git help commit</code>] | ||
<br/> | <br/> | ||
[http://progit.org/book/ch2-2.html Pro Git: Recording Changes] | [http://progit.org/book/ch2-2.html Pro Git: Recording Changes] | ||
Line 273: | Line 273: | ||
:<code>$ git pullall</code> | :<code>$ git pullall</code> | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-checkout.html <code>git help checkout</code>] | ||
<br/> | <br/> | ||
[http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.pullall</code>] | [http://paraview.org/gitweb?p=ParaView.git;a=blob;f=Utilities/Scripts/SetupGitAliases.sh;hb=HEAD <code>alias.pullall</code>] | ||
<br/> | <br/> | ||
([http:// | ([http://schacon.github.com/git/git-pull.html <code>pull</code>] and | ||
[http:// | [http://schacon.github.com/git/git-submodule.html <code>submodule</code>] <code>update</code>) | ||
|- | |- | ||
| | | | ||
Line 287: | Line 287: | ||
:''(If you changed VTK [[ParaView/Git/Develop/PVVTK#Delete_a_Topic|delete the VTK topic]].)'' | :''(If you changed VTK [[ParaView/Git/Develop/PVVTK#Delete_a_Topic|delete the VTK topic]].)'' | ||
|align="center"| | |align="center"| | ||
[http:// | [http://schacon.github.com/git/git-branch.html <code>git help branch</code>] | ||
|- | |- | ||
| | | |
Revision as of 14:22, 7 November 2011
This page documents how to develop ParaView through Git.
See our table of contents for more information.
Git is an extremely powerful version control tool that supports many different "workflows" for indivudal development and collaboration. Here we document procedures used by the ParaView development community. In the interest of simplicity and brevity we do not provide an explanation of why we use this approach. Furthermore, this is not a Git tutorial. Please see our Git resource links for third-party documentation, such as the ProGit Book.
Setup
Before you begin, perform initial setup:
1. Register Git push access. | |
2. Follow the download instructions to create a local ParaView clone: | |
|
|
3. Run the developer setup script to prepare your ParaView work tree and create Git command aliases used below: | |
|
Workflow
ParaView development uses a branchy workflow based on topic branches. We manage changes to the VTK submodule using an extended VTK workflow. Our collaboration workflow consists of three main steps:
1. Local Development |
2. Testing and Review |
3. Integration by Maintainers |
Update
Update your local master branch: | |
|
|
Create a Topic
All new work must be committed on topic branches. Name topics like you might name functions: concise but precise. A reader should have a general idea of the feature or fix to be developed given just the branch name.
To start a new topic branch: | |
|
|
Edit files and create commits (repeat as needed): | |
|
When a topic is ready for testing and review by others, share it by pushing it to the topic stage. Be sure you have registered for Git push access.
Checkout the topic if it is not your current branch: | |
|
|
Check what commits will be pushed to the topic stage: | |
|
|
Push commits in your topic branch to the topic stage: | |
|
|
The topic is now published on the ParaView Topic Stage and may be (optionally) reviewed by others.
Merge a Topic for Testing
When your topic is ready, merge it to the ParaView next branch for testing.
Checkout the topic if it is not your current branch: | |
|
|
Ask the topic stage to automatically merge the topic. It will merge to next by default. | |
|
|
The topic is now integrated into ParaView's next branch and will be tested by dashboard builds.
Extend a Topic
If your topic runs cleanly after a night of dashboard builds, it is ready for the next step. Otherwise, extend the topic with additional commits to fix the problems.
Checkout the topic if it is not your current branch: | |
|
|
Edit files and create commits (repeat as needed): | |
|
|
Return to the earlier step to share the extended topic. |
Merge a Topic for Inclusion
Only core maintainers have access to merge a topic to master. They meet weekly to evaluate topics in next (and PVVTK pv-next) based on dashboard results and manual review. If your topic is accepted it will be merged to master for permanent inclusion after which you may delete it. Otherwise the maintainers will contact you with feedback. Respond by returning to the above step to address their concerns.
Delete a Topic
After a topic has been merged upstream, delete your local branch for the topic.
Checkout and update the master branch: | |
|
|
Delete the local topic branch: | |
|
|
The |