From 983698bb34ecfbf8a172a59ee4edc2ab7bdfa3b8 Mon Sep 17 00:00:00 2001 From: johnkslang Date: Sun, 23 Aug 2020 01:31:49 -0600 Subject: [PATCH] Revert "Merge pull request #2371 from RafaelMarinheiro/master" This reverts commit f257e0ea6b9aeab2dc7af3207ac6d29d2bbc01d0, reversing changes made to 8f0c6bd7732331186b66118d4613cd0dc7076de4. --- BUILD.bazel | 47 +++++++++++++++++++++++++++++------------------ 1 file changed, 29 insertions(+), 18 deletions(-) diff --git a/BUILD.bazel b/BUILD.bazel index 82c4f66..bfb7797 100644 --- a/BUILD.bazel +++ b/BUILD.bazel @@ -51,10 +51,7 @@ py_binary( genrule( name = "gen_build_info_h", - srcs = [ - "CHANGES.md", - "build_info.h.tmpl", - ], + srcs = ["CHANGES.md", "build_info.h.tmpl"], outs = ["glslang/build_info.h"], cmd = "$(location build_info) $$(dirname $(location CHANGES.md)) -i $(location build_info.h.tmpl) -o $(location glslang/build_info.h)", tools = [":build_info"], @@ -95,8 +92,10 @@ cc_library( ) + [ "OGLCompilersDLL/InitializeDll.cpp", ] + select({ - "@bazel_tools//src/conditions:windows": ["glslang/OSDependent/Windows/ossource.cpp"], - "//conditions:default": ["glslang/OSDependent/Unix/ossource.cpp"], + "@bazel_tools//src/conditions:windows": + ["glslang/OSDependent/Windows/ossource.cpp"], + "//conditions:default": + ["glslang/OSDependent/Unix/ossource.cpp"], }), hdrs = glob([ "glslang/HLSL/*.h", @@ -119,10 +118,7 @@ cc_library( ], linkopts = select({ "@bazel_tools//src/conditions:windows": [""], - "//conditions:default": [ - "-lm", - "-lpthread", - ], + "//conditions:default": ["-lm", "-lpthread"], }), linkstatic = 1, ) @@ -228,6 +224,18 @@ cc_binary( ], ) +filegroup( + name = "test_files", + srcs = glob( + ["Test/**"], + exclude = [ + "Test/bump", + "Test/glslangValidator", + "Test/runtests", + ], + ), +) + cc_library( name = "glslang_test_lib", testonly = 1, @@ -241,9 +249,16 @@ cc_library( "gtests/main.cpp", ], copts = COMMON_COPTS, - defines = [ - "GLSLANG_TEST_DIRECTORY='\"USE_FLAG_INSTEAD\"'", - ], + data = [":test_files"], + defines = select({ + # Unfortunately we can't use $(location) in cc_library at the moment. + # See https://github.com/bazelbuild/bazel/issues/1023 + # So we'll specify the path manually. + "@bazel_tools//src/conditions:windows": + ["GLSLANG_TEST_DIRECTORY='\"../../../../../Test\"'"], + "//conditions:default": + ["GLSLANG_TEST_DIRECTORY='\"Test\"'"], + }), linkstatic = 1, deps = [ ":SPIRV", @@ -266,13 +281,9 @@ GLSLANG_TESTS = glob( [cc_test( name = test_file.replace("gtests/", "").replace(".FromFile.cpp", "") + "_test", srcs = [test_file], - args = [ - "--test-root", - "$(rootpath Test)", - ], copts = COMMON_COPTS, data = [ - "Test", + ":test_files", ], deps = [ ":SPIRV", -- 2.7.4