]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/mercurial.mdwn
thoughts
[git.ikiwiki.info.git] / doc / todo / mercurial.mdwn
1 * Need to get post commit hook working (or an example of how to use it.)
2   * See below. --[[bma]]
3 * rcs_notify is not implemented
4 * Is the code sufficiently robust? It just warns when mercurial fails.
5 * When rcs_commit is called with a $user that is an openid, it will be
6   passed through to mercurial -u. Will mercurial choke on this?
7  * Nope. Mercurial doesn't expect any particular format for the username, 
8    though "Name <address@domain>" is standard. --[[bma]]
9 * The way `-u $user` is passed to `hg commit`, there's no way to tell
10   if a given commit came in over the web or was done directly. So
11   rcs_recentchanges hardcodes 'committype => "mercurial"'. See the monotone
12   backend for an example of one that does this right.
13 * The rcs_commit implementation seems not to notice if the file has been
14   changed since a web edit started. Unlike all the other frontends, which
15   use the rcstoken to detect if the web commit started editing an earlier
16   version of the file, and if so, merge the two sets of changes together.
17   It seems that with the current mercurial commit code, it will always
18   blindly overwrite the current file with the web edited version, losing
19   any other changes.
21 Posthook: in `$srcdir/.hg/hgrc`, I have the following
23     [hooks]
24     incoming.update = hg up
25     update.ikiwiki = ikiwiki --setup /path/to/ikiwiki.setup --refresh
27 This should update the working directory and run ikiwiki every time a change is recorded (someone who knows mercurial better than I do may be able to suggest a better way, but this works for me.)
29 > Try running it with --post-commit instead of --refresh. That should
30 > work better, handling both the case where the edit was made via the web
31 > and then committed, and the case where a commit was made directly.
32 > It can deadlock if the post-commit hook runs with --refresh in the
33 > former case. --[[Joey]]