mozilla

Compare Revisions

Getting commit access to Mozilla source code

Change Revisions

Revision 161715:

Revision 161715 by Mitchell Baker on

Revision 161716:

Revision 161716 by Reed on

Title:
Getting commit access to Mozilla source code
Getting commit access to Mozilla source code
Slug:
Getting_commit_access_to_Mozilla_source_code
Getting_commit_access_to_Mozilla_source_code
Content:

Revision 161715
Revision 161716
n23      <li>File a bug (naturally). Product: mozilla.org&nbsp;; Comn23      <li>File a bug (naturally). Product: mozilla.org&nbsp;; Com
>ponent: CVS Account Request. Don't change the Default Assignee or>ponent: CVS Account Request. Don't change the Default Assignee or
> the Default QA Contact. It would be helpful if your summary says> the Default QA Contact. It would be helpful if your summary says
> something about creating a CVS account.> something about creating a CVS account ("CVS Account Request - J
 >ohn Doe &lt;jdoe@example.com&gt;").
n35      <li>Make sure to include your CVS SSH public Key as an attan35      <li>Make sure to include your CVS SSH public key as an atta
>chment to the bug. (Please mark it as text/plain when attaching i>chment to the bug. (Please mark it as text/plain when attaching i
>t!) Note that you will need to attach an SSH key for both webmonk>t!) Note that you will need to attach an SSH key for all types of
>ey and cvsuser access.> access.
nn47      <li>The Mozilla representative will then reassign the bug t
 >o IT to have your SSH public key added.
48      </li>
47      <li>The Mozilla representative will update the bug to note 49      <li>An IT representative will update the bug with account c
>the account has been created, and close the bug.>reation information and close the bug.
n54      To get commit access to Mozilla source code you basically nn56      To get commit access to Mozilla source code you basically n
>eed to demonstrate that you know what you're doing. You'll need t>eed to demonstrate that you know what you're doing. You'll need t
>o demonstrate competence with the code you're working with, other>o demonstrate competence with the code you're working with, other
> Mozilla code you might affect with your work, Mozilla check-in, > Mozilla code you might affect with your work, Mozilla check-in, 
>build and related processes (like watching the builds after you’v>build and related processes (like watching the builds after you’v
>e checked in code to make sure you haven't broken something) and >e checked in code to make sure you haven't broken something) and 
>basic good coding practices. It also helps if you can leap tall b>basic good coding practices. It also helps if you can leap tall b
>uildings in a single bound. You'll need to demonstrate this to a >uildings in a single bound. You'll need to demonstrate this to a 
>set of people who are willing to be associated with your check-in>set of people who are willing to be associated with your check-in
>s. We judge this by peer review, code review and our special X-ra>s. We judge this by peer review, code review, and our special X-r
>y vision.>ay vision.
n57      In all cases you need to demonstrate you know what's going n59      In all cases, you need to demonstrate you know what's going
>on, find a set of people who have adeuquate authority and will vo> on, find a set of people who have adequate authority and will vo
>uch for your competence, and complete and sign a CVS Contributor >uch for your competence, and complete and sign a CVS Contributor 
>Form. You'll start the process of getting CVS access by submittin>Form. You'll start the process of getting CVS access by submittin
>g good patches and having them reviewed by the module owner and o>g good patches and having them reviewed by the module owner and o
>ther appropriate people. When you think you have submitted patche>ther appropriate people. When you think you have submitted patche
>s that demonstrate the criteria above, you can start the process >s that demonstrate the criteria above, you can begin following th
>of obtaining commit access to the source tree.>e directions for obtaining commit access to the source tree.
n80      You need two vouchers. Your vouchers should be the owner orn82      You need two vouchers. Each voucher should be the owner or 
> a peer of a module or modules to which you've been submitting pa>a peer of a module or modules to which you've been submitting pat
>tches. Each voucher must already have CVS commit access and be co>ches. Each voucher must already have CVS commit access and be con
>nfident enough in you to be associated with your check-ins. Your >fident enough in you to be associated with your check-ins. Your v
>vouchers are responsible for your bustages in the unfortunate eve>ouchers are responsible for your bustages in the unfortunate even
>nt that you break the tree and leave. They are responsible for ma>t that you break the tree and leave. They are responsible for mak
>king sure you know and follow the rules in general, act promptly >ing sure you know and follow the rules in general, act promptly t
>to fix regressions, are aware of and respect tree closures, etc. >o fix regressions, are aware of and respect tree closures, etc. T
>The vouchers' responsibility extends for three months after you a>he vouchers' responsibility extends for three months after you ar
>re granted source code commit access. If you've lived in the tree>e granted source code commit access. If you've lived in the tree 
> without significant issues for three months, we assume you're re>without significant issues for three months, we assume you're rea
>ady to stand on your own. If somehow there are persistent problem>dy to stand on your own. If somehow there are persistent problems
>s during the first three months the vouchers have the authority t> during the first three months, the vouchers have the authority t
>o revoke access during this period. Vouching is a big responsibil>o request revocation of your access during this period. Vouching 
>ity, so people will make this commitment only after due considera>is a big responsibility, so people will make this commitment only
>tion. A voucher who helps people who aren't prepared get access t> after due consideration. A voucher who helps people who aren't p
>o the source tree will find his or her own credibility suffers as>repared get access to the source tree will find that his or her o
> well.>wn credibility suffers as well.
n82    <h4 name="Super-Reviewer_approval">n84    <h4 name="Super-Reviewer_Approval">
83      Super-Reviewer approval85      Super-Reviewer Approval
n86      If your code is in one or more modules that are included inn88      If your code is in one or more modules that are included in
> Firefox, Thunderbird or the SeaMonkey suite then you also need t> Firefox, Thunderbird, or the SeaMonkey suite, then you also need
>he approval of a "super-reviewer." Super-reviewers are a small se> the approval of a "super-reviewer." Super-reviewers are a small 
>t of contributors who have -- and are known to have -- a wide-ang>set of contributors who have -- and are known to have -- a wide-a
>le view of the codebase, and are particularly acute at identifyin>ngle view of the codebase, and are particularly acute at identify
>g cross-module issues, integration issues and other issues beyond>ing cross-module issues, integration issues, and other issues bey
> the ability of the patch to resolve the specified issue. More in>ond the ability of the patch to resolve the specified issue. More
>formation on super-review and a list of super-reviewers can be fo> information on super-review and a list of super-reviewers can be
>und at http://www.mozilla.org/hacking/reviewers.html> found at http://www.mozilla.org/hacking/reviewers.html.
n92      A nominee should have demonstrated both technical chops andn94      A nominee should have demonstrated both technical chops and
> an understanding of Mozilla workstyle (awareness of tree closure> an understanding of Mozilla workstyle (awareness of tree closure
>s, regressions and bustage processes, good citizenship, responds >s, regressions and bustage processes, good citizenship, good resp
>well to code reviews, makes changes when appropriate, etc.).>onsiveness to code reviews, makes changes when appropriate, etc.)
 >.
n104      <li>code qualityn106      <li>Code Quality
n106          <li>does the proposed committer's code solve the problen108          <li>Does the proposed committer's code solve the proble
>m it was intended to? does it do so well? does the code solve an >m it was intended to? does it do so well? does the code solve an 
>underlying problem rather than fix a symptom?>underlying problem rather than fix a symptom?
n108          <li>understanding of relevant aspects of Mozilla architn110          <li>Understanding of relevant aspects of Mozilla archit
>ecture; the definition of "relevant" will depend on the area(s) i>ecture; the definition of "relevant" will depend on the area(s) i
>n which one works.>n which one works.
n110          <li>understanding when one's code affects other modulesn112          <li>Understanding when one's code affects other modules
> and needs input beyond one's own area of expertise> and needs input beyond one's own area of expertise
n114      <li>workstlyen116      <li>Workstyle
n116          <li>understands and respects tree rules and related pron118          <li>Understands and respects tree rules and related pro
>cesses>cesses
n118          <li>availability to deal with issues in one's checkinsn120          <li>Availability to deal with issues in one's check-ins
n120          <li>addresses review comments responsiblyn122          <li>Addresses review comments responsibly
n124      <li>experiencen126      <li>Experience
n126          <li>should be a set of good-sized patches adequate to jn128          <li>Should be a set of good-sized patches adequate to j
>udge quality issues; and>udge quality issues; and
n128          <li>should have a track record that demonstrates other n130          <li>Should have a track record that demonstrates other 
>criteria -- at least a couple of months of active hacking that me>criteria -- at least a couple of months of active hacking that me
>ets the other criteria>ets the other criteria
n142      <li>NPSRn144      <li>NSPR
n144      <li>JSn146      <li>JavaScript
n149    <h4 name="Modules_not_associated_with_Firefox.2C_Thunderbird_n151    <h4 name="Modules_not_associated_with_Firefox.2C_Thunderbird.
>or_SeaMonkey">>2C_or_SeaMonkey">
150      Modules not associated with Firefox, Thunderbird or SeaMonk152      Modules not associated with Firefox, Thunderbird, or SeaMon
>ey>key
n153      Historically code which is not part of the browser and mailn155      Historically, code which is not part of the browser and mai
> products -- e.g., webtools, the source tree for the website, etc>l products -- e.g., webtools, the source tree for the website, et
>. -- have developed their own rules for source code commit access>c. -- have developed their own rules for source code commit acces
>. As of April 2007 we are evaluating whether this makes sense or >s. As of April 2007 we are evaluating whether this makes sense or
>whether one policy should address all Mozilla code. For now those> whether one policy should address all Mozilla code. For now thos
> projects should continue as they have been.>e projects should continue as they have been.
n159      You'll also need to fill out and submit the CVS Contributorn161      You'll also need to fill out and submit the <a class="exter
> Form to get your account set up. Signing the form indicates you >nal" href="http://www.mozilla.org/hacking/form.html">CVS Contribu
>understand and agree to our legal requirements. Breaking these ru>tor Form</a> to get your account set up. Signing the form indicat
>les could cause legal problems for Mozilla and may cause us to re>es you understand and agree to our legal requirements. Breaking t
>voke your access. If you have any questions, ask us.>hese rules could cause legal problems for Mozilla and may cause u
 >s to revoke your access. If you have any questions, ask us.
n162      <li>Some restrictions still apply to contributions of cryptn164      <li>Some restrictions still apply to contributions of crypt
>o source code and interfaces to crypto applications. Please consu>o source code and interfaces to crypto applications. Please consu
>lt with folks in the mozilla.dev.tech.crypto newsgroup before you>lt with folks in the <a class="external" href="https://lists.mozi
> prepare to check in anything new. (See the <a class="external" h>lla.org/listinfo/dev-tech-crypto/">mozilla.dev.tech.crypto newsgr
>ref="http://www.mozilla.org/crypto-faq.html">Mozilla Crypto FAQ</>oup</a> before you prepare to check-in anything new. (See the <a 
>a> for background information on Mozilla and crypto support.)>class="external" href="http://www.mozilla.org/crypto-faq.html">Mo
 >zilla Crypto FAQ</a> for background information on Mozilla and cr
 >ypto support.)
n164      <li>All new source files must contain a license header usinn166      <li>All new source files must contain a license header usin
>g the MPL/LGPL/GPL tri-license, or another license approved by mo>g the MPL/LGPL/GPL tri-license or another license approved by moz
>zilla.org (see the licensing policy); you must provide any legal >illa.org (see the licensing policy); you must provide any legal n
>notices required by the license.>otices required by the license.
n166      <li>The code you check in must belong to you or you must han168      <li>The code you check-in must belong to you or you must ha
>ve full rights to publish and license it.>ve full rights to publish and license it.
n168      <li>Your CVS account is for your own use only. Do not let on170      <li>Your CVS account is for your own use only. Do not let o
>thers use it. If you check in someone else's patch, attribute the>thers use it. If you check-in someone else's patch, attribute the
> patch to them in the CVS log.> patch to them in the CVS commit log.
t175      <li>Need to hear a rheeet?t177      <li>Need to hear a <a class="external" href="ftp://ftp.mozi
 >lla.org/pub/mozilla.org/mozilla/libraries/bonus-tracks/rheet.wav"
 >>rheeet</a>?

Back to History