Mercurial > hgbook
annotate ja/mq-collab.tex @ 350:5a5419eeab70
more mq-collab.tex
author | Yoshiki Yazawa <yaz@honeyplanet.jp> |
---|---|
date | Mon, 18 Aug 2008 11:21:45 +0900 |
parents | 84ae26ab0ac3 |
children | f3fa98815193 |
rev | line source |
---|---|
349 | 1 %\chapter{Advanced uses of Mercurial Queues} |
2 \chapter{Mercurial Queues$B$N9bEY$J;H$$J}(B} | |
224
34943a3d50d6
Start writing up extensions. Begin with inotify.
Bryan O'Sullivan <bos@serpentine.com>
parents:
107
diff
changeset
|
3 \label{chap:mq-collab} |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
4 |
349 | 5 %While it's easy to pick up straightforward uses of Mercurial Queues, |
6 %use of a little discipline and some of MQ's less frequently used | |
7 %capabilities makes it possible to work in complicated development | |
8 %environments. | |
9 | |
10 Mercurial Queues$B$NC1=c$J;H$$J}$r<h$j>e$2$?0lJ}!$>/!9$NE}@)$H(BMQ$B$N$"$^$jMQ(B | |
11 $B$$$i$l$J$$5!G=$rMQ$$$k$3$H$GJ#;($J3+H/4D6-$G$N:n6H$,2DG=$H$J$k!%(B | |
12 | |
13 %In this chapter, I will use as an example a technique I have used to | |
14 %manage the development of an Infiniband device driver for the Linux | |
15 %kernel. The driver in question is large (at least as drivers go), | |
16 %with 25,000 lines of code spread across 35 source files. It is | |
17 %maintained by a small team of developers. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
18 |
349 | 19 $B$3$N>O$G$O!$(BLinux$B%+!<%M%kMQ(BInfiniband$B%G%P%$%9%I%i%$%P$N3+H/$r4IM}$9$k$N$K(B |
20 $BMQ$$$?%F%/%K%C%/$rNc$H$7$FMQ$$$k!%$3$N%I%i%$%P$O!J%I%i%$%P$H$7$F$O!KBg5,(B | |
21 $BLO$G!$(B35$B$N%=!<%9%U%!%$%k$K$o$?$k(B25000$B9T$+$i$J$k!%$3$N%=!<%9$O>.5,LO$J3+H/(B | |
22 $B<T$N%A!<%`$K$h$C$F$$;~4IM}$5$l$F$$$k!%(B | |
23 | |
24 %While much of the material in this chapter is specific to Linux, the | |
25 %same principles apply to any code base for which you're not the | |
26 %primary owner, and upon which you need to do a lot of development. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
27 |
349 | 28 $B$3$N>O$G$NAG:`$O(BLinux$B$KFC2=$7$F$$$k$,!$F1$886B'$O$$$+$J$k%3!<%I%Y!<%9$KBP(B |
29 $B$7$F$bE,MQ2DG=$G!$<+J,<+?H$G=jM-$7$F$$$J$$$,!$B?$/$N3+H/$r9T$&I,MW$N$"$k(B | |
30 $B%3!<%I$J$I$K$bE,MQ$G$-$k!%(B | |
31 | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
32 |
349 | 33 %\section{The problem of many targets} |
34 \section{$B%?!<%2%C%H$,J#?t$"$k$H$$$&LdBj(B} | |
35 | |
36 %The Linux kernel changes rapidly, and has never been internally | |
37 %stable; developers frequently make drastic changes between releases. | |
38 %This means that a version of the driver that works well with a | |
39 %particular released version of the kernel will not even \emph{compile} | |
40 %correctly against, typically, any other version. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
41 |
349 | 42 Linux$B%+!<%M%k$O5^B.$KJQ2=$7!$$=$NFbIt$O7h$7$F0lDj$G$O$J$$!%3+H/<T$?$A$OIQ(B |
43 $BHK$K%j%j!<%94V$G7`E*$JJQ99$r9T$&!%$3$N$?$a!$FCDj$N%j%j!<%9%P!<%8%g%s$GNI(B | |
44 $B$/F0$$$?%I%i%$%P$,!$Nc$($PJL$N%P!<%8%g%s$G$O@5$7$/%3%s%Q%$%k$9$k$3$H$9$i(B | |
45 $B$G$-$J$/$J$k$H$$$&$3$H$r0UL#$9$k!%(B | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
46 |
349 | 47 %To maintain a driver, we have to keep a number of distinct versions of |
48 %Linux in mind. | |
49 | |
50 $B%I%i%$%P$r0];}$9$k$?$a$K!$3+H/<T$O$$$/$D$b$NJL$N%P!<%8%g%s$N(BLinux$B$rA[Dj(B | |
51 $B$7$J$1$l$P$J$i$J$$!%(B | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
52 \begin{itemize} |
349 | 53 %\item One target is the main Linux kernel development tree. |
54 | |
55 % Maintenance of the code is in this case partly shared by other | |
56 % developers in the kernel community, who make ``drive-by'' | |
57 % modifications to the driver as they develop and refine kernel | |
58 % subsystems. | |
59 \item $B0l$D$N%?!<%2%C%H$O(BLinux$B%+!<%M%k$N3+H/%D%j!<$G$"$k!%$3$N>l9g!$%3!<%I(B | |
60 $B$N0];}4IM}$O%+!<%M%k%3%_%e%K%F%#$NB>$N3+H/<T!J$3$l$O%+!<%M%k$N%5%V(B | |
61 $B%7%9%F%`$r2~NI$9$k$?$a$K%I%i%$%P$r(B``$BDI$$N)$F$k(B''$BJQ99$r9T$&3+H/<T$G(B | |
62 $B$"$k!K$HItJ,E*$K6&M-$5$l$F$$$k!%(B | |
63 %\item We also maintain a number of ``backports'' to older versions of | |
64 % the Linux kernel, to support the needs of customers who are running | |
65 % older Linux distributions that do not incorporate our drivers. (To | |
66 % \emph{backport} a piece of code is to modify it to work in an older | |
67 % version of its target environment than the version it was developed | |
68 % for.) | |
69 \item $B3+H/<T$O2f!9$N%I%i%$%P$,AH$_9~$^$l$F$$$J$$8E$$(BLinux$B%G%#%9%H%j%S%e!<(B | |
70 $B%7%g%s$r;H$C$F$$$k8\5R$r%5%]!<%H$9$k$?$a$K!$8E$$%P!<%8%g%s$N(BLinux$B%+!<(B | |
71 $B%M%k$X$N(B``$B%P%C%/%]!<%H(B''$B$rJz$($F$$$k!%%3!<%I$N0lIt$r(B\emph{$B%P%C%/%]!<(B | |
72 $B%H(B}$B$9$k$3$H$O!$%I%i%$%P$,3+H/$5$l$?%P!<%8%g%s$h$j$b8E$$%P!<%8%g%s(B | |
73 $B$N4D6-$GF0$/$h$&$K$9$k$3$H$r0UL#$9$k!%(B | |
74 %\item Finally, we make software releases on a schedule that is | |
75 % necessarily not aligned with those used by Linux distributors and | |
76 % kernel developers, so that we can deliver new features to customers | |
77 % without forcing them to upgrade their entire kernels or | |
78 % distributions. | |
79 \item $B:G=*E*$K(BLinux$B%G%#%9%H%j%S%e!<%?$d%+!<%M%k3+H/<T$rBT$?$;$k$3$H$J$/%9(B | |
80 $B%1%8%e!<%kDL$j$K%=%U%H%&%'%"$r%j%j!<%9$9$k$3$H$,I,MW$G$"$j!$$3$l$K$h$C$F(B | |
81 $B8\5R$K%+!<%M%k$d%G%#%9%H%j%S%e!<%7%g%sA4BN$r%"%C%W%0%l!<%I$5$;$k$3$H$J$/(B | |
82 $B?75!G=$r8\5R$KFO$1$i$l$k!%(B | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
83 \end{itemize} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
84 |
349 | 85 %\subsection{Tempting approaches that don't work well} |
86 \subsection{$B$d$C$F$7$^$$$,$A$J4V0c$C$?J}K!(B} | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
87 |
350 | 88 %There are two ``standard'' ways to maintain a piece of software that |
89 %has to target many different environments. | |
90 | |
91 $B$5$^$6$^$J4D6-$r%?!<%2%C%H$K$7$?%=%U%H%&%'%"$r4IM}$9$k(B2$B$D$NI8=`E*$JJ}K!(B | |
92 $B$,$"$k!%(B | |
93 | |
94 %The first is to maintain a number of branches, each intended for a | |
95 %single target. The trouble with this approach is that you must | |
96 %maintain iron discipline in the flow of changes between repositories. | |
97 %A new feature or bug fix must start life in a ``pristine'' repository, | |
98 %then percolate out to every backport repository. Backport changes are | |
99 %more limited in the branches they should propagate to; a backport | |
100 %change that is applied to a branch where it doesn't belong will | |
101 %probably stop the driver from compiling. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
102 |
350 | 103 $BBh(B1$B$NJ}K!$O!$C10l$N%?!<%2%C%H8~$1$NJ#?t$N%V%i%s%A$r0];}$9$k$3$H$G$"$k!%$3(B |
104 $B$NJ}K!$NLdBjE@$O!$%j%]%8%H%j4V$G$NJQ99$NN.$l$K$D$$$F873J$J5,N'$r0];}$7$J(B | |
105 $B$1$l$P$J$i$J$$$3$H$G$"$k!%?75!G=$d%P%0=$@5$O!$$-$l$$$J>uBV$N%j%]%8%H%j$G(B | |
106 $B3+;O$7!$%P%C%/%]!<%H%j%]%8%H%j$K?;F)$9$kI,MW$,$"$k!%%P%C%/%]!<%HJQ99$OGH(B | |
107 $B5Z$9$Y$-%V%i%s%AFb$K8BDj$5$l$F$$$J$1$l$P$J$i$J$$!%I,MW$N$J$$%V%i%s%A$X$N(B | |
108 $B%P%C%/%]!<%H$NGH5Z$O$*$=$i$/%I%i%$%P$r%3%s%Q%$%kITG=$K$7$F$7$^$&$@$m$&!%(B | |
109 | |
110 %The second is to maintain a single source tree filled with conditional | |
111 %statements that turn chunks of code on or off depending on the | |
112 %intended target. Because these ``ifdefs'' are not allowed in the | |
113 %Linux kernel tree, a manual or automatic process must be followed to | |
114 %strip them out and yield a clean tree. A code base maintained in this | |
115 %fashion rapidly becomes a rat's nest of conditional blocks that are | |
116 %difficult to understand and maintain. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
117 |
350 | 118 $BBh(B2$B$NJ}K!$O!$%A%c%s%/$d%3!<%I$r!$L\E*$H$9$k%?!<%2%C%HKh$K(Bon/off$B$9$k>r7oJ8(B |
119 $B$rDI2C$7$?C10l$N%=!<%9%D%j!<$r0];}$9$kJ}K!$G$"$k!%$3$l$i$N(B``ifdef''$B$O(B | |
120 linux$B%+!<%M%k%D%j!<$G$O5v$5$l$F$$$J$$$?$a!$<jF0$^$?$O<+F0$G$3$l$i$N%3!<%I(B | |
121 $B$r=|5n$7!$%/%j!<%s$J%D%j!<$r:n$k%W%m%;%9$,I,MW$K$J$k!%$3$N$h$&$J$d$jJ}$G(B | |
122 $B4IM}$5$l$?%3!<%I%Y!<%9$O$9$0$K>r7o@a$NAc7"$H2=$7!"M}2r$d4IM}$NK8$2$H$J$k!#(B | |
123 | |
124 %Neither of these approaches is well suited to a situation where you | |
125 %don't ``own'' the canonical copy of a source tree. In the case of a | |
126 | |
127 %Linux driver that is distributed with the standard kernel, Linus's | |
128 %tree contains the copy of the code that will be treated by the world | |
129 %as canonical. The upstream version of ``my'' driver can be modified | |
130 %by people I don't know, without me even finding out about it until | |
131 %after the changes show up in Linus's tree. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
132 |
350 | 133 $B$"$J$?$,@5<0$J%=!<%9%D%j!<$r=jM-$7$F$$$k$N$G$J$1$l$P!"$3$l$i$N$d$jJ}$N$I(B |
134 $B$A$i$b$=$0$o$J$$$@$m$&!#I8=`$N(Blinux$B%+!<%M%k$KF1:-$5$l$F$$$k%I%i%$%P$N>l(B | |
135 $B9g!$(BLinus$B$N%D%j!<$O@$3&Cf$G@5<0$J%3!<%I$H$7$F07$o$l$k%3%T!<$r4^$s$G$$$k!%(B | |
136 ``$B;d(B''$B$N%I%i%$%P$N>eN.HG$O!$CN$i$J$$?M$K$h$C$FJQ99$5$lF@$k$7!$<+J,$NJQ99(B | |
137 $B$,(BLinus$B$N%D%j!<$KF~$C$?$"$H$G$5$($bJQ99$5$lF@$k!%(B | |
138 | |
139 %These approaches have the added weakness of making it difficult to | |
140 %generate well-formed patches to submit upstream. | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
141 |
350 | 142 $B$3$l$i$N%"%W%m!<%A$O!$$&$^$/=q$+$l$?%Q%C%A$r>eN.$XDs=P$9$k$3$H$r:$Fq$K$7(B |
143 $B$F$7$^$&!%(B | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
144 |
350 | 145 %In principle, Mercurial Queues seems like a good candidate to manage a |
146 %development scenario such as the above. While this is indeed the | |
147 %case, MQ contains a few added features that make the job more | |
148 %pleasant. | |
149 | |
150 Mercurial Queues$B$O>e5-$N$h$&$J3+H/%7%J%j%*$r4IM}$9$k$h$$8uJd$N0l$D$G$"$k(B | |
151 $B$H$$$($k!%$3$N$h$&$J>l9g$N$?$a$K(BMQ$B$K$O:n6H$r$h$j2wE,$K$9$k$$$/$D$+$N5!G=(B | |
152 $B$,$"$k!%(B | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
153 |
349 | 154 %\section{Conditionally applying patches with guards} |
155 \section{$B%,!<%I$r;H$C$?%Q%C%A$N>r7oE*$JE,MQ(B} | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
156 |
350 | 157 %Perhaps the best way to maintain sanity with so many targets is to be |
158 %able to choose specific patches to apply for a given situation. MQ | |
159 %provides a feature called ``guards'' (which originates with quilt's | |
160 %\texttt{guards} command) that does just this. To start off, let's | |
161 %create a simple repository for experimenting in. | |
162 %\interaction{mq.guards.init} | |
163 %This gives us a tiny repository that contains two patches that don't | |
164 %have any dependencies on each other, because they touch different files. | |
165 | |
166 | |
167 | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
168 |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
169 The idea behind conditional application is that you can ``tag'' a |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
170 patch with a \emph{guard}, which is simply a text string of your |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
171 choosing, then tell MQ to select specific guards to use when applying |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
172 patches. MQ will then either apply, or skip over, a guarded patch, |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
173 depending on the guards that you have selected. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
174 |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
175 A patch can have an arbitrary number of guards; |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
176 each one is \emph{positive} (``apply this patch if this guard is |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
177 selected'') or \emph{negative} (``skip this patch if this guard is |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
178 selected''). A patch with no guards is always applied. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
179 |
349 | 180 %\section{Controlling the guards on a patch} |
181 \section{$B%Q%C%AFb$N%,!<%I$rA`:n$9$k(B} | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
182 |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
183 The \hgxcmd{mq}{qguard} command lets you determine which guards should |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
184 apply to a patch, or display the guards that are already in effect. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
185 Without any arguments, it displays the guards on the current topmost |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
186 patch. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
187 \interaction{mq.guards.qguard} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
188 To set a positive guard on a patch, prefix the name of the guard with |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
189 a ``\texttt{+}''. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
190 \interaction{mq.guards.qguard.pos} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
191 To set a negative guard on a patch, prefix the name of the guard with |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
192 a ``\texttt{-}''. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
193 \interaction{mq.guards.qguard.neg} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
194 |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
195 \begin{note} |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
196 The \hgxcmd{mq}{qguard} command \emph{sets} the guards on a patch; it |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
197 doesn't \emph{modify} them. What this means is that if you run |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
198 \hgcmdargs{qguard}{+a +b} on a patch, then \hgcmdargs{qguard}{+c} on |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
199 the same patch, the \emph{only} guard that will be set on it |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
200 afterwards is \texttt{+c}. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
201 \end{note} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
202 |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
203 Mercurial stores guards in the \sfilename{series} file; the form in |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
204 which they are stored is easy both to understand and to edit by hand. |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
205 (In other words, you don't have to use the \hgxcmd{mq}{qguard} command if |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
206 you don't want to; it's okay to simply edit the \sfilename{series} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
207 file.) |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
208 \interaction{mq.guards.series} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
209 |
349 | 210 %\section{Selecting the guards to use} |
211 \section{$B;HMQ$9$k%,!<%I$rA*$V(B} | |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
212 |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
213 The \hgxcmd{mq}{qselect} command determines which guards are active at a |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
214 given time. The effect of this is to determine which patches MQ will |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
215 apply the next time you run \hgxcmd{mq}{qpush}. It has no other effect; in |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
216 particular, it doesn't do anything to patches that are already |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
217 applied. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
218 |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
219 With no arguments, the \hgxcmd{mq}{qselect} command lists the guards |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
220 currently in effect, one per line of output. Each argument is treated |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
221 as the name of a guard to apply. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
222 \interaction{mq.guards.qselect.foo} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
223 In case you're interested, the currently selected guards are stored in |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
224 the \sfilename{guards} file. |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
225 \interaction{mq.guards.qselect.cat} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
226 We can see the effect the selected guards have when we run |
233
696b1e0c01df
Tag all MQ commands as belonging to the mq extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
224
diff
changeset
|
227 \hgxcmd{mq}{qpush}. |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
228 \interaction{mq.guards.qselect.qpush} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
229 |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
230 A guard cannot start with a ``\texttt{+}'' or ``\texttt{-}'' |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
231 character. The name of a guard must not contain white space, but most |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
232 othter characters are acceptable. If you try to use a guard with an |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
233 invalid name, MQ will complain: |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
234 \interaction{mq.guards.qselect.error} |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
235 Changing the selected guards changes the patches that are applied. |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
236 \interaction{mq.guards.qselect.quux} |
105
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
237 You can see in the example below that negative guards take precedence |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
238 over positive guards. |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
239 \interaction{mq.guards.qselect.foobar} |
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
240 |
349 | 241 %\section{MQ's rules for applying patches} |
242 \section{MQ$B$N%Q%C%AE,MQ%k!<%k(B} | |
105
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
243 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
244 The rules that MQ uses when deciding whether to apply a patch |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
245 are as follows. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
246 \begin{itemize} |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
247 \item A patch that has no guards is always applied. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
248 \item If the patch has any negative guard that matches any currently |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
249 selected guard, the patch is skipped. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
250 \item If the patch has any positive guard that matches any currently |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
251 selected guard, the patch is applied. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
252 \item If the patch has positive or negative guards, but none matches |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
253 any currently selected guard, the patch is skipped. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
254 \end{itemize} |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
255 |
349 | 256 %\section{Trimming the work environment} |
257 \section{$B:n6H4D6-$r=L>.$9$k(B} | |
105
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
258 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
259 In working on the device driver I mentioned earlier, I don't apply the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
260 patches to a normal Linux kernel tree. Instead, I use a repository |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
261 that contains only a snapshot of the source files and headers that are |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
262 relevant to Infiniband development. This repository is~1\% the size |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
263 of a kernel repository, so it's easier to work with. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
264 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
265 I then choose a ``base'' version on top of which the patches are |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
266 applied. This is a snapshot of the Linux kernel tree as of a revision |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
267 of my choosing. When I take the snapshot, I record the changeset ID |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
268 from the kernel repository in the commit message. Since the snapshot |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
269 preserves the ``shape'' and content of the relevant parts of the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
270 kernel tree, I can apply my patches on top of either my tiny |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
271 repository or a normal kernel tree. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
272 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
273 Normally, the base tree atop which the patches apply should be a |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
274 snapshot of a very recent upstream tree. This best facilitates the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
275 development of patches that can easily be submitted upstream with few |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
276 or no modifications. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
277 |
349 | 278 %\section{Dividing up the \sfilename{series} file} |
279 \section{\sfilename{series}$B%U%!%$%k$X%@%$%V$9$k(B} | |
105
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
280 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
281 I categorise the patches in the \sfilename{series} file into a number |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
282 of logical groups. Each section of like patches begins with a block |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
283 of comments that describes the purpose of the patches that follow. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
284 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
285 The sequence of patch groups that I maintain follows. The ordering of |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
286 these groups is important; I'll describe why after I introduce the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
287 groups. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
288 \begin{itemize} |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
289 \item The ``accepted'' group. Patches that the development team has |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
290 submitted to the maintainer of the Infiniband subsystem, and which |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
291 he has accepted, but which are not present in the snapshot that the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
292 tiny repository is based on. These are ``read only'' patches, |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
293 present only to transform the tree into a similar state as it is in |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
294 the upstream maintainer's repository. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
295 \item The ``rework'' group. Patches that I have submitted, but that |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
296 the upstream maintainer has requested modifications to before he |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
297 will accept them. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
298 \item The ``pending'' group. Patches that I have not yet submitted to |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
299 the upstream maintainer, but which we have finished working on. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
300 These will be ``read only'' for a while. If the upstream maintainer |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
301 accepts them upon submission, I'll move them to the end of the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
302 ``accepted'' group. If he requests that I modify any, I'll move |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
303 them to the beginning of the ``rework'' group. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
304 \item The ``in progress'' group. Patches that are actively being |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
305 developed, and should not be submitted anywhere yet. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
306 \item The ``backport'' group. Patches that adapt the source tree to |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
307 older versions of the kernel tree. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
308 \item The ``do not ship'' group. Patches that for some reason should |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
309 never be submitted upstream. For example, one such patch might |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
310 change embedded driver identification strings to make it easier to |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
311 distinguish, in the field, between an out-of-tree version of the |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
312 driver and a version shipped by a distribution vendor. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
313 \end{itemize} |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
314 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
315 Now to return to the reasons for ordering groups of patches in this |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
316 way. We would like the lowest patches in the stack to be as stable as |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
317 possible, so that we will not need to rework higher patches due to |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
318 changes in context. Putting patches that will never be changed first |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
319 in the \sfilename{series} file serves this purpose. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
320 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
321 We would also like the patches that we know we'll need to modify to be |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
322 applied on top of a source tree that resembles the upstream tree as |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
323 closely as possible. This is why we keep accepted patches around for |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
324 a while. |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
325 |
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
326 The ``backport'' and ``do not ship'' patches float at the end of the |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
327 \sfilename{series} file. The backport patches must be applied on top |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
328 of all other patches, and the ``do not ship'' patches might as well |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
329 stay out of harm's way. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
330 |
349 | 331 %\section{Maintaining the patch series} |
332 \section{$B%Q%C%A%7%j!<%:$r4IM}$9$k(B} | |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
333 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
334 In my work, I use a number of guards to control which patches are to |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
335 be applied. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
336 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
337 \begin{itemize} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
338 \item ``Accepted'' patches are guarded with \texttt{accepted}. I |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
339 enable this guard most of the time. When I'm applying the patches |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
340 on top of a tree where the patches are already present, I can turn |
271 | 341 this patch off, and the patches that follow it will apply cleanly. |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
342 \item Patches that are ``finished'', but not yet submitted, have no |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
343 guards. If I'm applying the patch stack to a copy of the upstream |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
344 tree, I don't need to enable any guards in order to get a reasonably |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
345 safe source tree. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
346 \item Those patches that need reworking before being resubmitted are |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
347 guarded with \texttt{rework}. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
348 \item For those patches that are still under development, I use |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
349 \texttt{devel}. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
350 \item A backport patch may have several guards, one for each version |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
351 of the kernel to which it applies. For example, a patch that |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
352 backports a piece of code to~2.6.9 will have a~\texttt{2.6.9} guard. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
353 \end{itemize} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
354 This variety of guards gives me considerable flexibility in |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
355 qdetermining what kind of source tree I want to end up with. For most |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
356 situations, the selection of appropriate guards is automated during |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
357 the build process, but I can manually tune the guards to use for less |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
358 common circumstances. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
359 |
349 | 360 %\subsection{The art of writing backport patches} |
361 \subsection{$B%P%C%/%]!<%H%Q%C%A$r=q$/5;=Q(B} | |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
362 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
363 Using MQ, writing a backport patch is a simple process. All such a |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
364 patch has to do is modify a piece of code that uses a kernel feature |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
365 not present in the older version of the kernel, so that the driver |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
366 continues to work correctly under that older version. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
367 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
368 A useful goal when writing a good backport patch is to make your code |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
369 look as if it was written for the older version of the kernel you're |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
370 targeting. The less obtrusive the patch, the easier it will be to |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
371 understand and maintain. If you're writing a collection of backport |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
372 patches to avoid the ``rat's nest'' effect of lots of |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
373 \texttt{\#ifdef}s (hunks of source code that are only used |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
374 conditionally) in your code, don't introduce version-dependent |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
375 \texttt{\#ifdef}s into the patches. Instead, write several patches, |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
376 each of which makes unconditional changes, and control their |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
377 application using guards. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
378 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
379 There are two reasons to divide backport patches into a distinct |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
380 group, away from the ``regular'' patches whose effects they modify. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
381 The first is that intermingling the two makes it more difficult to use |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
382 a tool like the \hgext{patchbomb} extension to automate the process of |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
383 submitting the patches to an upstream maintainer. The second is that |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
384 a backport patch could perturb the context in which a subsequent |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
385 regular patch is applied, making it impossible to apply the regular |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
386 patch cleanly \emph{without} the earlier backport patch already being |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
387 applied. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
388 |
349 | 389 %\section{Useful tips for developing with MQ} |
390 \section{MQ$B$K$h$k3+H/$N(Btips} | |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
391 |
349 | 392 %\subsection{Organising patches in directories} |
393 \subsection{$B%G%#%l%/%H%jFb$G%Q%C%A$r4IM}$9$k(B} | |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
394 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
395 If you're working on a substantial project with MQ, it's not difficult |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
396 to accumulate a large number of patches. For example, I have one |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
397 patch repository that contains over 250 patches. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
398 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
399 If you can group these patches into separate logical categories, you |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
400 can if you like store them in different directories; MQ has no |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
401 problems with patch names that contain path separators. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
402 |
349 | 403 %\subsection{Viewing the history of a patch} |
404 \subsection{$B%Q%C%A$N%R%9%H%j$r8+$k(B} | |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
405 \label{mq-collab:tips:interdiff} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
406 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
407 If you're developing a set of patches over a long time, it's a good |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
408 idea to maintain them in a repository, as discussed in |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
409 section~\ref{sec:mq:repo}. If you do so, you'll quickly discover that |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
410 using the \hgcmd{diff} command to look at the history of changes to a |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
411 patch is unworkable. This is in part because you're looking at the |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
412 second derivative of the real code (a diff of a diff), but also |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
413 because MQ adds noise to the process by modifying time stamps and |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
414 directory names when it updates a patch. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
415 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
416 However, you can use the \hgext{extdiff} extension, which is bundled |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
417 with Mercurial, to turn a diff of two versions of a patch into |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
418 something readable. To do this, you will need a third-party package |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
419 called \package{patchutils}~\cite{web:patchutils}. This provides a |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
420 command named \command{interdiff}, which shows the differences between |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
421 two diffs as a diff. Used on two versions of the same diff, it |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
422 generates a diff that represents the diff from the first to the second |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
423 version. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
424 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
425 You can enable the \hgext{extdiff} extension in the usual way, by |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
426 adding a line to the \rcsection{extensions} section of your \hgrc. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
427 \begin{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
428 [extensions] |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
429 extdiff = |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
430 \end{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
431 The \command{interdiff} command expects to be passed the names of two |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
432 files, but the \hgext{extdiff} extension passes the program it runs a |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
433 pair of directories, each of which can contain an arbitrary number of |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
434 files. We thus need a small program that will run \command{interdiff} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
435 on each pair of files in these two directories. This program is |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
436 available as \sfilename{hg-interdiff} in the \dirname{examples} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
437 directory of the source code repository that accompanies this book. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
438 \excode{hg-interdiff} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
439 |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
440 With the \sfilename{hg-interdiff} program in your shell's search path, |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
441 you can run it as follows, from inside an MQ patch directory: |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
442 \begin{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
443 hg extdiff -p hg-interdiff -r A:B my-change.patch |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
444 \end{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
445 Since you'll probably want to use this long-winded command a lot, you |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
446 can get \hgext{hgext} to make it available as a normal Mercurial |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
447 command, again by editing your \hgrc. |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
448 \begin{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
449 [extdiff] |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
450 cmd.interdiff = hg-interdiff |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
451 \end{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
452 This directs \hgext{hgext} to make an \texttt{interdiff} command |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
453 available, so you can now shorten the previous invocation of |
238 | 454 \hgxcmd{extdiff}{extdiff} to something a little more wieldy. |
106
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
455 \begin{codesample2} |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
456 hg interdiff -r A:B my-change.patch |
9cbc5d0db542
Finish off advanced MQ chapter (maybe).
Bryan O'Sullivan <bos@serpentine.com>
parents:
105
diff
changeset
|
457 \end{codesample2} |
105
ecacb6b4c9fd
Grouping patches in the series file.
Bryan O'Sullivan <bos@serpentine.com>
parents:
104
diff
changeset
|
458 |
107
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
459 \begin{note} |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
460 The \command{interdiff} command works well only if the underlying |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
461 files against which versions of a patch are generated remain the |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
462 same. If you create a patch, modify the underlying files, and then |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
463 regenerate the patch, \command{interdiff} may not produce useful |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
464 output. |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
465 \end{note} |
a0d7e11db169
A brief note on interdiff.
Bryan O'Sullivan <bos@serpentine.com>
parents:
106
diff
changeset
|
466 |
240
4119e57679f7
Twiddle wording trivially.
Bryan O'Sullivan <bos@serpentine.com>
parents:
239
diff
changeset
|
467 The \hgext{extdiff} extension is useful for more than merely improving |
239
fe8598eb99c4
Refer to main coverage of extdiff extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
238
diff
changeset
|
468 the presentation of MQ~patches. To read more about it, go to |
fe8598eb99c4
Refer to main coverage of extdiff extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
238
diff
changeset
|
469 section~\ref{sec:hgext:extdiff}. |
fe8598eb99c4
Refer to main coverage of extdiff extension.
Bryan O'Sullivan <bos@serpentine.com>
parents:
238
diff
changeset
|
470 |
349 | 471 %%% Local Variables: |
293
3b1291f24c0d
- replaved latex-mode to yatex-mode
Yoshiki Yazawa <yaz@cc.rim.or.jp>
parents:
290
diff
changeset
|
472 %%% mode: yatex |
104
32bf9a5f22c0
Refactor MQ chapter into three.
Bryan O'Sullivan <bos@serpentine.com>
parents:
diff
changeset
|
473 %%% TeX-master: "00book" |
349 | 474 %%% End: |