<div dir="ltr">Thanks NIco :-) I think it shouldn't be necessary for this to happen in any particular order, so you can start moving gn now, if you'd like (but also not a huge rush).</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 11:02 AM Nico Weber <<a href="mailto:thakis@google.com">thakis@google.com</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"><div dir="ltr">Yes, moving gn shouldn't be a big problem. It needs some minor bot and docs wrangling. Let me know once this is in and I'll work on moving the GN files. Should be doable in a week or two.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 2:00 PM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</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"><div dir="ltr">Thanks for the summary & other work with this proposal, Geoffrey!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 10:49 AM Geoffrey Martin-Noble <<a href="mailto:gcmn@google.com" target="_blank">gcmn@google.com</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"><div dir="ltr">This is an approval of the proposal (<a href="https://reviews.llvm.org/D98639" target="_blank">patch</a> to actually indicate that in the status field). The next step is landing the Bazel build files, which will be subject to the normal patch review process. Chris added notes from our discussion about the issues discussed, which includes the location of the build files. We agreed these should be in the root `utils/` directory and we also think the gn build should move there (it's current location predates the monorepo). I was going to start a separate thread, but I'll just <a class="gmail_plusreply" id="gmail-m_3685689051825239228gmail-m_2882116000586005793gmail-m_2598226780378178383plusReplyChip-0" href="mailto:thakis@google.com" target="_blank">+Nico Weber</a>. Nico can you take a look at moving the gn files? Hopefully this should be pretty trivial?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 10:30 AM David Blaikie via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.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"><div dir="ltr">Thanks for the update Chris - could you summarize what this means for the proposal/what stage in the proposal process this is? Does this represent approval, and the patch should now be submitted without further high level design review (that is covered by the proposal review)? Or are there further steps?<br><br>(does the approval indicate where these files should live? Next to the gn files? A new top level location? or is that still up to further community review)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 15, 2021 at 9:22 AM Chris Bieneman via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.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"><div>Hello LLVM-Dev,<div><br></div><div>Last week the review managers met to discuss this proposal. I've updated the proposal document with a summary of the meeting. You can find the proposal online <a href="https://github.com/llvm/llvm-www/blob/main/proposals/LP0002-BazelBuildConfiguration.md" target="_blank">here</a>.</div><div><br></div><div>The TL;DR is that the review managers agreed the proposal should be approved.</div><div><br></div><div>Thank you everyone who participated in the conversations around this proposal, and especially Geoffrey for putting the proposal together and shepherding it along.</div><div><br></div><div>-Chris<br><div><br><blockquote type="cite"><div>On Feb 19, 2021, at 1:46 PM, Geoffrey Martin-Noble <<a href="mailto:gcmn@google.com" target="_blank">gcmn@google.com</a>> wrote:</div><br><div><div dir="ltr">A reminder that the review period for this ends 2021-02-23, this coming Tuesday. Rest assured that if you expressed opinions in the previous RFC threads then review managers will also consider those points when discussing. We're not going to skip some point just because it wasn't posted in the correct thread :-D<div><br></div><div>Best,</div><div>Geoffrey</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 10, 2021 at 3:44 AM Renato Golin <<a href="mailto:rengolin@gmail.com" target="_blank">rengolin@gmail.com</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"><div dir="ltr"><div dir="ltr">On Tue, 9 Feb 2021 at 21:00, Geoffrey Martin-Noble <<a href="mailto:gcmn@google.com" target="_blank">gcmn@google.com</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>To expand a bit on Eric's response, the intent here is *not* to make Bazel a supported build system for LLVM or to replace CMake (which I believe the proposal makes clear), but rather to enable Bazel usage and shared configuration for people and projects that already use it. I do not expect that Bazel will cover all the use cases currently supported by LLVM CMake any time soon (ever?).I don't work on Bazel itself, so have no insight on the support plan for those architectures. Only developers interested in working with Bazel would be expected to use or update the configuration, so lack of support for specific architectures should not affect things, I think.</div></div></div></blockquote><div><br></div><div>My views exactly. Bazel will not be a "supported" build system and doesn't need to build on all platforms and environments LLVM builds. It should only concern people that actually use Bazel and be completely transparent to the rest who don't.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
</blockquote></div></div>
</blockquote></div></div>
</blockquote></div>
</div></blockquote></div><br></div></div>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>