[PATCH] D40971: [debuginfo-tests] Support moving debuginfo-tests to llvm/projects

[Adrian Prantl via llvm-commits]

It looks like they are now not being run at all:

If you compare
http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/41290/consoleFull <http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/41290/consoleFull>
and
http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/41291/consoleFull <http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/41291/consoleFull>
an search for, e.g., "forward-declare-class.cpp".

-- adrian

> On Dec 12, 2017, at 8:41 AM, Zachary Turner <zturner at google.com> wrote:
> 
> That’s fine 
> On Tue, Dec 12, 2017 at 8:13 AM Don Hinton <hintonda at gmail.com <mailto:hintonda at gmail.com>> wrote:
> Sure, but I'll need to do it in order:
> 
>   D41055
>   D40971 (this patch)
>   D40972
> 
> Just wanted to make sure that's okay before starting the process.
> 
> thanks...
> don
> 
> On Tue, Dec 12, 2017 at 8:05 AM, Zachary Turner <zturner at google.com <mailto:zturner at google.com>> wrote:
> Lgtm do you want to try committing today?
> 
> On Sat, Dec 9, 2017 at 9:36 PM Don Hinton via Phabricator <reviews at reviews.llvm.org <mailto:reviews at reviews.llvm.org>> wrote:
> hintonda added inline comments.
> 
> 
> ================
> Comment at: lit.site.cfg.py.in:25 <http://lit.site.cfg.py.in:25/>
> +# Let the main config do the real work.
> +lit_config.load_config(config, "@DEBUGINFO_TESTS_SOURCE_DIR@/lit.config.py <http://lit.config.py/>")
> ----------------
> zturner wrote:
> > hintonda wrote:
> > > This is how lit find lit.config.py <http://lit.config.py/>.
> > >
> > > I'll test the monorepo variant and get back to you.
> > 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 `lit.cfg.py <http://lit.cfg.py/>` and won't find it
> Changing the name back to lit.cfg.py <http://lit.cfg.py/> was required for the monorepo, but also required a change to clang, D41055, which needs to land first -- hope I set the dependency correctly.
> 
> 
> https://reviews.llvm.org/D40971 <https://reviews.llvm.org/D40971>
> 
> 
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171212/89fdddf0/attachment.html>