Compare Revisions

How to Write and Land Nanojit Patches

Change Revisions

Revision 61756:

Revision 61756 by nnethercote on

Revision 61757:

Revision 61757 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 61756
Revision 61757
t148      <li>In the hard case, you'll have a broken TM/TR repo.&nbspt148      <li>In the hard case, you'll have a broken TM/TR repo.&nbsp
>; This is because another recent external patch, P, landed on the>; This is because one or more recent external patch(es), P, lande
> other (TR/TM) repo, thus changing NJ's interface, but your TM/TR>d on the other (TR/TM) repo, thus changing NJ's API but your TM/T
> repo hasn't been updated.&nbsp; In this case, you have to [I'm n>R repo hasn't been updated to match the API&nbsp;change.&nbsp; In
>ot entirely sure how] work out the revision of P, find its bug re> this case, you have to look through the converted revisions you'
>port and apply P to your private repo.&nbsp; Then [somehow...] la>re about to push (run <code>hg outgoing</code> to list them) and 
>nd both P and the update copies to the public repo. [Is this a si>figure out which patch(es) caused the breakage. Then find the cor
>ngle step or two?]&nbsp; Then update the whiteboard for the bug c>responding bug(s) and commit the (hopefully already-reviewed!)&nb
>ontaining P and add a comment URL.&nbsp; And likewise for all rev>sp;TR/TM portion of the patch(es) <em>on top</em> of your current
>isions involved in the update.>, un-pushed tip. At this point the set of changes you are about t
 >o push will consist of a sequence of converted revisions from <co
 >de>nanojit-central</code>, followed by one of more "fixups"&nbsp;
 >to TR/TM to make them work properly. Test this entire group and l
 >and them with a single <code>hg push</code>. Then update the whit
 >eboard for the each such bug P and add a comment URL.&nbsp; And l
 >ikewise for all revisions involved in the update.

Back to History