[PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 24 06:46:07 PDT 2015


LGTM!

~Aaron

On Mon, Aug 24, 2015 at 9:45 AM, Angel Garcia via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> angelgarcia created this revision.
> angelgarcia added a reviewer: alexfh.
> angelgarcia added subscribers: cfe-commits, klimek.
>
> Update python script, so that it doesn't print that line in new tests.
>
> http://reviews.llvm.org/D12281
>
> Files:
>   clang-tidy/add_new_check.py
>
> Index: clang-tidy/add_new_check.py
> ===================================================================
> --- clang-tidy/add_new_check.py
> +++ clang-tidy/add_new_check.py
> @@ -184,7 +184,6 @@
>    with open(filename, 'w') as f:
>      f.write(
>  """// RUN: %%python %%S/check_clang_tidy.py %%s %(check_name_dashes)s %%t
> -// REQUIRES: shell
>
>  // FIXME: Add something that triggers the check here.
>  void f();
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>


More information about the cfe-commits mailing list