Difference between revisions of "VTK/Git"
|Line 278:||Line 278:|
If the following error
If the following error :
Patch format detection failed.
Patch format detection failed.
Revision as of 18:33, 2 July 2010
VTK version tracking and development is hosted by Git.
One may clone the repository using git clone through the native
$ git clone git://vtk.org/VTK.git VTK
or through the (less efficient)
$ git clone http://vtk.org/VTK.git VTK
All further commands work inside the local copy of the repository created by the clone:
$ cd VTK
For VTKData the URLs are
At the time of this writing the repository has the following branches:
- master: Development (default)
- hooks: Local commit hooks (place in .git/hooks)
Release branches converted from CVS have been artificially merged into master. Actual releases have tags named by the release version number.
We provide here a brief introduction to VTK development with Git. See the Resources below for further information such as Git tutorials.
We require all commits in VTK to record valid author/committer name and email information. Use git config to introduce yourself to Git:
$ git config --global user.name "Your Name" $ git config --global user.email "email@example.com"
Note that "Your Name" is your real name (e.g. "John Doe", not "jdoe"). While you're at it, optionally enable color output from Git commands:
$ git config --global color.ui auto
--global option stores the configuration settings in
~/.gitconfig in your home directory so that they apply to all repositories.
This step is optional but recommended.
The git commit command creates local commits.
A separate git push step is needed to publish commits to
vtk.org server enforces some rules on the commits it accepts and will reject non-conforming commits. See below.
In order to push rejected commits, one must edit history locally to repair them before publishing.
Since it is possible to create many commits locally and push them all at once, we provide local Git hooks to help developers keep their individual commits clean. Git provides no way to enable such hooks by default, giving developers maximum control over their local repositories. We recommend enabling our hooks manually in all clones.
Git looks for hooks in the
.git/hooks directory within the work tree of a local repository.
Create a new local repository in this directory to manage the hooks:
$ cd .git/hooks $ git init
vtk.org/VTK.git repository provides a
It will have already been fetched into your local clone.
Pull it from there:
$ git pull .. remotes/origin/hooks $ cd ../..
The hooks will now run in the outer repository to enforce some rules on commits. To update the hooks in the future, run
$ git fetch origin $ cd .git/hooks $ git pull .. remotes/origin/hooks
The above sequence maintains the following local hooks in your repository. See Git help on githooks for more details.
This runs during
git commit. It checks identity and content of changes:
user.emailare set to something reasonable
- Git's standard whitespace checks (see help on
git diff --check)
- The staged changes do not introduce any leading TABs in source files (we indent with spaces)
- File modes look reasonable (no executable .cxx files, scripts with shebang lines are executable)
- File size is not too large (don't commit big data files; prints limit and instructions on rejection)
One of Git's standard whitespace checks is to reject trailing whitespace on lines that were added or modified. Many people consider extra space characters at the end of a line to be an unprofessional style (including Git's own developers), but some don't care. Text editors typically have a mode to highlight trailing whitespace:
(custom-set-variables '(show-trailing-whitespace t))
:highlight ExtraWhitespace ctermbg=red guibg=red :match ExtraWhitespace /\s\+$/
To toggle viewing of white space characters, with a source file document active, choose the menu item: Edit > Advanced > View White Space (2-stroke keyboard shortcut: Ctrl+R, Ctrl+W)
To eliminate trailing white space, choose the menu item: Edit > Trim Trailing Space To toggle viewing of white space characters, choose from the menu items: View > Show Symbol > (multiple items, choose one...)
If you really don't want to keep your code clean of trailing whitespace, you can disable this part of Git's checks locally:
$ git config core.whitespace "-blank-at-eol"
This runs during
git commit. It checks the commit message format:
- The first line must be between 8 and 78 characters long. If you were writing an email to describe the change, this would be the Subject line.
(Do not bother using 'ENH:' or 'BUG:' prefixes; they take space and are less valuable than a good summary on this line).
- The first line must not have leading or trailing whitespace.
- The second line must be blank, if present.
- The third line and below may be free-form. Try to keep paragraph text formatted in 72 columns (this is not enforced).
GUI and text-based tools that help view history typically use the first line (Subject line) from the commit message to give a one-line summary of each commit. This allows a medium-level view of history, but works well only if developers write good Subject lines for their commits.
Examples of improper commit messages:
This is too short and not informative at all.
I did a really complicated change and I am trying to describe the entire thing with a big message entered on the command line.
Many CVS users develop the habit of using the "-m" commit option to specify the whole message on the command line.
This is probably because in CVS it is hard to abort a commit if it already brought up the message editor.
In Git this is trivial. Just leave the message blank and the whole commit will be aborted.
Furthermore, since commits are not published automatically it is easy to allow the commit to complete and then fix it with
git commit --amend.
We've chosen to approximate our previous CVS-based development workflow after the initial move to Git, at least while things get settled. See here for more detailed instructions. The basic rule is to rebase your work on origin/master before pushing:
git fetch origin git rebase origin/master
git pull --rebase
The server will refuse your push if it contains any merges. Later we will move to branchy development as described in the gitworkflows man page.
Authorized developers may publish work directly to
vtk.org/VTK.git using Git's SSH protocol.
Note that we may not grant all contributors push access to the
The distributed nature of Git allows contributors to retain authorship credit even if they do not publish changes directly.
If you are an external contributor without push access, you can submit a patch to the vtk-developers mailing list, see #Submitting_a_patch.
All publishers share the
firstname.lastname@example.org account but each uses a unique ssh key for authentication.
If you do not have a public/private ssh key pair, generate one:
$ ssh-keygen -C 'email@example.com' Generating public/private rsa key pair. Enter file in which to save the key ($HOME/.ssh/id_rsa): Enter passphrase (empty for no passphrase): (use-a-passphrase!!) Enter same passphrase again: (use-same-passphrase!!) Your identification has been saved in $HOME/.ssh/id_rsa. Your public key has been saved in $HOME/.ssh/id_rsa.pub.
To request access, fill out the Kitware Password form.
Include your ssh public key,
id_rsa.pub, and a reference to someone our administrators may contact to verify your privileges.
Generating an ssh key on Windows
If you are familiar with generating an ssh key on Linux or Mac, you can follow the same procedure on Windows in a "Git Bash" prompt. There is an ssh-keygen program installed with msysGit to help you set up an ssh identity on a Windows machine. By default it puts the ".ssh" directory in the HOME directory, which is typically "/c/Users/Username" on Vista and Windows 7; on XP, it's "/c/Documents and Settings/Username".
Alternatively, you can also set up a "normal" Windows command prompt shell such that it will work with msysGit, without ever invoking the Git Bash prompt if you like. If you install msysGit and accept all its default options, "git" will not be in the PATH. However, if you add "C:\Program Files (x86)\Git\cmd" to your PATH, then only the two commands git and gitk are available to use via *.cmd script wrappers installed by msysGit. Or, if you add "C:\Program Files (x86)\Git\bin" to your PATH, then all of the command line tools that git installs are available.
The full PuTTY suite of tools includes an application called PuTTYgen. If you already have a private key created with PuTTYgen, you may export it to an OpenSSH identity file. Open the key using PuTTYgen and choose "Conversions > Export OpenSSH key" from the menu bar. That will allow you to save an "id_rsa" file for use in the ".ssh" directory. You can also copy and paste the public key portion of the key from the PuTTYgen text field to save into an "id_rsa.pub" file if you like. Or email it to whoever needs the public side of your key pair.
If you routinely set up your own command prompt environment on Windows, using msysGit from that envrionment is a cinch: just add the full path to either Git\cmd or Git\bin to your PATH. (Or, write your own git.cmd wrapper that is in your PATH that simply calls the git.cmd installed with msysGit.) And make sure you have a HOME environment variable that points to the place where the .ssh directory is.
In order to publish new commits in the
vtk.org repository, developers must configure a push URL for the
Use git config to specify an ssh-protocol URL:
$ git config remote.origin.pushurl firstname.lastname@example.org:VTK.git
(Note that 'pushurl' requires Git >= 1.6.4. Use just 'url' for Git < 1.6.4.)
Failing to do so with result in the following error message: "fatal: The remote end hung up unexpectedly".
Once your push URL is configured and your key is installed for
email@example.com then you can try pushing changes.
For VTKData, the push URL is
The vtk.org repository has an
It runs when someone tries to update a ref on the server by pushing.
The hook checks all commits included in the push:
- Commit author and committer must have valid email address domains (DNS lookup succeeds).
- Commit message does not start with "
WIP:". (Use the prefix locally for work-in-progress that must be rewritten before publishing.)
- Changes to paths updated by robots (such as Utilities/kwsys) are not allowed.
- No "large" blobs may be pushed. The limit is 1MB at the time of this writing.
- No CRLF newlines may be added in the repository (see core.autocrlf in
git help config).
- Submodules (if any) must be pushed before the references to them are pushed.
Submitting a patch
If you are an external contributor without push access, you can submit a patch to the vtk-developers mailing list.
- subscribe to the vtk-developers mailing list: http://www.vtk.org/mailman/listinfo/vtk-developers
- prepare your patch:
# 1. pull from the central repository with rebase, not merge $ cd VTK $ git pull --rebase
# 2. create the patch $ git format-patch
3. send the patch to the vtk-developers list
A contributor with push access will be able to apply the patch and push it to the central repository. See the following section to apply a patch #Applying_a_patch .
Applying a patch
When you receive a patch by email, you can apply it locally with:
$ cd VTK $ git am -3 -i -s -u patch.txt
If the following error happens:
Patch format detection failed.
It means the patch was not generated with git format-patch but with git show
To apply a patch generated with git show, use
$ cd VTK $ git apply --whitespace=fix patch.txt
Now you have applied the patch, you can review it and test it.
If you are a contributor with push access, you can decide to reject or accept the patch and maybe apply additional commits. At this point, you are ready to publish the patch to the central repository #Publishing .
fatal: The remote end hung up unexpectedly
- If git push fails with "fatal: The remote end hung up unexpectedly", you probably forgot to set the push url with "git config" see #Pushing.
Restoring files locally
Q: "I cloned the VTK repository. Now I "rm -rf Hybrid". How do I get it back?"
A: git checkout Hybrid
Q: "I modified a file locally. I want to revert it."
A: git checkout myfile.cxx
Additional information about Git may be obtained at these sites: