mirror of
https://github.com/XRPLF/rippled.git
synced 2025-11-04 11:15:56 +00:00
3e08c390f5
Artifactory support was added to the `nix` builds with #4556. This extends that support to the Windows build. Now the Windows build works; CI will build and test a Windows release build. This only affects CI and does not change any C++ code. * Copy the remote setup step outcome fix from #4716 discussion * Allow the Windows job to succeed if tests fail: * Currently the tests do not always pass, even on a single threaded run on the GitHub runners. So we are using parallel runs and mark the test step as allowed to fail (continue-on-error). * At this point, it's more important that the build succeeds than that the tests succeed, because: * We've got plenty of test coverage on the other jobs. * Test failures are much rarer than build failures because of cross-platform issues. * Having a test failure locally doesn't interrupt a workflow nearly as much as a build failure. Note that Conan Center cannot hold the binaries we need. They do not build the configurations we need, and they will not add them. ## Future Tasks This introduces a new bottleneck since the build and test takes over an hour. Speed up the job by: * Making this job run on heavy Windows runners. * Increasing the number of hardware threads.
30 lines
777 B
YAML
30 lines
777 B
YAML
name: build
|
|
inputs:
|
|
generator:
|
|
default: null
|
|
configuration:
|
|
required: true
|
|
cmake-args:
|
|
default: null
|
|
# An implicit input is the environment variable `build_dir`.
|
|
runs:
|
|
using: composite
|
|
steps:
|
|
- name: configure
|
|
shell: bash
|
|
run: |
|
|
cd ${build_dir}
|
|
cmake \
|
|
${{ inputs.generator && format('-G "{0}"', inputs.generator) || '' }} \
|
|
-DCMAKE_TOOLCHAIN_FILE:FILEPATH=build/generators/conan_toolchain.cmake \
|
|
-DCMAKE_BUILD_TYPE=${{ inputs.configuration }} \
|
|
${{ inputs.cmake-args }} \
|
|
..
|
|
- name: build
|
|
shell: bash
|
|
run: |
|
|
cmake \
|
|
--build ${build_dir} \
|
|
--config ${{ inputs.configuration }} \
|
|
--parallel ${NUM_PROCESSORS:-$(nproc)}
|