| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1 | <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" | 
|  | 2 | "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd"> | 
|  | 3 | <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en"> | 
|  | 4 | <head> | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 5 | <meta http-equiv="Content-Type" content="application/xhtml+xml; charset=UTF-8" /> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 6 | <meta name="generator" content="AsciiDoc 8.6.9" /> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 7 | <title>How to revert a faulty merge</title> | 
|  | 8 | <style type="text/css"> | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 9 | /* Shared CSS for AsciiDoc xhtml11 and html5 backends */ | 
|  | 10 |  | 
|  | 11 | /* Default font. */ | 
|  | 12 | body { | 
|  | 13 | font-family: Georgia,serif; | 
|  | 14 | } | 
|  | 15 |  | 
|  | 16 | /* Title font. */ | 
|  | 17 | h1, h2, h3, h4, h5, h6, | 
|  | 18 | div.title, caption.title, | 
|  | 19 | thead, p.table.header, | 
|  | 20 | #toctitle, | 
|  | 21 | #author, #revnumber, #revdate, #revremark, | 
|  | 22 | #footer { | 
|  | 23 | font-family: Arial,Helvetica,sans-serif; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 24 | } | 
|  | 25 |  | 
|  | 26 | body { | 
|  | 27 | margin: 1em 5% 1em 5%; | 
|  | 28 | } | 
|  | 29 |  | 
|  | 30 | a { | 
|  | 31 | color: blue; | 
|  | 32 | text-decoration: underline; | 
|  | 33 | } | 
|  | 34 | a:visited { | 
|  | 35 | color: fuchsia; | 
|  | 36 | } | 
|  | 37 |  | 
|  | 38 | em { | 
|  | 39 | font-style: italic; | 
|  | 40 | color: navy; | 
|  | 41 | } | 
|  | 42 |  | 
|  | 43 | strong { | 
|  | 44 | font-weight: bold; | 
|  | 45 | color: #083194; | 
|  | 46 | } | 
|  | 47 |  | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 48 | h1, h2, h3, h4, h5, h6 { | 
|  | 49 | color: #527bbd; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 50 | margin-top: 1.2em; | 
|  | 51 | margin-bottom: 0.5em; | 
|  | 52 | line-height: 1.3; | 
|  | 53 | } | 
|  | 54 |  | 
|  | 55 | h1, h2, h3 { | 
|  | 56 | border-bottom: 2px solid silver; | 
|  | 57 | } | 
|  | 58 | h2 { | 
|  | 59 | padding-top: 0.5em; | 
|  | 60 | } | 
|  | 61 | h3 { | 
|  | 62 | float: left; | 
|  | 63 | } | 
|  | 64 | h3 + * { | 
|  | 65 | clear: left; | 
|  | 66 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 67 | h5 { | 
|  | 68 | font-size: 1.0em; | 
|  | 69 | } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 70 |  | 
|  | 71 | div.sectionbody { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 72 | margin-left: 0; | 
|  | 73 | } | 
|  | 74 |  | 
|  | 75 | hr { | 
|  | 76 | border: 1px solid silver; | 
|  | 77 | } | 
|  | 78 |  | 
|  | 79 | p { | 
|  | 80 | margin-top: 0.5em; | 
|  | 81 | margin-bottom: 0.5em; | 
|  | 82 | } | 
|  | 83 |  | 
|  | 84 | ul, ol, li > p { | 
|  | 85 | margin-top: 0; | 
|  | 86 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 87 | ul > li { color: #aaa; } | 
|  | 88 | ul > li > * { color: black; } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 89 |  | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 90 | .monospaced, code, pre { | 
|  | 91 | font-family: "Courier New", Courier, monospace; | 
|  | 92 | font-size: inherit; | 
|  | 93 | color: navy; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 94 | padding: 0; | 
|  | 95 | margin: 0; | 
|  | 96 | } | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 97 | pre { | 
|  | 98 | white-space: pre-wrap; | 
|  | 99 | } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 100 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 101 | #author { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 102 | color: #527bbd; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 103 | font-weight: bold; | 
|  | 104 | font-size: 1.1em; | 
|  | 105 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 106 | #email { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 107 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 108 | #revnumber, #revdate, #revremark { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 109 | } | 
|  | 110 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 111 | #footer { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 112 | font-size: small; | 
|  | 113 | border-top: 2px solid silver; | 
|  | 114 | padding-top: 0.5em; | 
|  | 115 | margin-top: 4.0em; | 
|  | 116 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 117 | #footer-text { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 118 | float: left; | 
|  | 119 | padding-bottom: 0.5em; | 
|  | 120 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 121 | #footer-badges { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 122 | float: right; | 
|  | 123 | padding-bottom: 0.5em; | 
|  | 124 | } | 
|  | 125 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 126 | #preamble { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 127 | margin-top: 1.5em; | 
|  | 128 | margin-bottom: 1.5em; | 
|  | 129 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 130 | div.imageblock, div.exampleblock, div.verseblock, | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 131 | div.quoteblock, div.literalblock, div.listingblock, div.sidebarblock, | 
|  | 132 | div.admonitionblock { | 
|  | 133 | margin-top: 1.0em; | 
|  | 134 | margin-bottom: 1.5em; | 
|  | 135 | } | 
|  | 136 | div.admonitionblock { | 
|  | 137 | margin-top: 2.0em; | 
|  | 138 | margin-bottom: 2.0em; | 
|  | 139 | margin-right: 10%; | 
|  | 140 | color: #606060; | 
|  | 141 | } | 
|  | 142 |  | 
|  | 143 | div.content { /* Block element content. */ | 
|  | 144 | padding: 0; | 
|  | 145 | } | 
|  | 146 |  | 
|  | 147 | /* Block element titles. */ | 
|  | 148 | div.title, caption.title { | 
|  | 149 | color: #527bbd; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 150 | font-weight: bold; | 
|  | 151 | text-align: left; | 
|  | 152 | margin-top: 1.0em; | 
|  | 153 | margin-bottom: 0.5em; | 
|  | 154 | } | 
|  | 155 | div.title + * { | 
|  | 156 | margin-top: 0; | 
|  | 157 | } | 
|  | 158 |  | 
|  | 159 | td div.title:first-child { | 
|  | 160 | margin-top: 0.0em; | 
|  | 161 | } | 
|  | 162 | div.content div.title:first-child { | 
|  | 163 | margin-top: 0.0em; | 
|  | 164 | } | 
|  | 165 | div.content + div.title { | 
|  | 166 | margin-top: 0.0em; | 
|  | 167 | } | 
|  | 168 |  | 
|  | 169 | div.sidebarblock > div.content { | 
|  | 170 | background: #ffffee; | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 171 | border: 1px solid #dddddd; | 
|  | 172 | border-left: 4px solid #f0f0f0; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 173 | padding: 0.5em; | 
|  | 174 | } | 
|  | 175 |  | 
|  | 176 | div.listingblock > div.content { | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 177 | border: 1px solid #dddddd; | 
|  | 178 | border-left: 5px solid #f0f0f0; | 
|  | 179 | background: #f8f8f8; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 180 | padding: 0.5em; | 
|  | 181 | } | 
|  | 182 |  | 
|  | 183 | div.quoteblock, div.verseblock { | 
|  | 184 | padding-left: 1.0em; | 
|  | 185 | margin-left: 1.0em; | 
|  | 186 | margin-right: 10%; | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 187 | border-left: 5px solid #f0f0f0; | 
|  | 188 | color: #888; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 189 | } | 
|  | 190 |  | 
|  | 191 | div.quoteblock > div.attribution { | 
|  | 192 | padding-top: 0.5em; | 
|  | 193 | text-align: right; | 
|  | 194 | } | 
|  | 195 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 196 | div.verseblock > pre.content { | 
|  | 197 | font-family: inherit; | 
|  | 198 | font-size: inherit; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 199 | } | 
|  | 200 | div.verseblock > div.attribution { | 
|  | 201 | padding-top: 0.75em; | 
|  | 202 | text-align: left; | 
|  | 203 | } | 
|  | 204 | /* DEPRECATED: Pre version 8.2.7 verse style literal block. */ | 
|  | 205 | div.verseblock + div.attribution { | 
|  | 206 | text-align: left; | 
|  | 207 | } | 
|  | 208 |  | 
|  | 209 | div.admonitionblock .icon { | 
|  | 210 | vertical-align: top; | 
|  | 211 | font-size: 1.1em; | 
|  | 212 | font-weight: bold; | 
|  | 213 | text-decoration: underline; | 
|  | 214 | color: #527bbd; | 
|  | 215 | padding-right: 0.5em; | 
|  | 216 | } | 
|  | 217 | div.admonitionblock td.content { | 
|  | 218 | padding-left: 0.5em; | 
|  | 219 | border-left: 3px solid #dddddd; | 
|  | 220 | } | 
|  | 221 |  | 
|  | 222 | div.exampleblock > div.content { | 
|  | 223 | border-left: 3px solid #dddddd; | 
|  | 224 | padding-left: 0.5em; | 
|  | 225 | } | 
|  | 226 |  | 
|  | 227 | div.imageblock div.content { padding-left: 0; } | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 228 | span.image img { border-style: none; vertical-align: text-bottom; } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 229 | a.image:visited { color: white; } | 
|  | 230 |  | 
|  | 231 | dl { | 
|  | 232 | margin-top: 0.8em; | 
|  | 233 | margin-bottom: 0.8em; | 
|  | 234 | } | 
|  | 235 | dt { | 
|  | 236 | margin-top: 0.5em; | 
|  | 237 | margin-bottom: 0; | 
|  | 238 | font-style: normal; | 
|  | 239 | color: navy; | 
|  | 240 | } | 
|  | 241 | dd > *:first-child { | 
|  | 242 | margin-top: 0.1em; | 
|  | 243 | } | 
|  | 244 |  | 
|  | 245 | ul, ol { | 
|  | 246 | list-style-position: outside; | 
|  | 247 | } | 
|  | 248 | ol.arabic { | 
|  | 249 | list-style-type: decimal; | 
|  | 250 | } | 
|  | 251 | ol.loweralpha { | 
|  | 252 | list-style-type: lower-alpha; | 
|  | 253 | } | 
|  | 254 | ol.upperalpha { | 
|  | 255 | list-style-type: upper-alpha; | 
|  | 256 | } | 
|  | 257 | ol.lowerroman { | 
|  | 258 | list-style-type: lower-roman; | 
|  | 259 | } | 
|  | 260 | ol.upperroman { | 
|  | 261 | list-style-type: upper-roman; | 
|  | 262 | } | 
|  | 263 |  | 
|  | 264 | div.compact ul, div.compact ol, | 
|  | 265 | div.compact p, div.compact p, | 
|  | 266 | div.compact div, div.compact div { | 
|  | 267 | margin-top: 0.1em; | 
|  | 268 | margin-bottom: 0.1em; | 
|  | 269 | } | 
|  | 270 |  | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 271 | tfoot { | 
|  | 272 | font-weight: bold; | 
|  | 273 | } | 
|  | 274 | td > div.verse { | 
|  | 275 | white-space: pre; | 
|  | 276 | } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 277 |  | 
|  | 278 | div.hdlist { | 
|  | 279 | margin-top: 0.8em; | 
|  | 280 | margin-bottom: 0.8em; | 
|  | 281 | } | 
|  | 282 | div.hdlist tr { | 
|  | 283 | padding-bottom: 15px; | 
|  | 284 | } | 
|  | 285 | dt.hdlist1.strong, td.hdlist1.strong { | 
|  | 286 | font-weight: bold; | 
|  | 287 | } | 
|  | 288 | td.hdlist1 { | 
|  | 289 | vertical-align: top; | 
|  | 290 | font-style: normal; | 
|  | 291 | padding-right: 0.8em; | 
|  | 292 | color: navy; | 
|  | 293 | } | 
|  | 294 | td.hdlist2 { | 
|  | 295 | vertical-align: top; | 
|  | 296 | } | 
|  | 297 | div.hdlist.compact tr { | 
|  | 298 | margin: 0; | 
|  | 299 | padding-bottom: 0; | 
|  | 300 | } | 
|  | 301 |  | 
|  | 302 | .comment { | 
|  | 303 | background: yellow; | 
|  | 304 | } | 
|  | 305 |  | 
|  | 306 | .footnote, .footnoteref { | 
|  | 307 | font-size: 0.8em; | 
|  | 308 | } | 
|  | 309 |  | 
|  | 310 | span.footnote, span.footnoteref { | 
|  | 311 | vertical-align: super; | 
|  | 312 | } | 
|  | 313 |  | 
|  | 314 | #footnotes { | 
|  | 315 | margin: 20px 0 20px 0; | 
|  | 316 | padding: 7px 0 0 0; | 
|  | 317 | } | 
|  | 318 |  | 
|  | 319 | #footnotes div.footnote { | 
|  | 320 | margin: 0 0 5px 0; | 
|  | 321 | } | 
|  | 322 |  | 
|  | 323 | #footnotes hr { | 
|  | 324 | border: none; | 
|  | 325 | border-top: 1px solid silver; | 
|  | 326 | height: 1px; | 
|  | 327 | text-align: left; | 
|  | 328 | margin-left: 0; | 
|  | 329 | width: 20%; | 
|  | 330 | min-width: 100px; | 
|  | 331 | } | 
|  | 332 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 333 | div.colist td { | 
|  | 334 | padding-right: 0.5em; | 
|  | 335 | padding-bottom: 0.3em; | 
|  | 336 | vertical-align: top; | 
|  | 337 | } | 
|  | 338 | div.colist td img { | 
|  | 339 | margin-top: 0.3em; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 340 | } | 
|  | 341 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 342 | @media print { | 
|  | 343 | #footer-badges { display: none; } | 
|  | 344 | } | 
|  | 345 |  | 
|  | 346 | #toc { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 347 | margin-bottom: 2.5em; | 
|  | 348 | } | 
|  | 349 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 350 | #toctitle { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 351 | color: #527bbd; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 352 | font-size: 1.1em; | 
|  | 353 | font-weight: bold; | 
|  | 354 | margin-top: 1.0em; | 
|  | 355 | margin-bottom: 0.1em; | 
|  | 356 | } | 
|  | 357 |  | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 358 | div.toclevel0, div.toclevel1, div.toclevel2, div.toclevel3, div.toclevel4 { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 359 | margin-top: 0; | 
|  | 360 | margin-bottom: 0; | 
|  | 361 | } | 
|  | 362 | div.toclevel2 { | 
|  | 363 | margin-left: 2em; | 
|  | 364 | font-size: 0.9em; | 
|  | 365 | } | 
|  | 366 | div.toclevel3 { | 
|  | 367 | margin-left: 4em; | 
|  | 368 | font-size: 0.9em; | 
|  | 369 | } | 
|  | 370 | div.toclevel4 { | 
|  | 371 | margin-left: 6em; | 
|  | 372 | font-size: 0.9em; | 
|  | 373 | } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 374 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 375 | span.aqua { color: aqua; } | 
|  | 376 | span.black { color: black; } | 
|  | 377 | span.blue { color: blue; } | 
|  | 378 | span.fuchsia { color: fuchsia; } | 
|  | 379 | span.gray { color: gray; } | 
|  | 380 | span.green { color: green; } | 
|  | 381 | span.lime { color: lime; } | 
|  | 382 | span.maroon { color: maroon; } | 
|  | 383 | span.navy { color: navy; } | 
|  | 384 | span.olive { color: olive; } | 
|  | 385 | span.purple { color: purple; } | 
|  | 386 | span.red { color: red; } | 
|  | 387 | span.silver { color: silver; } | 
|  | 388 | span.teal { color: teal; } | 
|  | 389 | span.white { color: white; } | 
|  | 390 | span.yellow { color: yellow; } | 
|  | 391 |  | 
|  | 392 | span.aqua-background { background: aqua; } | 
|  | 393 | span.black-background { background: black; } | 
|  | 394 | span.blue-background { background: blue; } | 
|  | 395 | span.fuchsia-background { background: fuchsia; } | 
|  | 396 | span.gray-background { background: gray; } | 
|  | 397 | span.green-background { background: green; } | 
|  | 398 | span.lime-background { background: lime; } | 
|  | 399 | span.maroon-background { background: maroon; } | 
|  | 400 | span.navy-background { background: navy; } | 
|  | 401 | span.olive-background { background: olive; } | 
|  | 402 | span.purple-background { background: purple; } | 
|  | 403 | span.red-background { background: red; } | 
|  | 404 | span.silver-background { background: silver; } | 
|  | 405 | span.teal-background { background: teal; } | 
|  | 406 | span.white-background { background: white; } | 
|  | 407 | span.yellow-background { background: yellow; } | 
|  | 408 |  | 
|  | 409 | span.big { font-size: 2em; } | 
|  | 410 | span.small { font-size: 0.6em; } | 
|  | 411 |  | 
|  | 412 | span.underline { text-decoration: underline; } | 
|  | 413 | span.overline { text-decoration: overline; } | 
|  | 414 | span.line-through { text-decoration: line-through; } | 
|  | 415 |  | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 416 | div.unbreakable { page-break-inside: avoid; } | 
|  | 417 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 418 |  | 
|  | 419 | /* | 
|  | 420 | * xhtml11 specific | 
|  | 421 | * | 
|  | 422 | * */ | 
|  | 423 |  | 
|  | 424 | div.tableblock { | 
|  | 425 | margin-top: 1.0em; | 
|  | 426 | margin-bottom: 1.5em; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 427 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 428 | div.tableblock > table { | 
|  | 429 | border: 3px solid #527bbd; | 
|  | 430 | } | 
|  | 431 | thead, p.table.header { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 432 | font-weight: bold; | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 433 | color: #527bbd; | 
|  | 434 | } | 
|  | 435 | p.table { | 
|  | 436 | margin-top: 0; | 
|  | 437 | } | 
|  | 438 | /* Because the table frame attribute is overriden by CSS in most browsers. */ | 
|  | 439 | div.tableblock > table[frame="void"] { | 
|  | 440 | border-style: none; | 
|  | 441 | } | 
|  | 442 | div.tableblock > table[frame="hsides"] { | 
|  | 443 | border-left-style: none; | 
|  | 444 | border-right-style: none; | 
|  | 445 | } | 
|  | 446 | div.tableblock > table[frame="vsides"] { | 
|  | 447 | border-top-style: none; | 
|  | 448 | border-bottom-style: none; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 449 | } | 
|  | 450 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 451 |  | 
|  | 452 | /* | 
|  | 453 | * html5 specific | 
|  | 454 | * | 
|  | 455 | * */ | 
|  | 456 |  | 
|  | 457 | table.tableblock { | 
|  | 458 | margin-top: 1.0em; | 
|  | 459 | margin-bottom: 1.5em; | 
|  | 460 | } | 
|  | 461 | thead, p.tableblock.header { | 
|  | 462 | font-weight: bold; | 
|  | 463 | color: #527bbd; | 
|  | 464 | } | 
|  | 465 | p.tableblock { | 
|  | 466 | margin-top: 0; | 
|  | 467 | } | 
|  | 468 | table.tableblock { | 
|  | 469 | border-width: 3px; | 
|  | 470 | border-spacing: 0px; | 
|  | 471 | border-style: solid; | 
|  | 472 | border-color: #527bbd; | 
|  | 473 | border-collapse: collapse; | 
|  | 474 | } | 
|  | 475 | th.tableblock, td.tableblock { | 
|  | 476 | border-width: 1px; | 
|  | 477 | padding: 4px; | 
|  | 478 | border-style: solid; | 
|  | 479 | border-color: #527bbd; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 480 | } | 
|  | 481 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 482 | table.tableblock.frame-topbot { | 
|  | 483 | border-left-style: hidden; | 
|  | 484 | border-right-style: hidden; | 
|  | 485 | } | 
|  | 486 | table.tableblock.frame-sides { | 
|  | 487 | border-top-style: hidden; | 
|  | 488 | border-bottom-style: hidden; | 
|  | 489 | } | 
|  | 490 | table.tableblock.frame-none { | 
|  | 491 | border-style: hidden; | 
|  | 492 | } | 
|  | 493 |  | 
|  | 494 | th.tableblock.halign-left, td.tableblock.halign-left { | 
|  | 495 | text-align: left; | 
|  | 496 | } | 
|  | 497 | th.tableblock.halign-center, td.tableblock.halign-center { | 
|  | 498 | text-align: center; | 
|  | 499 | } | 
|  | 500 | th.tableblock.halign-right, td.tableblock.halign-right { | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 501 | text-align: right; | 
|  | 502 | } | 
|  | 503 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 504 | th.tableblock.valign-top, td.tableblock.valign-top { | 
|  | 505 | vertical-align: top; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 506 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 507 | th.tableblock.valign-middle, td.tableblock.valign-middle { | 
|  | 508 | vertical-align: middle; | 
|  | 509 | } | 
|  | 510 | th.tableblock.valign-bottom, td.tableblock.valign-bottom { | 
|  | 511 | vertical-align: bottom; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 512 | } | 
|  | 513 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 514 |  | 
|  | 515 | /* | 
|  | 516 | * manpage specific | 
|  | 517 | * | 
|  | 518 | * */ | 
|  | 519 |  | 
|  | 520 | body.manpage h1 { | 
|  | 521 | padding-top: 0.5em; | 
|  | 522 | padding-bottom: 0.5em; | 
|  | 523 | border-top: 2px solid silver; | 
|  | 524 | border-bottom: 2px solid silver; | 
|  | 525 | } | 
|  | 526 | body.manpage h2 { | 
|  | 527 | border-style: none; | 
|  | 528 | } | 
|  | 529 | body.manpage div.sectionbody { | 
|  | 530 | margin-left: 3em; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 531 | } | 
|  | 532 |  | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 533 | @media print { | 
|  | 534 | body.manpage div#toc { display: none; } | 
|  | 535 | } | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 536 |  | 
|  | 537 |  | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 538 | </style> | 
|  | 539 | <script type="text/javascript"> | 
|  | 540 | /*<![CDATA[*/ | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 541 | var asciidoc = { // Namespace. | 
|  | 542 |  | 
|  | 543 | ///////////////////////////////////////////////////////////////////// | 
|  | 544 | // Table Of Contents generator | 
|  | 545 | ///////////////////////////////////////////////////////////////////// | 
|  | 546 |  | 
|  | 547 | /* Author: Mihai Bazon, September 2002 | 
|  | 548 | * http://students.infoiasi.ro/~mishoo | 
|  | 549 | * | 
|  | 550 | * Table Of Content generator | 
|  | 551 | * Version: 0.4 | 
|  | 552 | * | 
|  | 553 | * Feel free to use this script under the terms of the GNU General Public | 
|  | 554 | * License, as long as you do not remove or alter this notice. | 
|  | 555 | */ | 
|  | 556 |  | 
|  | 557 | /* modified by Troy D. Hanson, September 2006. License: GPL */ | 
|  | 558 | /* modified by Stuart Rackham, 2006, 2009. License: GPL */ | 
|  | 559 |  | 
|  | 560 | // toclevels = 1..4. | 
|  | 561 | toc: function (toclevels) { | 
|  | 562 |  | 
|  | 563 | function getText(el) { | 
|  | 564 | var text = ""; | 
|  | 565 | for (var i = el.firstChild; i != null; i = i.nextSibling) { | 
|  | 566 | if (i.nodeType == 3 /* Node.TEXT_NODE */) // IE doesn't speak constants. | 
|  | 567 | text += i.data; | 
|  | 568 | else if (i.firstChild != null) | 
|  | 569 | text += getText(i); | 
|  | 570 | } | 
|  | 571 | return text; | 
|  | 572 | } | 
|  | 573 |  | 
|  | 574 | function TocEntry(el, text, toclevel) { | 
|  | 575 | this.element = el; | 
|  | 576 | this.text = text; | 
|  | 577 | this.toclevel = toclevel; | 
|  | 578 | } | 
|  | 579 |  | 
|  | 580 | function tocEntries(el, toclevels) { | 
|  | 581 | var result = new Array; | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 582 | var re = new RegExp('[hH]([1-'+(toclevels+1)+'])'); | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 583 | // Function that scans the DOM tree for header elements (the DOM2 | 
|  | 584 | // nodeIterator API would be a better technique but not supported by all | 
|  | 585 | // browsers). | 
|  | 586 | var iterate = function (el) { | 
|  | 587 | for (var i = el.firstChild; i != null; i = i.nextSibling) { | 
|  | 588 | if (i.nodeType == 1 /* Node.ELEMENT_NODE */) { | 
|  | 589 | var mo = re.exec(i.tagName); | 
|  | 590 | if (mo && (i.getAttribute("class") || i.getAttribute("className")) != "float") { | 
|  | 591 | result[result.length] = new TocEntry(i, getText(i), mo[1]-1); | 
|  | 592 | } | 
|  | 593 | iterate(i); | 
|  | 594 | } | 
|  | 595 | } | 
|  | 596 | } | 
|  | 597 | iterate(el); | 
|  | 598 | return result; | 
|  | 599 | } | 
|  | 600 |  | 
|  | 601 | var toc = document.getElementById("toc"); | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 602 | if (!toc) { | 
|  | 603 | return; | 
|  | 604 | } | 
|  | 605 |  | 
|  | 606 | // Delete existing TOC entries in case we're reloading the TOC. | 
|  | 607 | var tocEntriesToRemove = []; | 
|  | 608 | var i; | 
|  | 609 | for (i = 0; i < toc.childNodes.length; i++) { | 
|  | 610 | var entry = toc.childNodes[i]; | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 611 | if (entry.nodeName.toLowerCase() == 'div' | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 612 | && entry.getAttribute("class") | 
|  | 613 | && entry.getAttribute("class").match(/^toclevel/)) | 
|  | 614 | tocEntriesToRemove.push(entry); | 
|  | 615 | } | 
|  | 616 | for (i = 0; i < tocEntriesToRemove.length; i++) { | 
|  | 617 | toc.removeChild(tocEntriesToRemove[i]); | 
|  | 618 | } | 
|  | 619 |  | 
|  | 620 | // Rebuild TOC entries. | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 621 | var entries = tocEntries(document.getElementById("content"), toclevels); | 
|  | 622 | for (var i = 0; i < entries.length; ++i) { | 
|  | 623 | var entry = entries[i]; | 
|  | 624 | if (entry.element.id == "") | 
|  | 625 | entry.element.id = "_toc_" + i; | 
|  | 626 | var a = document.createElement("a"); | 
|  | 627 | a.href = "#" + entry.element.id; | 
|  | 628 | a.appendChild(document.createTextNode(entry.text)); | 
|  | 629 | var div = document.createElement("div"); | 
|  | 630 | div.appendChild(a); | 
|  | 631 | div.className = "toclevel" + entry.toclevel; | 
|  | 632 | toc.appendChild(div); | 
|  | 633 | } | 
|  | 634 | if (entries.length == 0) | 
|  | 635 | toc.parentNode.removeChild(toc); | 
|  | 636 | }, | 
|  | 637 |  | 
|  | 638 |  | 
|  | 639 | ///////////////////////////////////////////////////////////////////// | 
|  | 640 | // Footnotes generator | 
|  | 641 | ///////////////////////////////////////////////////////////////////// | 
|  | 642 |  | 
|  | 643 | /* Based on footnote generation code from: | 
|  | 644 | * http://www.brandspankingnew.net/archive/2005/07/format_footnote.html | 
|  | 645 | */ | 
|  | 646 |  | 
|  | 647 | footnotes: function () { | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 648 | // Delete existing footnote entries in case we're reloading the footnodes. | 
|  | 649 | var i; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 650 | var noteholder = document.getElementById("footnotes"); | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 651 | if (!noteholder) { | 
|  | 652 | return; | 
|  | 653 | } | 
|  | 654 | var entriesToRemove = []; | 
|  | 655 | for (i = 0; i < noteholder.childNodes.length; i++) { | 
|  | 656 | var entry = noteholder.childNodes[i]; | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 657 | if (entry.nodeName.toLowerCase() == 'div' && entry.getAttribute("class") == "footnote") | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 658 | entriesToRemove.push(entry); | 
|  | 659 | } | 
|  | 660 | for (i = 0; i < entriesToRemove.length; i++) { | 
|  | 661 | noteholder.removeChild(entriesToRemove[i]); | 
|  | 662 | } | 
|  | 663 |  | 
|  | 664 | // Rebuild footnote entries. | 
|  | 665 | var cont = document.getElementById("content"); | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 666 | var spans = cont.getElementsByTagName("span"); | 
|  | 667 | var refs = {}; | 
|  | 668 | var n = 0; | 
|  | 669 | for (i=0; i<spans.length; i++) { | 
|  | 670 | if (spans[i].className == "footnote") { | 
|  | 671 | n++; | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 672 | var note = spans[i].getAttribute("data-note"); | 
|  | 673 | if (!note) { | 
|  | 674 | // Use [\s\S] in place of . so multi-line matches work. | 
|  | 675 | // Because JavaScript has no s (dotall) regex flag. | 
|  | 676 | note = spans[i].innerHTML.match(/\s*\[([\s\S]*)]\s*/)[1]; | 
|  | 677 | spans[i].innerHTML = | 
|  | 678 | "[<a id='_footnoteref_" + n + "' href='#_footnote_" + n + | 
|  | 679 | "' title='View footnote' class='footnote'>" + n + "</a>]"; | 
|  | 680 | spans[i].setAttribute("data-note", note); | 
|  | 681 | } | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 682 | noteholder.innerHTML += | 
|  | 683 | "<div class='footnote' id='_footnote_" + n + "'>" + | 
|  | 684 | "<a href='#_footnoteref_" + n + "' title='Return to text'>" + | 
|  | 685 | n + "</a>. " + note + "</div>"; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 686 | var id =spans[i].getAttribute("id"); | 
|  | 687 | if (id != null) refs["#"+id] = n; | 
|  | 688 | } | 
|  | 689 | } | 
|  | 690 | if (n == 0) | 
|  | 691 | noteholder.parentNode.removeChild(noteholder); | 
|  | 692 | else { | 
|  | 693 | // Process footnoterefs. | 
|  | 694 | for (i=0; i<spans.length; i++) { | 
|  | 695 | if (spans[i].className == "footnoteref") { | 
|  | 696 | var href = spans[i].getElementsByTagName("a")[0].getAttribute("href"); | 
|  | 697 | href = href.match(/#.*/)[0]; // Because IE return full URL. | 
|  | 698 | n = refs[href]; | 
|  | 699 | spans[i].innerHTML = | 
|  | 700 | "[<a href='#_footnote_" + n + | 
|  | 701 | "' title='View footnote' class='footnote'>" + n + "</a>]"; | 
|  | 702 | } | 
|  | 703 | } | 
|  | 704 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 705 | }, | 
|  | 706 |  | 
|  | 707 | install: function(toclevels) { | 
|  | 708 | var timerId; | 
|  | 709 |  | 
|  | 710 | function reinstall() { | 
|  | 711 | asciidoc.footnotes(); | 
|  | 712 | if (toclevels) { | 
|  | 713 | asciidoc.toc(toclevels); | 
|  | 714 | } | 
|  | 715 | } | 
|  | 716 |  | 
|  | 717 | function reinstallAndRemoveTimer() { | 
|  | 718 | clearInterval(timerId); | 
|  | 719 | reinstall(); | 
|  | 720 | } | 
|  | 721 |  | 
|  | 722 | timerId = setInterval(reinstall, 500); | 
|  | 723 | if (document.addEventListener) | 
|  | 724 | document.addEventListener("DOMContentLoaded", reinstallAndRemoveTimer, false); | 
|  | 725 | else | 
|  | 726 | window.onload = reinstallAndRemoveTimer; | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 727 | } | 
|  | 728 |  | 
|  | 729 | } | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 730 | asciidoc.install(); | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 731 | /*]]>*/ | 
|  | 732 | </script> | 
|  | 733 | </head> | 
| Junio C Hamano | 9d97115 | 2012-12-19 00:43:11 | [diff] [blame] | 734 | <body class="article"> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 735 | <div id="header"> | 
|  | 736 | <h1>How to revert a faulty merge</h1> | 
|  | 737 | </div> | 
|  | 738 | <div id="content"> | 
|  | 739 | <div id="preamble"> | 
|  | 740 | <div class="sectionbody"> | 
|  | 741 | <div class="paragraph"><p>Alan <<a href="mailto:alan@clueserver.org">alan@clueserver.org</a>> said:</p></div> | 
|  | 742 | <div class="literalblock"> | 
|  | 743 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 744 | <pre><code>I have a master branch. We have a branch off of that that some | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 745 | developers are doing work on. They claim it is ready. We merge it | 
|  | 746 | into the master branch. It breaks something so we revert the merge. | 
|  | 747 | They make changes to the code. they get it to a point where they say | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 748 | it is ok and we merge again.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 749 | </div></div> | 
|  | 750 | <div class="literalblock"> | 
|  | 751 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 752 | <pre><code>When examined, we find that code changes made before the revert are | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 753 | not in the master branch, but code changes after are in the master | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 754 | branch.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 755 | </div></div> | 
|  | 756 | <div class="paragraph"><p>and asked for help recovering from this situation.</p></div> | 
|  | 757 | <div class="paragraph"><p>The history immediately after the "revert of the merge" would look like | 
|  | 758 | this:</p></div> | 
|  | 759 | <div class="literalblock"> | 
|  | 760 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 761 | <pre><code>---o---o---o---M---x---x---W | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 762 | / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 763 | ---A---B</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 764 | </div></div> | 
|  | 765 | <div class="paragraph"><p>where A and B are on the side development that was not so good, M is the | 
|  | 766 | merge that brings these premature changes into the mainline, x are changes | 
|  | 767 | unrelated to what the side branch did and already made on the mainline, | 
|  | 768 | and W is the "revert of the merge M" (doesn’t W look M upside down?). | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 769 | IOW, <code>"diff W^..W"</code> is similar to <code>"diff -R M^..M"</code>.</p></div> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 770 | <div class="paragraph"><p>Such a "revert" of a merge can be made with:</p></div> | 
|  | 771 | <div class="literalblock"> | 
|  | 772 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 773 | <pre><code>$ git revert -m 1 M</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 774 | </div></div> | 
|  | 775 | <div class="paragraph"><p>After the developers of the side branch fix their mistakes, the history | 
|  | 776 | may look like this:</p></div> | 
|  | 777 | <div class="literalblock"> | 
|  | 778 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 779 | <pre><code>---o---o---o---M---x---x---W---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 780 | / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 781 | ---A---B-------------------C---D</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 782 | </div></div> | 
|  | 783 | <div class="paragraph"><p>where C and D are to fix what was broken in A and B, and you may already | 
|  | 784 | have some other changes on the mainline after W.</p></div> | 
|  | 785 | <div class="paragraph"><p>If you merge the updated side branch (with D at its tip), none of the | 
| Junio C Hamano | d75148a | 2014-04-08 19:48:38 | [diff] [blame] | 786 | changes made in A or B will be in the result, because they were reverted | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 787 | by W. That is what Alan saw.</p></div> | 
|  | 788 | <div class="paragraph"><p>Linus explains the situation:</p></div> | 
|  | 789 | <div class="literalblock"> | 
|  | 790 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 791 | <pre><code>Reverting a regular commit just effectively undoes what that commit | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 792 | did, and is fairly straightforward. But reverting a merge commit also | 
|  | 793 | undoes the _data_ that the commit changed, but it does absolutely | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 794 | nothing to the effects on _history_ that the merge had.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 795 | </div></div> | 
|  | 796 | <div class="literalblock"> | 
|  | 797 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 798 | <pre><code>So the merge will still exist, and it will still be seen as joining | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 799 | the two branches together, and future merges will see that merge as | 
|  | 800 | the last shared state - and the revert that reverted the merge brought | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 801 | in will not affect that at all.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 802 | </div></div> | 
|  | 803 | <div class="literalblock"> | 
|  | 804 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 805 | <pre><code>So a "revert" undoes the data changes, but it's very much _not_ an | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 806 | "undo" in the sense that it doesn't undo the effects of a commit on | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 807 | the repository history.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 808 | </div></div> | 
|  | 809 | <div class="literalblock"> | 
|  | 810 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 811 | <pre><code>So if you think of "revert" as "undo", then you're going to always | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 812 | miss this part of reverts. Yes, it undoes the data, but no, it doesn't | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 813 | undo history.</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 814 | </div></div> | 
|  | 815 | <div class="paragraph"><p>In such a situation, you would want to first revert the previous revert, | 
|  | 816 | which would make the history look like this:</p></div> | 
|  | 817 | <div class="literalblock"> | 
|  | 818 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 819 | <pre><code>---o---o---o---M---x---x---W---x---Y | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 820 | / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 821 | ---A---B-------------------C---D</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 822 | </div></div> | 
|  | 823 | <div class="paragraph"><p>where Y is the revert of W. Such a "revert of the revert" can be done | 
|  | 824 | with:</p></div> | 
|  | 825 | <div class="literalblock"> | 
|  | 826 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 827 | <pre><code>$ git revert W</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 828 | </div></div> | 
|  | 829 | <div class="paragraph"><p>This history would (ignoring possible conflicts between what W and W..Y | 
| Junio C Hamano | d75148a | 2014-04-08 19:48:38 | [diff] [blame] | 830 | changed) be equivalent to not having W or Y at all in the history:</p></div> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 831 | <div class="literalblock"> | 
|  | 832 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 833 | <pre><code>---o---o---o---M---x---x-------x---- | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 834 | / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 835 | ---A---B-------------------C---D</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 836 | </div></div> | 
|  | 837 | <div class="paragraph"><p>and merging the side branch again will not have conflict arising from an | 
|  | 838 | earlier revert and revert of the revert.</p></div> | 
|  | 839 | <div class="literalblock"> | 
|  | 840 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 841 | <pre><code>---o---o---o---M---x---x-------x-------* | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 842 | / / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 843 | ---A---B-------------------C---D</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 844 | </div></div> | 
|  | 845 | <div class="paragraph"><p>Of course the changes made in C and D still can conflict with what was | 
|  | 846 | done by any of the x, but that is just a normal merge conflict.</p></div> | 
|  | 847 | <div class="paragraph"><p>On the other hand, if the developers of the side branch discarded their | 
|  | 848 | faulty A and B, and redone the changes on top of the updated mainline | 
|  | 849 | after the revert, the history would have looked like this:</p></div> | 
|  | 850 | <div class="literalblock"> | 
|  | 851 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 852 | <pre><code>---o---o---o---M---x---x---W---x---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 853 | / \ | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 854 | ---A---B A'--B'--C'</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 855 | </div></div> | 
|  | 856 | <div class="paragraph"><p>If you reverted the revert in such a case as in the previous example:</p></div> | 
|  | 857 | <div class="literalblock"> | 
|  | 858 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 859 | <pre><code>---o---o---o---M---x---x---W---x---x---Y---* | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 860 | / \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 861 | ---A---B A'--B'--C'</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 862 | </div></div> | 
|  | 863 | <div class="paragraph"><p>where Y is the revert of W, A' and B' are rerolled A and B, and there may | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 864 | also be a further fix-up C' on the side branch. <code>"diff Y^..Y"</code> is similar | 
|  | 865 | to <code>"diff -R W^..W"</code> (which in turn means it is similar to <code>"diff M^..M"</code>), | 
|  | 866 | and <code>"diff A'^..C'"</code> by definition would be similar but different from that, | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 867 | because it is a rerolled series of the earlier change. There will be a | 
|  | 868 | lot of overlapping changes that result in conflicts. So do not do "revert | 
|  | 869 | of revert" blindly without thinking..</p></div> | 
|  | 870 | <div class="literalblock"> | 
|  | 871 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 872 | <pre><code>---o---o---o---M---x---x---W---x---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 873 | / \ | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 874 | ---A---B A'--B'--C'</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 875 | </div></div> | 
|  | 876 | <div class="paragraph"><p>In the history with rebased side branch, W (and M) are behind the merge | 
|  | 877 | base of the updated branch and the tip of the mainline, and they should | 
|  | 878 | merge without the past faulty merge and its revert getting in the way.</p></div> | 
|  | 879 | <div class="paragraph"><p>To recap, these are two very different scenarios, and they want two very | 
|  | 880 | different resolution strategies:</p></div> | 
|  | 881 | <div class="ulist"><ul> | 
|  | 882 | <li> | 
|  | 883 | <p> | 
|  | 884 | If the faulty side branch was fixed by adding corrections on top, then | 
|  | 885 | doing a revert of the previous revert would be the right thing to do. | 
|  | 886 | </p> | 
|  | 887 | </li> | 
|  | 888 | <li> | 
|  | 889 | <p> | 
|  | 890 | If the faulty side branch whose effects were discarded by an earlier | 
|  | 891 | revert of a merge was rebuilt from scratch (i.e. rebasing and fixing, | 
|  | 892 | as you seem to have interpreted), then re-merging the result without | 
|  | 893 | doing anything else fancy would be the right thing to do. | 
|  | 894 | (See the ADDENDUM below for how to rebuild a branch from scratch | 
|  | 895 | without changing its original branching-off point.) | 
|  | 896 | </p> | 
|  | 897 | </li> | 
|  | 898 | </ul></div> | 
|  | 899 | <div class="paragraph"><p>However, there are things to keep in mind when reverting a merge (and | 
|  | 900 | reverting such a revert).</p></div> | 
|  | 901 | <div class="paragraph"><p>For example, think about what reverting a merge (and then reverting the | 
|  | 902 | revert) does to bisectability. Ignore the fact that the revert of a revert | 
|  | 903 | is undoing it - just think of it as a "single commit that does a lot". | 
|  | 904 | Because that is what it does.</p></div> | 
|  | 905 | <div class="paragraph"><p>When you have a problem you are chasing down, and you hit a "revert this | 
|  | 906 | merge", what you’re hitting is essentially a single commit that contains | 
|  | 907 | all the changes (but obviously in reverse) of all the commits that got | 
|  | 908 | merged. So it’s debugging hell, because now you don’t have lots of small | 
|  | 909 | changes that you can try to pinpoint which <em>part</em> of it changes.</p></div> | 
|  | 910 | <div class="paragraph"><p>But does it all work? Sure it does. You can revert a merge, and from a | 
| Junio C Hamano | 076ffcc | 2013-02-06 05:13:21 | [diff] [blame] | 911 | purely technical angle, Git did it very naturally and had no real | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 912 | troubles. It just considered it a change from "state before merge" to | 
|  | 913 | "state after merge", and that was it. Nothing complicated, nothing odd, | 
|  | 914 | nothing really dangerous. Git will do it without even thinking about it.</p></div> | 
|  | 915 | <div class="paragraph"><p>So from a technical angle, there’s nothing wrong with reverting a merge, | 
|  | 916 | but from a workflow angle it’s something that you generally should try to | 
|  | 917 | avoid.</p></div> | 
|  | 918 | <div class="paragraph"><p>If at all possible, for example, if you find a problem that got merged | 
|  | 919 | into the main tree, rather than revert the merge, try <em>really</em> hard to | 
|  | 920 | bisect the problem down into the branch you merged, and just fix it, or | 
|  | 921 | try to revert the individual commit that caused it.</p></div> | 
|  | 922 | <div class="paragraph"><p>Yes, it’s more complex, and no, it’s not always going to work (sometimes | 
|  | 923 | the answer is: "oops, I really shouldn’t have merged it, because it wasn’t | 
|  | 924 | ready yet, and I really need to undo <em>all</em> of the merge"). So then you | 
|  | 925 | really should revert the merge, but when you want to re-do the merge, you | 
|  | 926 | now need to do it by reverting the revert.</p></div> | 
|  | 927 | <div class="paragraph"><p>ADDENDUM</p></div> | 
|  | 928 | <div class="paragraph"><p>Sometimes you have to rewrite one of a topic branch’s commits <strong>and</strong> you can’t | 
|  | 929 | change the topic’s branching-off point. Consider the following situation:</p></div> | 
|  | 930 | <div class="literalblock"> | 
|  | 931 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 932 | <pre><code>P---o---o---M---x---x---W---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 933 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 934 | A---B---C</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 935 | </div></div> | 
|  | 936 | <div class="paragraph"><p>where commit W reverted commit M because it turned out that commit B was wrong | 
|  | 937 | and needs to be rewritten, but you need the rewritten topic to still branch | 
|  | 938 | from commit P (perhaps P is a branching-off point for yet another branch, and | 
|  | 939 | you want be able to merge the topic into both branches).</p></div> | 
|  | 940 | <div class="paragraph"><p>The natural thing to do in this case is to checkout the A-B-C branch and use | 
|  | 941 | "rebase -i P" to change commit B. However this does not rewrite commit A, | 
|  | 942 | because "rebase -i" by default fast-forwards over any initial commits selected | 
|  | 943 | with the "pick" command. So you end up with this:</p></div> | 
|  | 944 | <div class="literalblock"> | 
|  | 945 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 946 | <pre><code>P---o---o---M---x---x---W---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 947 | \ / | 
|  | 948 | A---B---C <-- old branch | 
|  | 949 | \ | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 950 | B'---C' <-- naively rewritten branch</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 951 | </div></div> | 
|  | 952 | <div class="paragraph"><p>To merge A-B'-C' into the mainline branch you would still have to first revert | 
|  | 953 | commit W in order to pick up the changes in A, but then it’s likely that the | 
|  | 954 | changes in B' will conflict with the original B changes re-introduced by the | 
|  | 955 | reversion of W.</p></div> | 
|  | 956 | <div class="paragraph"><p>However, you can avoid these problems if you recreate the entire branch, | 
|  | 957 | including commit A:</p></div> | 
|  | 958 | <div class="literalblock"> | 
|  | 959 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 960 | <pre><code> A'---B'---C' <-- completely rewritten branch | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 961 | / | 
|  | 962 | P---o---o---M---x---x---W---x | 
|  | 963 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 964 | A---B---C</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 965 | </div></div> | 
|  | 966 | <div class="paragraph"><p>You can merge A'-B'-C' into the mainline branch without worrying about first | 
|  | 967 | reverting W. Mainline’s history would look like this:</p></div> | 
|  | 968 | <div class="literalblock"> | 
|  | 969 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 970 | <pre><code> A'---B'---C'------------------ | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 971 | / \ | 
|  | 972 | P---o---o---M---x---x---W---x---M2 | 
|  | 973 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 974 | A---B---C</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 975 | </div></div> | 
|  | 976 | <div class="paragraph"><p>But if you don’t actually need to change commit A, then you need some way to | 
|  | 977 | recreate it as a new commit with the same changes in it. The rebase command’s | 
|  | 978 | --no-ff option provides a way to do this:</p></div> | 
|  | 979 | <div class="literalblock"> | 
|  | 980 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 981 | <pre><code>$ git rebase [-i] --no-ff P</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 982 | </div></div> | 
|  | 983 | <div class="paragraph"><p>The --no-ff option creates a new branch A'-B'-C' with all-new commits (all the | 
|  | 984 | SHA IDs will be different) even if in the interactive case you only actually | 
|  | 985 | modify commit B. You can then merge this new branch directly into the mainline | 
|  | 986 | branch and be sure you’ll get all of the branch’s changes.</p></div> | 
|  | 987 | <div class="paragraph"><p>You can also use --no-ff in cases where you just add extra commits to the topic | 
|  | 988 | to fix it up. Let’s revisit the situation discussed at the start of this howto:</p></div> | 
|  | 989 | <div class="literalblock"> | 
|  | 990 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 991 | <pre><code>P---o---o---M---x---x---W---x | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 992 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 993 | A---B---C----------------D---E <-- fixed-up topic branch</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 994 | </div></div> | 
|  | 995 | <div class="paragraph"><p>At this point, you can use --no-ff to recreate the topic branch:</p></div> | 
|  | 996 | <div class="literalblock"> | 
|  | 997 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 998 | <pre><code>$ git checkout E | 
|  | 999 | $ git rebase --no-ff P</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1000 | </div></div> | 
|  | 1001 | <div class="paragraph"><p>yielding</p></div> | 
|  | 1002 | <div class="literalblock"> | 
|  | 1003 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 1004 | <pre><code> A'---B'---C'------------D'---E' <-- recreated topic branch | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1005 | / | 
|  | 1006 | P---o---o---M---x---x---W---x | 
|  | 1007 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 1008 | A---B---C----------------D---E</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1009 | </div></div> | 
|  | 1010 | <div class="paragraph"><p>You can merge the recreated branch into the mainline without reverting commit W, | 
|  | 1011 | and mainline’s history will look like this:</p></div> | 
|  | 1012 | <div class="literalblock"> | 
|  | 1013 | <div class="content"> | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 1014 | <pre><code> A'---B'---C'------------D'---E' | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1015 | / \ | 
|  | 1016 | P---o---o---M---x---x---W---x---M2 | 
|  | 1017 | \ / | 
| Junio C Hamano | c14e6ad | 2014-10-31 20:25:53 | [diff] [blame] | 1018 | A---B---C</code></pre> | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1019 | </div></div> | 
|  | 1020 | </div> | 
|  | 1021 | </div> | 
|  | 1022 | </div> | 
|  | 1023 | <div id="footnotes"><hr /></div> | 
|  | 1024 | <div id="footer"> | 
|  | 1025 | <div id="footer-text"> | 
| Junio C Hamano | 2ef0ba3 | 2018-01-26 23:13:53 | [diff] [blame] | 1026 | Last updated | 
| Junio C Hamano | c756926 | 2018-03-20 21:57:48 | [diff] [blame^] | 1027 | 2018-03-20 14:55:29 PDT | 
| Junio C Hamano | f2b7494 | 2012-11-20 21:06:26 | [diff] [blame] | 1028 | </div> | 
|  | 1029 | </div> | 
|  | 1030 | </body> | 
|  | 1031 | </html> |