<div dir="ltr">So I guess I'm getting a little lost here.<div><br></div><div>Is the conclusion here that my patch isn't the way we want to go, and instead we should implement something that either marks the commit public once the tag is created or does so on push.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 14, 2016 at 1:53 PM, Sean Farley <span dir="ltr"><<a href="mailto:sean@farley.io" target="_blank">sean@farley.io</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>
Pierre-Yves David <<a href="mailto:pierre-yves.david@ens-lyon.org">pierre-yves.david@ens-lyon.org</a>> writes:<br>
<br>
> On 03/12/2016 12:58 AM, Augie Fackler wrote:<br>
>> On Fri, Mar 11, 2016 at 11:32:54AM -0800, Sean Farley wrote:<br>
>>><br>
>>> Nathan Goldbaum <<a href="mailto:nathan12343@gmail.com">nathan12343@gmail.com</a>> writes:<br>
>>><br>
>>>> # HG changeset patch<br>
>>>> # User Nathan Goldbaum <<a href="mailto:ngoldbau@illinois.edu">ngoldbau@illinois.edu</a>><br>
>>>> # Date 1457664319 21600<br>
>>>> #      Thu Mar 10 20:45:19 2016 -0600<br>
>>>> # Node ID 189ea0e68f3c50c28bbb060c8ef59beef43c8a8d<br>
>>>> # Parent  88738948f5d27cebe673ceb655c6161c34efda7d<br>
>>>> push: warn after pushing draft changesets with tags<br>
>>><br>
>>> I don't know if a warning is going to be enough. After dealing with<br>
>>> users over the last year, I'm inclined to think that 'hg tag foo' should<br>
>>> just mark the tagged commit as public (or possibly the signed commit).<br>
>><br>
>> I have a foggy memory that 'hg tag' was supposed to make the tagged<br>
>> revision public. That could be nuts though, as the context I remember<br>
>> it in was one of the sprints in Denmark.<br>
><br>
> This have been mention multiple time but we never really made our mind<br>
> on this. Mistake will taking are not unheard of and it is useful to be<br>
> able to fix them while still local only.<br>
<br>
</span>Perhaps. My fear is that delaying stuff until push time (in general) is<br>
too late. I always get bitten in the ass by accidentally pushing to a<br>
publishing repo and now everything locally is public. Perhaps at the<br>
sprint we can talk about some warning flags and also about tagged<br>
commits?<br>
<span class=""><br>
> But publishing anything tagged make sense and also help the issue of<br>
> repository that all purely local and grow full draft for ever.<br>
<br>
</span>Righto.<br>
<span class=""><br>
> The current motivation for this is that we don't do any rewriting of the<br>
> tag itself when a tagged changeset is rewritten. I wonder to what extend<br>
> this would be easy to introduce.<br>
<br>
</span>I've experimented with this during converts. I think it's tricky /<br>
fragile but might be possible.<br>
</blockquote></div><br></div>