<div dir="ltr">I don't think it's risky, I think it does not add much value, but would require some work and would be a little inconvenient for those touching this code (merge conflicts, integrates, etc).<div><div><br></div><div>If an option not to do it is out of the table, I'd vote to rename the CMake target as that seems to cause strictly less problems: no merge conflicts due to moved files, no changes needed to source code outside build files.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 24, 2019 at 6:28 PM Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.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"><div dir="ltr">On Fri, May 24, 2019 at 4:34 AM Ilya Biryukov <<a href="mailto:ibiryukov@google.com" target="_blank">ibiryukov@google.com</a>> wrote:<br></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>While I personally like consistent naming myself, I'd prefer to be conservative with this one and avoid changing something that is not broken for other reasons and was like this for years.</div></div></blockquote><div><br></div><div>I find this kind of argument not very convincing. See e.g. the first 7 slides of <a href="https://www.usenix.org/sites/default/files/conference/protected-files/srecon17americas_slides_reese.pdf" target="_blank">https://www.usenix.org/sites/default/files/conference/protected-files/srecon17americas_slides_reese.pdf</a></div><div><br>We've renamed many libraries to increase consistency, and we know from experience it's a pretty safe thing to do.</div><div><br></div><div>If we do rename, do folks prefer:</div><div><br></div><div>1. Renaming the directory to lib/clang/Tooling/Refactor. Requires updating all #include lines referring to it, and updating a handful of CMake files.</div><div><br></div><div>2. Renaming the library to clangToolingRefactoring. Requires updating all cmake files adding a dependency to use the new library name.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 23, 2019 at 8:05 PM Yitzhak Mandelbaum <<a href="mailto:yitzhakm@google.com" target="_blank">yitzhakm@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"><div style="font-family:arial,helvetica,sans-serif">Agreed that it's a bit confusing. I have no preference on what the final (consistent) name should be.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 23, 2019 at 2:04 PM Eric Liu <<a href="mailto:ioeric@waymo.com" target="_blank">ioeric@waymo.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"><a class="gmail_plusreply" id="gmail-m_-8655606204809677565gmail-m_-1708479218577196122gmail-m_4332246099564441448gmail-m_2941005729564839286plusReplyChip-4" href="mailto:arphaman@gmail.com" target="_blank">+Alex L</a> <a class="gmail_plusreply" id="gmail-m_-8655606204809677565gmail-m_-1708479218577196122gmail-m_4332246099564441448gmail-m_2941005729564839286plusReplyChip-0" href="mailto:ibiryukov@google.com" target="_blank">+Ilya Biryukov</a> <a class="gmail_plusreply" id="gmail-m_-8655606204809677565gmail-m_-1708479218577196122gmail-m_4332246099564441448gmail-m_2941005729564839286plusReplyChip-1" href="mailto:sammccall@google.com" target="_blank">+Sam McCall</a> <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 23, 2019 at 11:01 AM Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.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">Hi,<div><br></div><div>all clang libs are usually in clang/lib/Foo/Bar and are the called clangFooBar.</div><div><br></div><div>However, lib/Tooling/Refactoring is called clangToolingRefactor without the "ing".</div><div><br>Can we make that consistent? Would you prefer if we renamed the directory or the library?</div><div><br></div><div>(I can do the actual renaming if we agree that it's a good thing to do.)</div><div><br></div><div>Nico</div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_-8655606204809677565gmail-m_-1708479218577196122gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>
</blockquote></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>