<div dir="ltr">Whichever way this goes, is it possible to have the pre-announce include a link to the git repo/branch in question?<div><br></div><div>This way, even if it's not part of the main repo, people can easily find it and check out the pre-release in full.</div><div><br></div><div>FWIW: An advantage of not requiring it to be in the main repo is that if for whatever reason the main repo isn't available on the day (eg: outage, major re-org, gitlab down for a few hours, network issues, etc), it can be uploaded elsewhere and people can still proceed to check things out till the main repo is available again.</div><div><br></div><div>Personally I think it makes sense to be in the main repo where possible, but allowing for the possibility of it not being there in certain circumstances, by explicitly linking to it in the pre-release so it's unambiguous. Of course, it should be in the main repo for final release.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, 20 Jun 2018 at 03:23, Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Jun 18, 2018 at 12:56 PM, Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>> wrote:<br>
> Quoting Juan A. Suarez Romero (2018-06-18 00:22:02)<br>
>> On Fri, 2018-06-15 at 12:47 -0400, Ilia Mirkin wrote:<br>
>> > On Fri, Jun 15, 2018 at 12:36 PM, Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>> wrote:<br>
>> > > I don't even understand why we make these announcements TBH. I have a public<br>
>> > > 18.1-proposed branch that I push to *every weekday*. Anyone can pull that branch<br>
>> > > *anytime* to get the latest version. The only thing the announce email really<br>
>> > > serves AFAICT, is to say "the staging/proposed branch has been merged to the<br>
>> > > release branch". I don't think that's all that interesting TBH.<br>
>> ><br>
>> > "any time" means "no time". The announcement is "speak now or forever<br>
>> > hold your peace". Gives driver teams a chance to review the list and<br>
>> > test things out before they go out into a full release. Should they be<br>
>> > doing this daily/continuously? Probably. But that's not the current<br>
>> > state.<br>
>> ><br>
>><br>
>> I agree on this. And I think the reason to wait 48h is to give time enough for<br>
>> teams to do proper testing, specially when there are many timezones that people<br>
>> get the pre-announcement several hours later.<br>
>><br>
>><br>
>> But, I also think that the pre-announcement is too much verbose. It includes lot<br>
>> of information that can be easily get from the git branch itself. The only thing<br>
>> I would probably keep is about trivial conflicts, sending an explicit email to<br>
>> the authors to say "I [slightly] changed your commit; please, take a look just<br>
>> in case", and the rejected patch list, also mailing the authors.<br>
><br>
> I follow up with authors about changes immediately. If it was rejected I reply<br>
> to the original patch letting them know I'm not planning to pull the commit and<br>
> why. If there's conflicts I ask the original author to look at the changes I've<br>
> made before pulling them into the staging branch.<br>
<br>
That's really helpful, thanks!<br>
<br>
> It feels much too late to be<br>
> asking someone to do that when the release is happening shortly.<br>
<br>
In order for me to find it useful, the flow has to be:<br>
<br>
1. Pre-announce saying which patches you're taking<br>
2. Take feedback from driver maintainers regarding patches to remove or add<br>
3. Do release (or go to 1 if the requested changes were substantial,<br>
at your discretion)<br>
<br>
I realize you're doing this as part of your job, and since I'm not<br>
your employer, I have little say over how you actually do this ... my<br>
options are whether I opt into the stable process or not. I've been<br>
frustrated with the process for a long time - since the mid 10.x's, as<br>
evidenced by my periodic outbursts on the matter, and I've largely<br>
opted out since around 18.0 - it's not worth the heartache for me.<br>
Which is fine - I just tell people to build from HEAD and all is well.<br>
Perhaps I'm in a unique situation and can be ignored -- like I said,<br>
I'm perfectly fine with the status quo of not caring about stable.<br>
Basically no one uses nouveau anyways.<br>
<br>
  -ilia<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Stuart Young (aka Cefiar)</div>