canmove, Confirmed users
1,126
edits
No edit summary |
(Few tweaks) |
||
| Line 4: | Line 4: | ||
* <u><b>Verify that the patch has proper review before doing anything else</b></u> | * <u><b>Verify that the patch has proper review before doing anything else</b></u> | ||
** [https://wiki.mozilla.org/Modules Module information] | ** [https://wiki.mozilla.org/Modules Module information] | ||
* Add the patch to the appropriate repository's queue via saving the attachment from the bug and using hg import or using a tool like | * Add the patch to the appropriate repository's queue via saving the attachment from the bug and using hg import or using a tool like qimportbz to automate the process. | ||
** For general core patches, land on mozilla-inbound. | ** For general core patches, land on mozilla-inbound. | ||
** For core B2G work (*NOT* platform-independent core patches that happen to be fixing a B2G bug), use b2g-inbound. | ** For core B2G work (*NOT* platform-independent core patches that happen to be fixing a B2G bug), use b2g-inbound. | ||
** For Firefox front-end work, use fx-team. | ** For Firefox front-end work/any Firefox for Android-specific work, use fx-team. | ||
* Apply the patch via qpush. | * Apply the patch via qpush. | ||
* If the patch does not apply cleanly, you can look at the rejects and decide if the rebase is simple enough resolve yourself or you can remove the checkin-needed request and ask the developer to post an updated patch for checkin. <b>Don't forget to qrefresh after resolving conflicts!</b> | * If the patch does not apply cleanly, you can look at the rejects and decide if the rebase is simple enough resolve yourself or you can remove the checkin-needed request and ask the developer to post an updated patch for checkin. <b>Don't forget to qrefresh after resolving conflicts!</b> | ||