tests/test-hgweb-descend-empties.out
author Henrik Stuart <henrik.stuart@edlund.dk>
Sat, 23 May 2009 17:02:49 +0200
changeset 8562 e3495c399006
parent 8372 783c8c78db86
child 8936 1de6e7e1bb9f
permissions -rw-r--r--
named branches: server branchmap wire protocol support (issue736) The repository command, 'branchmap', returns a dictionary, branchname -> [branchheads], and will be implemented for localrepo, httprepo and sshrepo. The following wire format is used for returning data: branchname1 branch1head2 branch1head2 ... branchname2 ... ... Branch names are URL encoded to escape white space, and branch heads are sent as hex encoded node ids. All branches and all their heads are sent. The background and motivation for this command is the desire for a richer named branch semantics when pushing changesets. The details are explained in the original proposal which is included below. 1. BACKGROUND The algorithm currently implemented in Mercurial only considers the graph theoretical heads when determining whether new heads are created, rather than using the branch heads as a count (the algorithm considers a branch head effectively closed when it is merged into another branch or a new named branch is started from that point onward). Our particular problem with the algorithm is that we'd like to see the following case working without forcing a push: Upsteam has: (0:dev) ---- (1:dev) \ `--- (2:stable) Someone merges stable into dev: (0:dev) ---- (1:dev) ------(3:dev) \ / `--- (2:stable) --------´ This can be pushed without --force (as it should). Now someone else does some coding on stable (a bug fix, say): (0:dev) ---- (1:dev) ------(3:dev) \ / `--- (2:stable) ---------´---------(4:stable) This time we need --force to push. We allow this to be pushed without using --force by getting all the remote branch heads (by extending the wire protocol with a new function). We would, furthermore, also prefer if it is impossible to push a new branch without --force (or a later --newbranch option so --force isn't shoe-horned into too many disparate functions, if need be), except of course in the case where the remote repository is empty. This is what our patches accomplish. 2. ALTERNATIVES We have, of course, considered some alternatives to reconstructing enough information to decide whether we are creating new remote branch heads, before we added the new wire protocol command. 2.1. LOOKUP ON REMOTE The main alternative is to use the information from remote.heads() and remote.lookup() to try to reconstruct enough graph information to decide whether we are creating new heads. This is not adequate as illustrated below. Remember that each lookup is typically a request-response pair over SSH or HTTP(S). If we have a simple repository at the remote end like this: (0:dev) ---- (1:dev) ---- (3:stable) \ `--- (2:dev) then remote.heads() will yield [2, 3]. Assume we have nodes [0, 1, 2] locally and want to create a new node, 4:dev, as a descendant from (1:dev), which should be OK as 1:dev is a branch head. If we do remote.lookup('dev') we will get [2]. Thus, we can get information about whether a branch exists on the remote server or not, but this does not solve our problem of figuring out whether we are creating new heads or not. Pushing 4:dev ought to be OK, since after the push, we still only have two heads on branch a. Using remote.lookup() and remote.heads() is thus not adequate to consistently decide whether we are creating new remote heads (e.g. in this situation the latter would never return 1:dev). 2.2. USING INCOMING TO RECONSTRUCT THE GRAPH An alternative would be to use information equivalent to hg incoming to get the full remote graph in addition to the local graph. To do this, we would have to get a changegroup(subset) bundle representing the remote end (which may be a substantial amount of data), getting the branch heads from an instantiated bundlerepository, deleting the bundle, and finally, we can compute the prepush logic. While this is backwards compatible, it will cause a possibly substantial slowdown of the push command as it first needs to pull in all changes. 3. FURTHER ARGUMENTS IN FAVOUR OF THE BRANCHMAP WIRE-PROTOCOL EXTENSION Currently, the commands incoming and pull, work based on the tip of a given branch if used with "-r branchname", making it hard to get all revisions of a certain branch only (if it has multiple heads). This can be solved by requesting the remote's branchheads and letting the revisions to be used with the command be these heads. This can be done by extending the commands with a new option, e.g.: hg pull -b branchname which will be turned into the equivalent of: hg pull -r branchhead1 -r branchhead2 -r branchhead3 We have a simple follow-up patch that can do this ready as well (although not submitted yet as it is pending the acceptance of the branch patch). 4. WRAP-UP We generally find that the branchmap wire protocol extension can provide better named branch support to Mercurial. Currently, some things, like the initial push scenario in this mail, are fairly counter-intuitive, and the more often you have to force push, the more it is likely you will get a lot of spurious and unnecessary merge nodes. Also, restricting incoming and pull to all changes on a branch rather than changes on the tip-most head would be a sensible extension to making named branches a first class citizen in Mercurial. Currently, named branches sometimes feel like a late-coming unwanted step-child. We have run it in a production environment for a while, with fewer multiple heads occurring in our repositories and fewer confused users as a result. Also, it fixes the long-standing issue 736. Co-contributor: Sune Foldager <cryo@cyanite.org>
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     1
% Set up the repo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     2
adding a1/a2/a3/a4/foo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     3
adding b1/b2/b3/b4/foo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     4
adding b1/b2/c3/c4/foo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     5
adding d1/d2/d3/d4/foo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     6
adding d1/d2/foo
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     7
% manifest with descending
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     8
200 Script output follows
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
     9
7725
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    10
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    11
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en-US">
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    12
<head>
7725
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    13
<link rel="icon" href="/static/hgicon.png" type="image/png" />
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    14
<meta name="robots" content="index, nofollow" />
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    15
<link rel="stylesheet" href="/static/style-paper.css" type="text/css" />
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    16
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    17
<title>test: 9087c84a0f5d /</title>
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    18
</head>
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    19
<body>
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    20
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    21
<div class="container">
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    22
<div class="menu">
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    23
<div class="logo">
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    24
<a href="http://www.selenic.com/mercurial/">
7725
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    25
<img src="/static/hglogo.png" alt="mercurial" /></a>
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
    26
</div>
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    27
<ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    28
<li><a href="/shortlog/9087c84a0f5d">log</a></li>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    29
<li><a href="/graph/9087c84a0f5d">graph</a></li>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    30
<li><a href="/tags">tags</a></li>
8353
6058d291abdf paper/coal: add branches page
Sune Foldager <cryo@cyanite.org>
parents: 7725
diff changeset
    31
<li><a href="/branches">branches</a></li>
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    32
</ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    33
<ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    34
<li><a href="/rev/9087c84a0f5d">changeset</a></li>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    35
<li class="active">browse</li>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    36
</ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    37
<ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    38
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    39
</ul>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    40
</div>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    41
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    42
<div class="main">
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    43
<h2><a href="/">test</a></h2>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    44
<h3>directory / @ 0:9087c84a0f5d <span class="tag">tip</span> </h3>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    45
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    46
<form class="search" action="/log">
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    47
7725
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    48
<p><input name="rev" id="search1" type="text" size="30" /></p>
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    49
<div id="hint">find changesets by author, revision,
6a888d491eaf coal/paper: switch to XHTML 1.1
Benoit Allard <benoit@aeteurope.nl>
parents: 7347
diff changeset
    50
files, or words in the commit message</div>
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    51
</form>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    52
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    53
<table class="bigtable">
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    54
<tr>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    55
  <th class="name">name</th>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    56
  <th class="size">size</th>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    57
  <th class="permissions">permissions</th>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    58
</tr>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    59
<tr class="fileline parity0">
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    60
  <td class="name"><a href="/file/9087c84a0f5d/">[up]</a></td>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    61
  <td class="size"></td>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    62
  <td class="permissions">drwxr-xr-x</td>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
    63
</tr>
8372
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    64
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    65
<tr class="fileline parity1">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    66
<td class="name">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    67
<a href="/file/9087c84a0f5d/a1">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    68
<img src="/static/coal-folder.png" alt="dir."/> a1/
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    69
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    70
<a href="/file/9087c84a0f5d/a1/a2/a3/a4">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    71
a2/a3/a4
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    72
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    73
</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    74
<td class="size"></td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    75
<td class="permissions">drwxr-xr-x</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    76
</tr>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    77
<tr class="fileline parity0">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    78
<td class="name">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    79
<a href="/file/9087c84a0f5d/b1">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    80
<img src="/static/coal-folder.png" alt="dir."/> b1/
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    81
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    82
<a href="/file/9087c84a0f5d/b1/b2">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    83
b2
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    84
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    85
</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    86
<td class="size"></td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    87
<td class="permissions">drwxr-xr-x</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    88
</tr>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    89
<tr class="fileline parity1">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    90
<td class="name">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    91
<a href="/file/9087c84a0f5d/d1">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    92
<img src="/static/coal-folder.png" alt="dir."/> d1/
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    93
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    94
<a href="/file/9087c84a0f5d/d1/d2">
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    95
d2
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    96
</a>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    97
</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    98
<td class="size"></td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
    99
<td class="permissions">drwxr-xr-x</td>
783c8c78db86 paper: break long lines in map file
Martin Geisler <mg@lazybytes.net>
parents: 8353
diff changeset
   100
</tr>
7339
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
   101
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
   102
</table>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
   103
</div>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
   104
</div>
d11473955d30 hgweb: update tests for paper style
Matt Mackall <mpm@selenic.com>
parents: 7305
diff changeset
   105
7305
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
   106
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
   107
</body>
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
   108
</html>
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
   109
c21d236ca897 hgweb: descend empty directories in web view
Ry4an Brase <ry4an-hg@ry4an.org>
parents:
diff changeset
   110
% ERRORS ENCOUNTERED