Mercurial > hg
changeset 29462:71ed5a3ef8a9
chgserver: document why we don't merge mtimehash and confighash
People may get confused about chg's mtimehash and confighash design: why two
hashes instead of just one. This patch adds text addressing the concern.
| author | Jun Wu <quark@fb.com> |
|---|---|
| date | Wed, 29 Jun 2016 23:53:20 +0100 |
| parents | 7d88fde2309f |
| children | 4cf0542dcbe7 |
| files | hgext/chgserver.py |
| diffstat | 1 files changed, 4 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/hgext/chgserver.py Thu Jun 30 10:31:50 2016 +0100 +++ b/hgext/chgserver.py Wed Jun 29 23:53:20 2016 +0100 @@ -154,6 +154,10 @@ for chgserver, it is designed that once mtimehash changes, the server is considered outdated immediately and should no longer provide service. + + mtimehash is not included in confighash because we only know the paths of + extensions after importing them (there is imp.find_module but that faces + race conditions). We need to calculate confighash without importing. """ def trystat(path): try: