pyrox.dev
1From 3923f0df76d24b73d57f15eec61ab190ea048093 Mon Sep 17 00:00:00 2001
2From: "Matthew \"strager\" Glazar" <strager.nds@gmail.com>
3Date: Thu, 26 Oct 2023 18:08:30 -0400
4Subject: [PATCH] fix(build): fix installing build tools for cross-compilation
5
6'cmake --install . --component build-tools' copies no files [1]. This
7was caused by commit 1f2e1a47 where the code calling install() became
8dead code on accident. Call install() so that 'cmake --install' copies
9the build artifacts as intended.
10
11[1] https://github.com/quick-lint/quick-lint-js/issues/1099
12
13Refs: 1f2e1a4701793cac24eaac44d7af81a8b820b1bc
14---
15 docs/CHANGELOG.md | 7 +++++++
16 tools/CMakeLists.txt | 1 -
17 2 files changed, 7 insertions(+), 1 deletion(-)
18
19 (docs/CHANGELOG.md changes omitted to reduce conflicts.)
20
21diff --git a/tools/CMakeLists.txt b/tools/CMakeLists.txt
22index 71ccbdf1b..b541afb52 100644
23--- a/tools/CMakeLists.txt
24+++ b/tools/CMakeLists.txt
25@@ -68,7 +68,6 @@ if (QUICK_LINT_JS_ENABLE_BUILD_TOOLS)
26 COMMENT "Building all quick-lint-js build-time tools"
27 DEPENDS ${QUICK_LINT_JS_BUILD_TOOL_TARGETS}
28 )
29-elseif (QUICK_LINT_JS_ENABLE_BUILD_TOOLS)
30 install(
31 TARGETS ${QUICK_LINT_JS_BUILD_TOOL_TARGETS}
32 COMPONENT build-tools