Lgtm do you want to try committing today?<br><div class="gmail_quote"><div dir="ltr">On Sat, Dec 9, 2017 at 9:36 PM Don Hinton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hintonda added inline comments.<br>
<br>
<br>
================<br>
Comment at: <a href="http://lit.site.cfg.py.in:25" rel="noreferrer" target="_blank">lit.site.cfg.py.in:25</a><br>
+# Let the main config do the real work.<br>
+lit_config.load_config(config, "@DEBUGINFO_TESTS_SOURCE_DIR@/<a href="http://lit.config.py" rel="noreferrer" target="_blank">lit.config.py</a>")<br>
----------------<br>
zturner wrote:<br>
> hintonda wrote:<br>
> > This is how lit find <a href="http://lit.config.py" rel="noreferrer" target="_blank">lit.config.py</a>.<br>
> ><br>
> > I'll test the monorepo variant and get back to you.<br>
> Yes, but you need to be able to type `llvm-lit.py path/to/debuginfo-tests`, and if you do that, lit is going to be looking specifically for a `<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>` and won't find it<br>
Changing the name back to <a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a> was required for the monorepo, but also required a change to clang, D41055, which needs to land first -- hope I set the dependency correctly.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D40971" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40971</a><br>
<br>
<br>
<br>
</blockquote></div>