<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jul 7, 2015, at 9:18 AM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" class="">dexonsmith@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br class="Apple-interchange-newline">On 2015-Jul-07, at 09:17, Chris Bieneman <<a href="mailto:beanz@apple.com" class="">beanz@apple.com</a>> wrote:<br class=""><br class="">I see the other thread now.<br class=""><br class="">What if instead we did something like this:<br class=""><br class="">diff --git a/.gitignore b/.gitignore<br class="">index f3424d2..e3d191d 100644<br class="">--- a/.gitignore<br class="">+++ b/.gitignore<br class="">@@ -43,7 +43,9 @@ autoconf/autom4te.cache<br class=""># Directories to ignore (do not add trailing '/'s, they skip symlinks).<br class="">#==============================================================================#<br class=""># External projects that are tracked independently.<br class="">-projects/*/<br class="">+projects/*<br class="">+!projects/*.*<br class="">+!projects/Makefile<br class=""># Clang, which is tracked independently.<br class="">tools/clang<br class=""># LLDB, which is tracked independently.<br class=""><br class="">It still wouldn’t be perfect, but it is less error-prone since most files other than Makefiles have extensions.<br class=""><br class="">Thoughts?<br class=""></blockquote><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Great idea, LGTM :).</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""></div></blockquote><div><br class=""></div><div>Committed r241622.</div><div><br class=""></div><div>Thanks,</div><div>-Chris</div><br class=""><blockquote type="cite" class=""><div class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">-Chris<br class=""><br class=""><blockquote type="cite" class="">On Jul 7, 2015, at 10:55 AM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" class="">dexonsmith@apple.com</a>> wrote:<br class=""><br class=""><br class=""><blockquote type="cite" class="">On 2015-Jul-07, at 07:44, Chris Bieneman <<a href="mailto:beanz@apple.com" class="">beanz@apple.com</a>> wrote:<br class=""><br class="">Was there a driving reason for this change or was this just to simplify things?<br class=""></blockquote><br class="">It was in response to r240771. It's error prone to require explicit<br class="">inclusions for each file under version control.<br class=""><br class=""><blockquote type="cite" class="">The reason I’m asking is because this is not functionally equivalent to what existed before. With this change, if you symlink projects under the projects folder the symlinks don’t get ignored by git.<br class=""></blockquote><br class="">I knew it wasn't functionally equivalent -- that was actually the<br class="">point -- but I hadn't considered symlink-ing in projects since that's<br class="">not part of my flow. Feel free to revert!<br class=""><br class=""><blockquote type="cite" class=""><blockquote type="cite" class="">On Jun 29, 2015, at 12:43 PM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" class="">dexonsmith@apple.com</a>> wrote:<br class=""><br class="">Author: dexonsmith<br class="">Date: Mon Jun 29 12:43:26 2015<br class="">New Revision: 240973<br class=""><br class="">URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D240973-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=4raMxsidNYWvCxOqY9_7ypzkaqlVx0r3NGGeeY0pj3s&s=tQyF96GhsZGewb2HjmajzWwW-sNbfXUQRsqFIYUiRKI&e=" class="">http://llvm.org/viewvc/llvm-project?rev=240973&view=rev</a><br class="">Log:<br class="">Simplify .gitignore: projects/* => projects/*/<br class=""><br class="">Avoid listing inclusions (like `!projects/LLVMBuild.txt`) for files<br class="">directly underneath `projects/` in `.gitignore`. Instead, change the<br class="">`projects/*` exclusion to the more specific `projects/*/`.<br class=""><br class="">Modified:<br class="">llvm/trunk/.gitignore<br class=""><br class="">Modified: llvm/trunk/.gitignore<br class="">URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_llvm_trunk_.gitignore-3Frev-3D240973-26r1-3D240972-26r2-3D240973-26view-3Ddiff&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=4raMxsidNYWvCxOqY9_7ypzkaqlVx0r3NGGeeY0pj3s&s=cCRLZabKReFW7FPrKknHPB6NLMpKSdqDzUdGzSykSok&e=" class="">http://llvm.org/viewvc/llvm-project/llvm/trunk/.gitignore?rev=240973&r1=240972&r2=240973&view=diff</a><br class="">==============================================================================<br class="">--- llvm/trunk/.gitignore (original)<br class="">+++ llvm/trunk/.gitignore Mon Jun 29 12:43:26 2015<br class="">@@ -43,10 +43,7 @@ autoconf/autom4te.cache<br class=""># Directories to ignore (do not add trailing '/'s, they skip symlinks).<br class="">#==============================================================================#<br class=""># External projects that are tracked independently.<br class="">-projects/*<br class="">-!projects/CMakeLists.txt<br class="">-!projects/LLVMBuild.txt<br class="">-!projects/Makefile<br class="">+projects/*/<br class=""># Clang, which is tracked independently.<br class="">tools/clang<br class=""># LLDB, which is tracked independently.<br class=""><br class=""><br class="">_______________________________________________<br class="">llvm-commits mailing list<br class=""><a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</blockquote></blockquote></blockquote></blockquote></div></blockquote></div><br class=""></body></html>