<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 10, 2017 at 11:00 AM, Pierre-Yves David <span dir="ltr"><<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 04/10/2017 07:39 PM, Gregory Szorc wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
On Mon, Apr 10, 2017 at 9:48 AM, Pierre-Yves David<br></span>
<<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.or<wbr>g</a> <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-<wbr>lyon.org</a>>><span class=""><br>
wrote:<br>
<br>
# HG changeset patch<br>
# User Pierre-Yves David <<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.or<wbr>g</a><br></span>
<mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-<wbr>lyon.org</a>>><span class=""><br>
# Date 1491583997 -7200<br>
# Fri Apr 07 18:53:17 2017 +0200<br>
# Node ID 421260c23fe9fb0afaa77bbb8c4d9d<wbr>32e631d114<br>
# Parent e0dc40530c5aa514feb6a09cf79ab6<wbr>a3aa2ec331<br>
# EXP-Topic upgraderepo<br>
# Available At<br>
<a href="https://www.mercurial-scm.org/repo/users/marmoute/mercurial/" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>repo/users/marmoute/mercurial/</a><br>
<<a href="https://www.mercurial-scm.org/repo/users/marmoute/mercurial/" rel="noreferrer" target="_blank">https://www.mercurial-scm.org<wbr>/repo/users/marmoute/mercurial<wbr>/</a>><br>
# hg pull<br>
<a href="https://www.mercurial-scm.org/repo/users/marmoute/mercurial/" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>repo/users/marmoute/mercurial/</a><br>
<<a href="https://www.mercurial-scm.org/repo/users/marmoute/mercurial/" rel="noreferrer" target="_blank">https://www.mercurial-scm.org<wbr>/repo/users/marmoute/mercurial<wbr>/</a>> -r<br>
421260c23fe9<br>
upgrade: extract code in its own module<br>
<br>
Given about 2/3 or 'mercurial.repair' is now about repository<br>
upgrade, I think<br>
it is fair to move it into its own module.<br>
<br>
An expected benefit is the ability to drop the 'upgrade' prefix of many<br>
functions. This will be done in coming changesets.<br>
<br>
<br>
Is there an objective to this series other than code cleanup?<br>
</span></blockquote>
<br>
Yep, ultimately, my goal is to refactor this code in order to have access to all "format" item outside of a pure upgrade goal (close to current "deficiencies").<br>
<br>
general idea is to be able to do something like things:<br>
<br>
getformatfoo(repo) -> list of "format" object with "name" and "value"<br>
<br>
The target usage is to provide this data to configexpress (so that client can tell server what format they use and server can also recommend format upgrade to their client).<br>
<br>
Do you need more details?<br></blockquote><div><br></div><div>This sounds fine. No, I don't need any more details.</div><div><br></div><div>(I just didn't want to proceed with a refactor until I knew what the end goal was.)<br></div></div></div></div>