Skip to content

Commit 028de95

Browse files
justsmthjustsmth
committed
Allow prebuilt-NASM w/ Cmake toolchain
1 parent 78258d2 commit 028de95

File tree

5 files changed

+98
-34
lines changed

5 files changed

+98
-34
lines changed

‎.github/workflows/cross.yml

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,43 @@ jobs:
9494
unset AWS_LC_SYS_EXTERNAL_BINDGEN
9595
cross test -p aws-lc-sys --release "${CROSS_TEST_EXTRA_FLAG}" --features ssl --target ${{ matrix.target[0] }}
9696
97+
aws-lc-rs-cross-prebuilt-nasm:
98+
if: github.repository_owner == 'aws'
99+
name: cross prebuilt-NASM - ${{ matrix.target[0] }}
100+
runs-on: ubuntu-latest
101+
env:
102+
AWS_LC_SYS_PREBUILT_NASM: 1
103+
strategy:
104+
fail-fast: false
105+
matrix:
106+
target:
107+
- [ x86_64-pc-windows-gnu, 0, 1 ]
108+
steps:
109+
- uses: actions/checkout@v3
110+
with:
111+
submodules: "recursive"
112+
- uses: dtolnay/rust-toolchain@master
113+
with:
114+
toolchain: "stable"
115+
- name: Install cross
116+
run: cargo install cross --git https://github.com/cross-rs/cross
117+
- uses: dtolnay/rust-toolchain@master
118+
id: toolchain
119+
with:
120+
toolchain: "stable"
121+
target: ${{ matrix.target[0] }}
122+
- name: Set Rust toolchain override
123+
run: rustup override set ${{ steps.toolchain.outputs.name }}
124+
- if: ${{ !startsWith(matrix.target[0], 'x86_64') }}
125+
run: |
126+
echo 'AWS_LC_RS_DISABLE_SLOW_TESTS=1' >> "$GITHUB_ENV"
127+
- name: Cross-compilation (test release)
128+
run: CROSS_CONFIG=$(pwd)/Cross.toml.passthrough-only cross test -p aws-lc-rs --no-run --features unstable --target ${{ matrix.target[0] }}
129+
- name: Cross-compilation (test aws-lc-sys ssl feature)
130+
run: |
131+
unset AWS_LC_SYS_EXTERNAL_BINDGEN
132+
CROSS_CONFIG=$(pwd)/Cross.toml.passthrough-only cross test -p aws-lc-sys --no-run --features ssl --target ${{ matrix.target[0] }}
133+
97134
aws-lc-rs-cross-0_2_5-test:
98135
if: github.repository_owner == 'aws'
99136
name: cross tests ${{ matrix.target }}

‎Cross.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ passthrough = [
3434
"AWS_LC_SYS_CMAKE_BUILDER",
3535
"AWS_LC_SYS_EXTERNAL_BINDGEN",
3636
"AWS_LC_SYS_NO_PREFIX",
37+
"AWS_LC_SYS_PREBUILT_NASM",
3738
"AWS_LC_SYS_PREGENERATING_BINDINGS",
3839
"AWS_LC_SYS_STATIC",
3940
"BINDGEN_EXTRA_CLANG_ARGS",

‎Cross.toml.passthrough-only

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
2+
[build.env]
3+
passthrough = [
4+
"AWS_LC_FIPS_SYS_EXTERNAL_BINDGEN",
5+
"AWS_LC_FIPS_SYS_NO_PREFIX",
6+
"AWS_LC_FIPS_SYS_PREGENERATING_BINDINGS",
7+
"AWS_LC_FIPS_SYS_STATIC",
8+
"AWS_LC_RS_DISABLE_SLOW_TESTS",
9+
"AWS_LC_SYS_CC_SRC_COLLECTOR",
10+
"AWS_LC_SYS_CFLAGS",
11+
"AWS_LC_SYS_CMAKE_BUILDER",
12+
"AWS_LC_SYS_EXTERNAL_BINDGEN",
13+
"AWS_LC_SYS_NO_PREFIX",
14+
"AWS_LC_SYS_PREBUILT_NASM",
15+
"AWS_LC_SYS_PREGENERATING_BINDINGS",
16+
"AWS_LC_SYS_STATIC",
17+
"BINDGEN_EXTRA_CLANG_ARGS",
18+
"GOPROXY",
19+
]

‎Cross.toml.x86_64-unknown-linux-gnu

Whitespace-only changes.

‎aws-lc-sys/builder/cmake_builder.rs

Lines changed: 41 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,9 @@ impl CmakeBuilder {
166166
// Allow environment to specify CMake toolchain.
167167
if let Some(toolchain) = optional_env_optional_crate_target("CMAKE_TOOLCHAIN_FILE") {
168168
set_env_for_target("CMAKE_TOOLCHAIN_FILE", toolchain);
169+
if use_prebuilt_nasm() {
170+
self.configure_prebuilt_nasm(&mut cmake_cfg);
171+
}
169172
return cmake_cfg;
170173
}
171174
// We only consider compiler CFLAGS when no cmake toolchain is set
@@ -244,7 +247,7 @@ impl CmakeBuilder {
244247
};
245248
emit_warning(
246249
&format!(
247-
"Neither script could be tested for execution, falling back to target-based selection: {}",
250+
"Neither script could be tested for execution, falling back to target-based selection: {}",
248251
fallback_script.file_name().unwrap().to_str().unwrap()));
249252
fallback_script
250253
}
@@ -304,42 +307,46 @@ impl CmakeBuilder {
304307
}
305308
}
306309
if use_prebuilt_nasm() {
307-
emit_warning("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
308-
emit_warning("!!! Using pre-built NASM binaries !!!");
309-
emit_warning("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
310-
311-
let script_path = self.select_prebuilt_nasm_script();
312-
let script_path = script_path.display().to_string();
313-
let script_path = script_path.replace('\\', "/");
314-
315-
cmake_cfg.define("CMAKE_ASM_NASM_COMPILER", script_path.as_str());
316-
// Without the following definition, the build fails with a message similar to the one
317-
// reported here: https://gitlab.kitware.com/cmake/cmake/-/issues/19453
318-
// The variables below were found in the associated fix:
319-
// https://gitlab.kitware.com/cmake/cmake/-/merge_requests/4257/diffs
320-
cmake_cfg.define(
321-
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreaded",
322-
"",
323-
);
324-
cmake_cfg.define(
325-
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDLL",
326-
"",
327-
);
328-
cmake_cfg.define(
329-
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDebug",
330-
"",
331-
);
332-
cmake_cfg.define(
333-
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDebugDLL",
334-
"",
335-
);
336-
cmake_cfg.define(
337-
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_DEBUG_INFORMATION_FORMAT_ProgramDatabase",
338-
"",
339-
);
310+
self.configure_prebuilt_nasm(cmake_cfg);
340311
}
341312
}
342313

314+
fn configure_prebuilt_nasm(&self, cmake_cfg: &mut cmake::Config) {
315+
emit_warning("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
316+
emit_warning("!!! Using pre-built NASM binaries !!!");
317+
emit_warning("!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
318+
319+
let script_path = self.select_prebuilt_nasm_script();
320+
let script_path = script_path.display().to_string();
321+
let script_path = script_path.replace('\\', "/");
322+
323+
cmake_cfg.define("CMAKE_ASM_NASM_COMPILER", script_path.as_str());
324+
// Without the following definition, the build fails with a message similar to the one
325+
// reported here: https://gitlab.kitware.com/cmake/cmake/-/issues/19453
326+
// The variables below were found in the associated fix:
327+
// https://gitlab.kitware.com/cmake/cmake/-/merge_requests/4257/diffs
328+
cmake_cfg.define(
329+
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreaded",
330+
"",
331+
);
332+
cmake_cfg.define(
333+
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDLL",
334+
"",
335+
);
336+
cmake_cfg.define(
337+
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDebug",
338+
"",
339+
);
340+
cmake_cfg.define(
341+
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_MultiThreadedDebugDLL",
342+
"",
343+
);
344+
cmake_cfg.define(
345+
"CMAKE_ASM_NASM_COMPILE_OPTIONS_MSVC_DEBUG_INFORMATION_FORMAT_ProgramDatabase",
346+
"",
347+
);
348+
}
349+
343350
fn configure_open_harmony(cmake_cfg: &mut cmake::Config) {
344351
let mut cflags = vec!["-Wno-unused-command-line-argument"];
345352
let mut asmflags = vec![];

0 commit comments

Comments
 (0)