<div dir="ltr">If you could deal with Daniel's feedback and v3 just this patch sooner rather than later, that would be good.  I'd like to merge it before we switch to gitlab so we can start using gitlab CI artifacts for the website right away.  That way, it'll be ready for when the rest of the series lands.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 31, 2018 at 12:52 AM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Laura,<br>
<span class=""><br>
On 30 May 2018 at 23:53, Laura Ekstrand <<a href="mailto:laura@jlekstrand.net">laura@jlekstrand.net</a>> wrote:<br>
> @@ -0,0 +1,9 @@<br>
> +pages:<br>
> +   stage: deploy<br>
> +   script:<br>
> +   - mkdir .public<br>
> +   - cp -r docs/* .public<br>
> +   - mv .public public<br>
<br>
</span>You should be able to just write into the 'public' folder directly.<br>
<br>
It seems like creating a new directory and then moving it is an<br>
attempt to be atomic, which is absolutely the right thing to do! But<br>
the Pages deploy is separate: the build starts from a completely clean<br>
slate, and does not overwrite the live website being served by Pages.<br>
It's only once the build has finished and the artifacts captured that<br>
they're later moved in-place to serve Pages.<br>
<br>
Cheers,<br>
Daniel<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">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/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div>