Compare Revisions

How to Write and Land Nanojit Patches

Change Revisions

Revision 61757:

Revision 61757 by Graydon on

Revision 61758:

Revision 61758 by Graydon on

Title:
How to Write and Land Nanojit Patches
How to Write and Land Nanojit Patches
Slug:
NanojitMerge
NanojitMerge
Tags:
Obsolete
Obsolete
Content:

Revision 61757
Revision 61758
t161      <li>Once all the patches pass review, first land the nanojit161      <li>Once <em>all</em> the patches pass review:
>t patch on nanojit-central, mark the bug's whiteboard as <span st 
>yle="font-family: Courier New;">fixed-in-nanojit</span>, and note 
> the revision in a comment. 
162        <ul>
163          <li>If you are only <em>deleting</em> API, land the TM/
 >TR portions first, retiring all uses of the to-be-killed nanojit 
 >code. Only <em>then</em> should you land the nanojit code to nano
 >jit-central.
162      </li>164          </li>
163      <li>Then do a TM/TR update as in scenario 2, but with the a165          <li>If you're <em>adding or changing</em> API, first la
>dded complication that you've got additional outstanding changes >nd the nanojit patch on nanojit-central
>in the patch you're working on.&nbsp; Who knows how that's suppos 
>ed to work out... 
166          </li>
167        </ul>
168      </li>
169      <li>Mark the bug as <code>fixed-in-nanojit</code>
170      </li>
171      <li>Then do a TM/TR update as in scenario 2. If you were <e
 >m>changing</em> API, you will probably find yourself in the case 
 >marked "<strong>the hard case</strong>". Remember not to push to 
 >TR/TM if you're breaking their build.

Back to History