<div dir="ltr"><div dir="ltr">.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 1, 2019 at 9:22 PM Pierre-Yves David <<a href="mailto:pierre-yves.david@ens-lyon.org">pierre-yves.david@ens-lyon.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">.<br>
<br>
On 5/1/19 2:14 PM, Sushil Khanchi wrote:<br>
> .<br>
> <br>
> On Wed, May 1, 2019 at 5:14 PM Pierre-Yves David <br>
> <<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a> <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a>>> <br>
> wrote:<br>
> <br>
> <br>
> <br>
>     On 5/1/19 1:14 PM, Sushil Khanchi wrote:<br>
>      > .<br>
>      ><br>
>      > On Fri, Apr 26, 2019 at 8:29 PM Pierre-Yves David<br>
>      > <<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a><br>
>     <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a>><br>
>     <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a><br>
>     <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a>>>><br>
>      > wrote:<br>
>      ><br>
>      ><br>
>      ><br>
>      >     On 4/19/19 9:49 AM, khanchi97 (Sushil khanchi) wrote:<br>
>      >      > khanchi97 created this revision.<br>
>      >      > Herald added a subscriber: mercurial-devel.<br>
>      >      > Herald added a reviewer: hg-reviewers.<br>
>      >      ><br>
>      >      > REVISION SUMMARY<br>
>      >      >    This patch make sure that we abort if the user is trying to<br>
>      >      >    close a branch from a cset which is not a branch head.<br>
>      >      >    Changes in test file reflect the fixed behaviour.<br>
>      ><br>
>      >     Is there a way to override this ? There can be situation<br>
>     where the<br>
>      >     changeset is not a local head but could be a remote head.<br>
>      ><br>
>      > By closing remote head, you mean first close that locally (where<br>
>     it is<br>
>      > not a head) and push that to a remote (where it is a head)?<br>
> <br>
>     Yes<br>
> <br>
> I don't think in present code there is any way to override this. Do you <br>
> have any idea? Or any other place in mercurial you remember where we <br>
> handle similar situation?<br>
> And just for knowledge I would like to know how often developers use <br>
> this remote head closing thing (when locally it is not a head)?<br>
<br>
Something similar coming to mind is `hg tag`. It will -warn- when <br>
tagging a non-head but will still tag. Maybe we should align on this <br>
(warning instead of aborting)<br></blockquote><div><br></div><div>How about prompting the user to confirm? One upside in prompting I see is that user don't have to run `hg strip/prune`. Also the downside is it requires user intervention.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
Pierre-Yves David<br>
</blockquote></div></div>