update gerrit list
[openafs-wiki.git] / devel / GitDevelopers.mdwn
index 6e67232..ab2075e 100644 (file)
@@ -245,8 +245,7 @@ Gerrit introduces the concept of "change IDs". This is a unique reference for a
 
 The hook should be downloaded from the [[OpenAFS]] gerrit server by running the following, in the top level of your git tree
 
-    mkdir -p ~/.git/hooks
-    scp -oStrictHostKeyChecking=no -p -P 29418 gerrit.openafs.org:hooks/commit-msg ~/.git/hooks/
+    scp -p -P 29418 gerrit.openafs.org:hooks/commit-msg .git/hooks/
 
 ## <a name="Uploading to gerrit">Uploading to gerrit</a> 
 
@@ -268,11 +267,18 @@ Secondly, each commit should have a meaningful revision log. The internals of gi
 
 Thirdly, each commit should have a valid changeID. Manually maintaining these is difficult and error prone, so we would strong advise that you install the changeID hook detailed earlier. This will automatically add a [[ChangeId]] line to your commit message if it doesn't already contain one.
 
-Fourthly, each commit must adhere to the OpenAFS whitespace policy whereby new commits will not be accepted if they have trailing spaces, spaces before tabs, or indentation without tabs.  Git can be configured to highlight the whitespace policy violation with the following global setting:
+Fourthly, each commit must adhere to the OpenAFS whitespace policy whereby new
+commits will not be accepted if they have trailing spaces, spaces before tabs,
+or indentation without tabs for new files and functions. The prevailing
+formating conventions for exsiting code should be followed.  Unrelated
+formatting changes to code is discouraged.
+
+Git can be configured to highlight the whitespace policy violation with the
+following global setting:
 
     git config --global core.whitespace trailing-space,space-before-tab,indent-with-non-tab
 
-and 
+and
 
     git rebase --whitespace=fix