No subject
Tue Dec 8 09:22:11 UTC 2009
e
major issue of eliminate bfrefresh is how the status is reported to users.
Therefore, Integrating bfiles status, bfrefresh, as well as detecting if
user adds/modifies/removes standin file should be combined. I propose the
following solutions:
1. Implement all these features in bfiles (it is mandatorily enabled)
2. Allow user to enable/disable "auto-status", but "auto-bfrefresh" i=
s
mandatory (so that it is easier to detect standin changes). However, if
disable "auto-status", the status report may be surprising.
3. Have these three features as one big option (enable/disable
all-together in configuration).
-Wendy
On Mon, Feb 15, 2010 at 3:48 PM, Qi Yang <qi.yang137 at gmail.com> wrote:
> Hi,
>
> Given "auto-status" proposal is set (
> http://selenic.com/pipermail/mercurial-devel/2010-February/018892.html).
> I'd like to propose to directly call bfrefresh when doing commit in bfile=
s
> and make "bfrefresh" command private.
>
> The benifit of doing so include the following:
>
> 1. No worries about inconsistency between standin and bfile
>
> - Standin and bfiles will be synchronized when commit. The only time
> when they are out of sync would be: a. when "hg up -r <revision#>" is =
called
> or b. when standin file is manually changed by user.
> - In scenario a. we can either enable =93auto-update=94 or give a warn=
ing
> message to user when =93hg up=94 is called and =93auto-update=94 is no=
t enabled.
> - In scenario b. we can detect if the standin file is modified by
> compare the mtime(standin) and the latest commit time.
>
> 2. Easier to detect when standin is modified by user
>
> - There has been a lot of discussion about how to detect this problem =
(
> http://selenic.com/pipermail/mercurial-devel/2010-February/018539.html=
)
> if we this proposal is possible, we could simplify this problem and so=
lve it
> as solution in scenario b.
>
> 3. No need for .hg/bfiles/pending
>
> - Since the standin file will only be update when commit is called,
> there=92s no need for the =93pending=94 standin files to exist.
>
> From earlier discussion with Greg (
> http://selenic.com/pipermail/mercurial-devel/2010-February/018381.html),
> the major issue of eliminate bfrefresh is how the status is reported to
> users. Therefore, Integrating bfiles status, bfrefresh, as well as detect=
ing
> if user adds/modifies/removes standin file should be combined. I propose =
the
> following solutions:
>
> 1. Implement all these features in bfiles (it is mandatorily
> enabled)
>
> 2. Allow user to enable/disable =93auto-status=94, but =93auto-bfre=
fresh=94
> is mandatory (so that it is easier to detect standin changes). However, i=
f
> disable =93auto-status=94, the status report may be surprising.
>
> 3. Have these three features as one big option (enable/disable
> all-together in configuration).
>
>
> -Wendy
>
--001636b2af53aaf852047fcd90dc
Content-Type: text/html; charset=windows-1252
Content-Transfer-Encoding: quoted-printable
Sorry, there were some symbols messed up in last e-mail. I'd appreciate=
if anyone could give some feedbacks or suggestions on the following propos=
al.<br><br>Thanks,<br>Wendy<br><br><br>Hi,<br><br>Given "auto-status&q=
uot; proposal is set (<a href=3D"http://selenic.com/pipermail/mercurial-dev=
el/2010-February/018892.html" target=3D"_blank">http://selenic.com/pipermai=
l/mercurial-devel/2010-February/018892.html</a>). I'd like to propose t=
o directly call bfrefresh when doing commit in bfiles and make "bfrefr=
esh" command private.<br>
<br>The benifit of doing so include the following:<br><br>1.=A0=A0=A0=A0=A0=
=A0 No worries about inconsistency between standin and bfile<br><br>=A0=A0=
=A0 * Standin and bfiles will be synchronized when commit. The only time wh=
en they are out of sync would be: a. when "hg up -r <revision#>&=
quot; is called or b. when standin file is manually changed by user.<br>
=A0=A0=A0 * In scenario a. we can either enable "auto-update" or =
give a warning message to user when "hg up" is called and "a=
uto-update" is not enabled. <br>=A0=A0=A0 * In scenario b. we can dete=
ct if the standin file is modified by compare the mtime(standin) and the la=
test commit time.<br>
<br>2.=A0=A0=A0=A0=A0=A0=A0 Easier to detect when standin is modified by us=
er<br><br>=A0=A0=A0 * There has been a lot of discussion about how to detec=
t this problem (<a href=3D"http://selenic.com/pipermail/mercurial-devel/201=
0-February/018539.html" target=3D"_blank">http://selenic.com/pipermail/merc=
urial-devel/2010-February/018539.html</a>) if we this proposal is possible,=
we could simplify this problem and solve it as solution in scenario b.<br>
<br>3.=A0=A0=A0=A0=A0=A0 No need for .hg/bfiles/pending<br><br>=A0=A0=A0 * =
Since the standin file will only be update when commit is called, there'=
;s no need for the "pending" standin files to exist.<br><br>From =
earlier discussion with Greg (<a href=3D"http://selenic.com/pipermail/mercu=
rial-devel/2010-February/018381.html" target=3D"_blank">http://selenic.com/=
pipermail/mercurial-devel/2010-February/018381.html</a>), the major issue o=
f eliminate bfrefresh is how the status is reported to users. Therefore, In=
tegrating bfiles status, bfrefresh, as well as detecting if user adds/modif=
ies/removes standin file should be combined. I propose the following soluti=
ons:<br>
<br>1.=A0=A0=A0=A0=A0=A0 Implement all these features in bfiles (it is mand=
atorily enabled)<br><br>2.=A0=A0=A0=A0=A0=A0 Allow user to enable/disable &=
quot;auto-status", but "auto-bfrefresh" is mandatory (so tha=
t it is easier to detect standin changes). However, if disable "auto-s=
tatus", the status report may be surprising.<br>
<br>3.=A0=A0=A0=A0=A0=A0 Have these three features as one big option (enabl=
e/disable all-together in configuration).<br><br><br>-Wendy<br><br><div cla=
ss=3D"gmail_quote">On Mon, Feb 15, 2010 at 3:48 PM, Qi Yang <span dir=3D"lt=
r"><<a href=3D"mailto:qi.yang137 at gmail.com" target=3D"_blank">qi.yang137=
@gmail.com</a>></span> wrote:<br>
<blockquote class=3D"gmail_quote" style=3D"border-left: 1px solid rgb(204, =
204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<p style=3D"font-family: arial,helvetica,sans-serif;" class=3D"MsoNormal">H=
i, </p><p style=3D"font-family: arial,helvetica,sans-serif;" class=3D"MsoNo=
rmal">Given "auto-status" proposal is
set (<a href=3D"http://selenic.com/pipermail/mercurial-devel/2010-February/=
018892.html" target=3D"_blank">http://selenic.com/pipermail/mercurial-devel=
/2010-February/018892.html</a>). I'd like to propose to directly call b=
frefresh when doing commit in bfiles
and make "bfrefresh" command private.</p>
<p style=3D"font-family: arial,helvetica,sans-serif;" class=3D"MsoNormal">T=
he benifit of doing so include the
following:</p>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>1.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span>No worries about inconsistency between standin
and bfile</p>
<ul style=3D"font-family: arial,helvetica,sans-serif;" type=3D"disc"><li cl=
ass=3D"MsoNormal" style=3D"line-height: normal;"><span style=3D"font-size: =
12pt;">Standin and bfiles will be synchronized when commit.
The only time when they are out of sync would be: a. when "hg up =
-r
<revision#>" is called or b. when standin file is manually
changed by user.</span></li><li class=3D"MsoNormal" style=3D"line-heig=
ht: normal;"><span style=3D"font-size: 12pt;">In scenario a. we can either =
enable =93auto-update=94 or
give a warning message to user when =93hg up=94 is called and =93auto-=
update=94 is
not enabled.=A0 </span></li><li class=3D"MsoNormal" style=3D"line-heig=
ht: normal;"><span style=3D"font-size: 12pt;">In scenario b. we can detect =
if the standin file is
modified by compare the mtime(standin) and the latest commit time.</sp=
an></li></ul>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>2.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span><span>=A0</span>Easier to
detect when standin is modified by user</p>
<ul style=3D"font-family: arial,helvetica,sans-serif;" type=3D"disc"><li cl=
ass=3D"MsoNormal" style=3D"line-height: normal;"><span style=3D"font-size: =
12pt;">There has been a lot of discussion about how to detect
this problem (<a href=3D"http://selenic.com/pipermail/mercurial-devel/=
2010-February/018539.html" target=3D"_blank">http://selenic.com/pipermail/m=
ercurial-devel/2010-February/018539.html</a>)
if we this proposal is possible, we could simplify this problem and so=
lve
it as solution in scenario b.</span></li></ul>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>3.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span>No need for .hg/bfiles/pending</p>
<ul style=3D"font-family: arial,helvetica,sans-serif;" type=3D"disc"><li cl=
ass=3D"MsoNormal" style=3D"line-height: normal;"><span style=3D"font-size: =
12pt;">Since the standin file will only be update when commit
is called, there=92s no need for the =93pending=94 standin files to ex=
ist.</span></li></ul>
<p style=3D"font-family: arial,helvetica,sans-serif;" class=3D"MsoNormal">F=
rom earlier discussion with Greg (<a href=3D"http://selenic.com/pipermail/m=
ercurial-devel/2010-February/018381.html" target=3D"_blank">http://selenic.=
com/pipermail/mercurial-devel/2010-February/018381.html</a>),
the major issue of eliminate bfrefresh is how the status is reported to use=
rs.
Therefore, Integrating bfiles status, bfrefresh, as well as detecting if us=
er
adds/modifies/removes standin file should be combined. I propose the follow=
ing
solutions:</p>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>1.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span>Implement all these features in bfiles (it is
mandatorily enabled)</p>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>2.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span>Allow user to enable/disable =93auto-status=94, but
=93auto-bfrefresh=94 is mandatory (so that it is easier to detect standin c=
hanges).
However, if disable =93auto-status=94, the status report may be surprising.=
</p>
<p style=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;"=
><span><span>3.<span style=3D"font-style: normal; font-variant: normal; fon=
t-weight: normal; font-size: 7pt; line-height: normal; font-size-adjust: no=
ne; font-stretch: normal;">=A0=A0=A0=A0=A0=A0
</span></span></span>Have these three features as one big option
(enable/disable all-together in configuration). <br></p><p style=3D"margin-=
left: 20.25pt; font-family: arial,helvetica,sans-serif;"><br></p><p style=
=3D"margin-left: 20.25pt; font-family: arial,helvetica,sans-serif;">
-Wendy<br></p>
</blockquote></div><br>
--001636b2af53aaf852047fcd90dc--
More information about the Mercurial-devel
mailing list