| 1 | <html><head><meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"><title>Chapter 7. Contributing code</title><link rel="stylesheet" href="../samba.css" type="text/css"><meta name="generator" content="DocBook XSL Stylesheets V1.74.0"><link rel="home" href="index.html" title="SAMBA Developers Guide"><link rel="up" href="pt02.html" title="Part II. Samba Basics"><link rel="prev" href="CodingSuggestions.html" title="Chapter 6. Coding Suggestions"><link rel="next" href="modules.html" title="Chapter 8. Modules"></head><body bgcolor="white" text="black" link="#0000FF" vlink="#840084" alink="#0000FF"><div class="navheader"><table width="100%" summary="Navigation header"><tr><th colspan="3" align="center">Chapter 7. Contributing code</th></tr><tr><td width="20%" align="left"><a accesskey="p" href="CodingSuggestions.html">Prev</a> </td><th width="60%" align="center">Part II. Samba Basics</th><td width="20%" align="right"> <a accesskey="n" href="modules.html">Next</a></td></tr></table><hr></div><div class="chapter" lang="en"><div class="titlepage"><div><div><h2 class="title"><a name="contributing"></a>Chapter 7. Contributing code</h2></div><div><div class="author"><h3 class="author"><span class="firstname">Jelmer</span> <span class="othername">R.</span> <span class="orgname">The Samba Team</span> <span class="surname">Vernooij</span></h3><div class="affiliation"><span class="orgname">The Samba Team<br></span><div class="address"><p><code class="email"><<a class="email" href="mailto:[email protected]">[email protected]</a>></code></p></div></div></div></div></div></div><p>Here are a few tips and notes that might be useful if you are
|
|---|
| 2 | interested in modifying samba source code and getting it into
|
|---|
| 3 | samba's main branch.</p><div class="variablelist"><dl><dt><span class="term">Retrieving the source</span></dt><dd><p>In order to contribute code to samba, make sure you have the
|
|---|
| 4 | latest source. Retrieving the samba source code from CVS is
|
|---|
| 5 | documented in the appendix of the Samba HOWTO Collection.
|
|---|
| 6 | </p></dd><dt><span class="term">Discuss large modifications with team members</span></dt><dd><p>Please discuss large modifications you are going to make
|
|---|
| 7 | with members of the samba team. Some parts of the samba code
|
|---|
| 8 | have one or more 'owners' - samba developers who wrote most
|
|---|
| 9 | of the code and maintain it.
|
|---|
| 10 | </p><p>This way you can avoid spending your time and effort on
|
|---|
| 11 | something that is not going to make it into the main samba branch
|
|---|
| 12 | because someone else was working on the same thing or because your
|
|---|
| 13 | implementation is not the correct one.
|
|---|
| 14 | </p></dd><dt><span class="term">Patch format</span></dt><dd><p>Patches to the samba tree should be in unified diff format,
|
|---|
| 15 | e.g. files generated by <strong class="userinput"><code>diff -u</code></strong>.
|
|---|
| 16 | </p><p>If you are modifying a copy of samba you retrieved from CVS,
|
|---|
| 17 | you can easily generate a diff file of these changes by running
|
|---|
| 18 | <strong class="userinput"><code>cvs diff -u</code></strong>.</p></dd><dt><span class="term">Points of attention when modifying samba source code</span></dt><dd><p>
|
|---|
| 19 | </p><div class="itemizedlist"><ul type="disc"><li><p>Don't simply copy code from other places and modify it until it
|
|---|
| 20 | works. Code needs to be clean and logical. Duplicate
|
|---|
| 21 | code is to be avoided.</p></li><li><p>Test your patch. It might take a while before one of us looks
|
|---|
|
|---|
