[llvm] r260386 - Disable all llvm-config tests for now, will investigate later

Chris Bieneman via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 10:59:27 PST 2016


I know this is an old patch, but I just became aware of it.

This patch is not okay. Disabling a whole test suite without explanation and leaving it disabled for 10 months is very bad.

Ehsan, do you intend to investigate and re-enable these tests?

In r260281 you also deleted a test out of this test suite (also bad), are you going to fix that as well?

-Chris

> On Feb 10, 2016, at 9:29 AM, Ehsan Akhgari via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Author: ehsan
> Date: Wed Feb 10 11:29:50 2016
> New Revision: 260386
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=260386&view=rev
> Log:
> Disable all llvm-config tests for now, will investigate later
> 
> Added:
>    llvm/trunk/test/tools/llvm-config/lit.local.cfg
> 
> Added: llvm/trunk/test/tools/llvm-config/lit.local.cfg
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-config/lit.local.cfg?rev=260386&view=auto
> ==============================================================================
> --- llvm/trunk/test/tools/llvm-config/lit.local.cfg (added)
> +++ llvm/trunk/test/tools/llvm-config/lit.local.cfg Wed Feb 10 11:29:50 2016
> @@ -0,0 +1,2 @@
> +# Disable all llvm-config tests for now.
> +config.unsupported = True
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list