이 번역은 완료되지 않았습니다. 이 문서를 번역해 주세요.
이것은 Firefox Developer Tools에 기여할 수 있는 방법을 다루고 있는 가이드입니다. 일반적으로, 공헌자가 밟아야 할 작업 순서는 다음과 같습니다.
- Firefox 코드를 받으세요.
- Firefox를 개발할 수 있도록 개발 환경을 구축해야 합니다.
- 버그 고치기
- 해결해야 할 버그를 찾습니다.
- 버그를 고치고, 몇가지 테스트를 작성합니다.
- 당신이 작성한 픽스를 다른 사람에게 리뷰받으세요.
- Check in your fix
- Make a patch that's ready to check in
- Run integration tests
- Ask a release engineer to check in your patch
This guide assumes that:
- You know Git but not Mercurial, and are not interested in learning Mercurial at the moment
- You are potentially interested in making more than one contribution, so are willing to invest a little time in getting set up to run integration tests
This guide does not cover the design and implementation of the DevTools code itself. For this, you're better off asking for help in the #devtools IRC channel or in the bug you are working on.
Mozilla source control: Mercurial and Git
Some of the complexities in working with Mozilla source code derive from the fact that Mozilla uses Mercurial as a version control system, not Git. If you know Mercurial, this is fine: but most people coming to Mozilla are familiar with Git but not Mercurial. What's more, most people are not interested in learning a new version control system just to make a few contributions to Firefox.
There is a Git mirror of the Firefox source code at https://github.com/mozilla/gecko-dev. It's kept up to date, and you can use it to build Firefox and work on your contribution, all the way through to getting final approval for your patch. However, the Git mirror is read-only: you can't push changes back to Firefox using only Git. Changes in Firefox must be committed to the Mercurial repository.
Usually, for your first patch, your mentor will take care of committing the patch for you, and you don't need to worry about Mercurial at all. However, if you want to become a regular contributor, you will need to be able to run integration tests (called pushing to try in Mozilla jargon), and to do this, you'll need to be able to push to a Mercurial repository.
In this guide we'll describe a setup which enables you to do this while still being able to use Git for your development, although there are other such setups.
Getting set up
Getting the Firefox source
Clone the GitHub mirror from https://github.com/mozilla/gecko-dev. This might take a while.
git clone firstname.lastname@example.org:mozilla/gecko-dev.git
The DevTools team uses the
fx-team branch for development, so check that branch out:
cd gecko-dev git checkout fx-team
The source code includes a tool called
mach, that you'll use to build, run, and test Firefox. It also has a
bootstrap command that sets up our build environment, so run that now:
This could also take a while, as it installs all your prerequisites.
Getting set up with Mercurial
.git/config for your
gecko-dev clone, add this line:
[alias] hgp = "show --binary --find-renames --format=\"# HG changeset patch%n# User %an <%ae>%n%B\" -U8"
This adds a command to
hgp, which will take the last commit you made and convert it into an Mercurial-style patch.
Next, clone the Mercurial Firefox repository into a parallel directory:
cd .. hg clone http://hg.mozilla.org/integration/fx-team
mach bootstrap should already have installed Mercurial for you.)
Next, install the Mozilla git tools in another parallel directory:
git clone https://github.com/mozilla/moz-git-tools
Make sure the scripts in this repository are executable by your shell by adding this directory to your
$PATH in your
At this point you should have a directory structure like:
dev /fx-team /gecko-dev /moz-git-tools
To build Firefox, navigate to the
gecko-dev directory and execute
./mach build. This will take a while:
cd gecko-dev ./mach build
Once you've built everything, you can build just the DevTools using:
./mach build devtools/*
This is much faster.
Note that whenever you pull the latest changes from fx-team into your local repository, you may need to "clobber". A "clobber" is similar to a "make clean". You'll know you need to clobber when you get a big error message telling you to do a clobber build after you tried to do an incremental build. To do a clobber build, enter these commands:
./mach clobber ./mach build
To run the Firefox you built:
./mach run -P my-profile
This will instruct
mach to run Firefox with the profile named "my-profile" (you can name the profile whatever you want). A profile is where all customizations of the basic Firefox installation are kept, including browsing history, tweaked settings, add-ons, and bookmarks. If the profile you specify does not already exist, it will be created.
To run tests, use
mach test, passing it a single test file or a directory containing test files:
./mach test path/to/test
Fixing a bug
We use Bugzilla for bug tracking, so you'll need to sign up there.
Finding a bug
If you don't already know what you want to work on, check out the list of good first bugs and mentored bugs at the firefox-dev.tools site (alternatively, you can find these lists on Bugzilla: good-first, mentored). This list of all open developer tools bugs might have other items you'd find interesting to work on.
While working on the bug, feel free to ask for help. If you don't know who to contact, this page should help you. Don't hesitate to ask to be mentored even on bugs that don't have a mentor assigned yet.
If you add a comment to your bug and need a response from someone, you can set the "Needs more information" (commonly abbreviated to NI) check box underneath the comment box, and add their email address:
This will flag the bug for their attention.
Asking for feedback
If you have a patch that's not yet ready for a proper review, but want feedback on a possible approach to solving a problem, you can attach your work-in-progress patch and ask for feedback.
To attach a patch, click the link in the bug labeled "Add an attachment". This opens a new page in which you can attach the patch, as well as ask for feedback or ask for review:
Asking for review
Once you think you are ready for review, create a patch. At this point the patch does not need any particular formatting, as long as it applies cleanly to the
fx-team branch. For example, you could just use the output of
git diff fx-team. If you have trouble finding the right reviewer, you can refer to this page.
However, it must apply cleanly to the
fx-team branch, so rebase your working branch before you create the patch, and fix up any conflicts.
You can ask for a review without including tests. But you will have to write tests, and get them reviewed as well, before you can check in your contribution.
Dealing with review comments
If the reviewer finds things that need to be fixed they will note them in the bug, and mark the review rejected (r-). Don't feel discouraged about this: it very common for even experienced Firefox developers to have their patches rejected.
Create a new patch that addresses the comments, attach it, and ask for review again. When you attach the patch, check the box that says that this new patch obsoletes the old one.
When your patch is approved, the reviewer will mark it r+.
It's possible that the reviewer will have some additional comments called "nits": these are things that should be fixed, but don't require another review round.
If you've got an r+ on a patch, but then fix the nits, or for some other reason have to regenerate the patch, then you can "carry over the r+": this means that you can mark it as "r+" yourself, when you attach the new patch.
So you've now implemented a feature, written tests, and it's all been reviewed. The final step is to run integration tests and, if they pass, ask for checkin.
Creating a patch to check in
First you'll need to rebase your changes into a single commit, with a commit message in the following format:
git commit -m "Bug 12345 - Summary of bug. r=<name of reviewer>" .
Check that your name and email address are set correctly in your .gitconfig.
git hgp HEAD > Bug12345.patch
This will generate a patch in Mercurial format. Attach this patch to the bug.
Pushing to try
Mozilla runs integration tests using servers called "try servers". To use a try server, you apply your patch, add a specially formatted commit message which tells the server which platforms to test, and push it to a specific Mercurial repository. The try servers then run integration tests as directed by the commit message, and you can see the results online.
First, you need commit access level 1 to push to the try server. File a bug asking for commit access level 1, attach the SSH public key you will be using (you can use the same one you use for GitHub, if you like), and ask your mentor to vouch for you.
Next, make sure your Mercurial repository (fx-team in Getting set up) is up to date. Make sure that your Git repository (gecko-dev in Getting set up) is in sync with it, with the addition of your patch.
Next, navigate to your gecko-dev directory and push to try with a command like this:
git push-to-try ../fx-team -b do -p linux,linux64,macosx64,win32 -u xpcshell,mochitests -t none
The long string of parameters here tells the server which platforms to use, and which tests to run. The set quoted above is a common set for DevTools changes, but your mentor can help you choose a different set if necessary.
If this is successful, the output from the command will give you a URL you can use to check on the progress of your build/test run.
Interpreting try results/retriggering
A complete try build/test run takes several hours. The page that shows you the results looks something like this:
Each row represents a platform, and each linked item in the row represents a test suite (for example, dt1, dt2, dt3 are DevTools test suites). Ideally you want all of these to be green. Red is a build error, orange is a test failure, and purple is an infrastructure failure.
Quite often, there are test failures, and these show up as orange. If you click the linked item, you'll see a detailed view about the test failure in the pane at the bottom:
Test failures are often associated with known intermittent bugs in Bugzilla, and if this is the case, that fact is noted in the detailed view. The screenshot above is showing us that this test failure is associated with bug 1177730.
If you have test or build failures, you'll need to determine whether or not they are caused by your patch. If they seem to be unconnected with the changes you've made, and are associated with a known intermittent, there's a good chance that they are not caused by your patch. One way to help determine this is to retrigger just that job: you can do this by clicking the "reload" icon:
Discuss any test failures with your mentor and decide whether it's OK to ask for checkin, or whether there are problems in your patch that need to be addressed.
Setting the checkin-needed keyword
If you've decided to go ahead with checkin, add a comment to your bug containing a link to the try run. If there are any test failures that you think are not caused by your patch, explain why you think that. Then add the "checkin-needed" keyword to the bug. To do this, just click in the "Keywords" text box in the bug, type "checkin-needed" and press Enter.
Within a few hours, a release engineer will check your patch into the tree.
Once your patch is checked in, there will be another round of build and test. If that's successful the patch will be merged into the mozilla-central branch of the Firefox code. Then, in the next day or so, your patch will appear in Nightly builds of Firefox, and will then appear in the next release of the Developer Edition.
The two best ways to get help are:
- Asking questions in the bug
- Asking questions in the #devtools channel on IRC. If you haven't used IRC before, see this guide to using IRC at Mozilla.