Skip to content

Commit

Permalink
Fix CMake configuration error when CLI targets are not included (#1213)
Browse files Browse the repository at this point in the history
Doesn't really make sense to configure sources list based on target
presence.

Should fix #1208
  • Loading branch information
vegorov-rbx authored Mar 25, 2024
1 parent c1830d8 commit bac8511
Showing 1 changed file with 7 additions and 13 deletions.
20 changes: 7 additions & 13 deletions Sources.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -356,19 +356,13 @@ target_sources(isocline PRIVATE
extern/isocline/src/isocline.c
)


if (TARGET Luau.Repl.CLI OR TARGET Luau.Analyze.CLI OR
TARGET Luau.Ast.CLI OR TARGET Luau.CLI.Test OR
TARGET Luau.Reduce.CLI OR TARGET Luau.Compile.CLI OR
TARGET Luau.Bytecode.CLI)
# Common sources shared between all CLI apps.
target_sources(Luau.CLI.lib PRIVATE
CLI/FileUtils.cpp
CLI/Flags.cpp
CLI/Flags.h
CLI/FileUtils.h
)
endif()
# Common sources shared between all CLI apps
target_sources(Luau.CLI.lib PRIVATE
CLI/FileUtils.cpp
CLI/Flags.cpp
CLI/Flags.h
CLI/FileUtils.h
)

if(TARGET Luau.Repl.CLI)
# Luau.Repl.CLI Sources
Expand Down

0 comments on commit bac8511

Please sign in to comment.