mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-04-20 19:45:12 +00:00
Meta: Ensure BUILD_LAGOM is set when running serenity.sh test lagom
If the superbuild created the lagom binary directory, it won't set BUILD_LAGOM=ON in the CMake cache for that binary directory. Insert a check into build_target() to make sure that if the user explicitly asks for the lagom target, we have all our ducks in a row.
This commit is contained in:
parent
454a839f49
commit
d0506730b8
Notes:
sideshowbarker
2024-07-18 03:50:36 +09:00
Author: https://github.com/ADKaster Commit: https://github.com/SerenityOS/serenity/commit/d0506730b84 Pull-request: https://github.com/SerenityOS/serenity/pull/10041 Reviewed-by: https://github.com/bgianfo
1 changed files with 5 additions and 0 deletions
|
@ -199,6 +199,11 @@ run_tests() {
|
|||
}
|
||||
|
||||
build_target() {
|
||||
if [ "$TARGET" = "lagom" ]; then
|
||||
# Ensure that all lagom binaries get built, in case user first
|
||||
# invoked superbuild for serenity target that doesn't set -DBUILD_LAGOM=ON
|
||||
cmake -S "$SERENITY_SOURCE_DIR/Meta/Lagom" -B "$BUILD_DIR" -DBUILD_LAGOM=ON
|
||||
fi
|
||||
# With zero args, we are doing a standard "build"
|
||||
# With multiple args, we are doing an install/image/run
|
||||
if [ $# -eq 0 ]; then
|
||||
|
|
Loading…
Add table
Reference in a new issue