
<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">I’m fine with renaming the directory lib/Tooling/Refactoring to lib/Tooling/Refactor. Renaming the library to be consistent is a good step forward as well, so thumbs up from me. If there’s still interest after renaming the library, I can make a patch that renames the directory as well. We have some not-yet-upstreamed code in apple/swift-clang on Github that will be affected, but I will take care of it.<div class=""><br class=""></div><div class="">Thanks,</div><div class="">Alex</div><div class=""><div class=""><div><br class=""><blockquote type="cite" class=""><div class="">On May 24, 2019, at 12:40 PM, Nico Weber via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" class="">cfe-dev@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><div dir="ltr" class=""><div class="">On Fri, May 24, 2019 at 2:31 PM Richard Smith <<a href="mailto:richard@metafoo.co.uk" class="">richard@metafoo.co.uk</a>> wrote:<br class=""></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div dir="ltr" class="">On Fri, 24 May 2019 at 09:57, Sam McCall via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank" class="">cfe-dev@lists.llvm.org</a>> wrote:<br class=""></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div dir="ltr" class="">On Fri, May 24, 2019 at 6:28 PM Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank" class="">thakis@chromium.org</a>> wrote:<br class=""></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div dir="ltr" class="">On Fri, May 24, 2019 at 4:34 AM Ilya Biryukov <<a href="mailto:ibiryukov@google.com" target="_blank" class="">ibiryukov@google.com</a>> wrote:<br class=""></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div class="">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 class=""><br class=""></div><div class="">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" class="">https://www.usenix.org/sites/default/files/conference/protected-files/srecon17americas_slides_reese.pdf</a></div><div class=""><br class="">We've renamed many libraries to increase consistency, and we know from experience it's a pretty safe thing to do.</div></div></div></blockquote><div class="">I've dealt with the fallout from one of these renames recently - we silently lost some changes during a (non-git, non-svn) merge.</div><div class="">The unsafeness of it may not be visible from upstream LLVM :-)</div></div></div></blockquote><div class=""><br class=""></div><div class="">I agree. That said, the fact that we accept upstream churn without regard to out-of-tree users (even when those users are ourselves!) is a forcing function that we use to encourage people to upstream their changes. It's unsustainable to hold back upstream cleanups in order to make it easier to maintain out-of-tree patches.</div><div class=""><br class=""></div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div dir="ltr" class=""><div class="gmail_quote"><div class="">If we do rename, do folks prefer:</div><div class=""><br class=""></div><div class="">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 class=""><br class=""></div><div class="">2. Renaming the library to clangToolingRefactoring. Requires updating all cmake files adding a dependency to use the new library name.</div></div></div></blockquote><div class="">Renaming the library is a less invasive change, less likely to screw with out-of-tree modifications, pending patches, other build systems.</div><div class="">So unless anyone has a strong opinion on what the better name is (I don't), I'd prefer #2.</div></div></div></blockquote><div class=""><br class=""></div><div class="">+1 to fixing the build system to give the library a consistent name. </div></div></div></blockquote><div class=""><br class=""></div><div class="">Cool, <a href="https://reviews.llvm.org/D62420" class="">https://reviews.llvm.org/D62420</a><span class="Apple-converted-space"> </span>. Turns out the patch for 2 is tiny. </div></div></div><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">_______________________________________________</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">cfe-dev mailing list</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><a href="mailto:cfe-dev@lists.llvm.org" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">cfe-dev@lists.llvm.org</a><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a></div></blockquote></div><br class=""></div></div></body></html>