From 1cbc01e288efc1d953f4a3908a650fc346d0caeb Mon Sep 17 00:00:00 2001 From: Ben Wiederhake Date: Fri, 29 Jan 2021 20:40:09 +0100 Subject: [PATCH] Lagom: Don't include Shell/main.cpp This caused some confusion: Apparently, clang has no trouble overriding Shell's main, and this issue only surfaced when I tried to build the fuzzers with wrong configuration (i.e., without the clang-injected 'main'). The diff is suggested by, and work of, @alimpfard. --- Meta/Lagom/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Meta/Lagom/CMakeLists.txt b/Meta/Lagom/CMakeLists.txt index 47c1a9cf71..56e13cc3bb 100644 --- a/Meta/Lagom/CMakeLists.txt +++ b/Meta/Lagom/CMakeLists.txt @@ -71,7 +71,7 @@ file(GLOB LIBTLS_SOURCES CONFIGURE_DEPENDS "../../Userland/Libraries/LibTLS/*.cp file(GLOB LIBTTF_SOURCES CONFIGURE_DEPENDS "../../Userland/Libraries/LibTTF/*.cpp") file(GLOB SHELL_SOURCES CONFIGURE_DEPENDS "../../Userland/Shell/*.cpp") file(GLOB SHELL_TESTS CONFIGURE_DEPENDS "../../Userland/Shell/Tests/*.sh") -list(REMOVE_ITEM SHELL_SOURCES ../../Userland/Shell/main.cpp) +list(FILTER SHELL_SOURCES EXCLUDE REGEX ".*main.cpp$") set(LAGOM_REGEX_SOURCES ${LIBREGEX_LIBC_SOURCES} ${LIBREGEX_SOURCES}) set(LAGOM_CORE_SOURCES ${AK_SOURCES} ${LIBCORE_SOURCES})