[PATCH remotefilelog-ext] fileserverclient: add option to provide file path to cacheprocess
Durham Goode
durham at fb.com
Tue Jan 19 17:50:11 CST 2016
On 1/19/16 7:48 AM, Augie Fackler wrote:
> # HG changeset patch
> # User Augie Fackler <augie at google.com>
> # Date 1453218309 18000
> # Tue Jan 19 10:45:09 2016 -0500
> # Node ID acf7a6bb3ce489c12065b171e4e7f44f49ce37b7
> # Parent b57b617d85adb373dc5c7b22e9fa750430163632
> fileserverclient: add option to provide file path to cacheprocess
>
> For our uses of remotefilelog, life is significantly easier if we also
> have the file path rather than just a hash of the file path. Hide this
> behind a config knob so users can enable it or not as makes sense.
>
> diff --git a/remotefilelog/fileserverclient.py b/remotefilelog/fileserverclient.py
> --- a/remotefilelog/fileserverclient.py
> +++ b/remotefilelog/fileserverclient.py
> @@ -166,6 +166,8 @@ class fileserverclient(object):
> self.cacheprocess = ui.config("remotefilelog", "cacheprocess")
> if self.cacheprocess:
> self.cacheprocess = util.expandpath(self.cacheprocess)
> + self.cacheprocesspath = ui.configbool(
I don't really like calling it 'cacheprocesspath', since it's not the
path to the cache process (which is what self.cacheprocess is). Mind if
I change it inflight? I'll leave the ui facing option name alone.
> + "remotefilelog", "cacheprocess.includepath")
> self.debugoutput = ui.configbool("remotefilelog", "debug")
>
> self.localcache = localcache(repo)
> @@ -204,6 +206,8 @@ class fileserverclient(object):
> reponame = repo.name
> for file, id in fileids:
> fullid = getcachekey(reponame, file, id)
> + if self.cacheprocesspath:
> + request += file + '\0'
> request += fullid + "\n"
> idmap[fullid] = file
>
More information about the Mercurial-devel
mailing list