Skip to content

Commit

Permalink
Add tmate debugging
Browse files Browse the repository at this point in the history
  • Loading branch information
bifurcation committed Dec 14, 2023
1 parent 2de208d commit c354632
Showing 1 changed file with 53 additions and 46 deletions.
99 changes: 53 additions & 46 deletions .github/workflows/main_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,13 @@ env:
CMAKE_TOOLCHAIN_FILE: ${{ github.workspace }}/vcpkg/scripts/buildsystems/vcpkg.cmake

jobs:
debug:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Setup tmate session
uses: mxschmitt/action-tmate@v3

# formatting-check:
# name: Formatting Check
# runs-on: ubuntu-latest
Expand All @@ -27,53 +34,53 @@ jobs:
# clang-format-version: 16
# include-regex: '^\./(src|include|test|cmd)/.*\.(cpp|h)$'
# fallback-style: 'Mozilla'

build-and-unit-test:
#
# build-and-unit-test:
# needs: formatting-check
name: Build and test
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [windows-latest] #, ubuntu-latest, macos-latest]
crypto: [openssl_1.1] #, openssl_3, boringssl]

env:
BUILD_DIR: "${RUNNER_TEMP}/build_${{ matrix.crypto }}"
CRYPTO_DIR: "./alternatives/${{ matrix.crypto }}"

steps:
- uses: actions/checkout@v4
with:
submodules: recursive
fetch-depth: 0

- uses: ./.github/actions/prepare-build
with:
os: ${{ matrix.os }}
crypto: ${{ matrix.crypto }}
cache-dir: ${{ github.workspace }}/vcpkg_cache

- name: Build
run: |
cmake -B "${{ env.BUILD_DIR }}" -DVCPKG_MANIFEST_DIR="${{ env.CRYPTO_DIR }}" -DTESTING=ON
cmake --build "${{ env.BUILD_DIR }}"
- name: Unit Test (non-Windows)
if: matrix.os != 'windows-latest'
run: |
cmake --build "${{ env.BUILD_DIR }}" --target test
# XXX(RLB): Unit tests are currently disabled on Windows because of two
# conflicting bugs. On the one hand, doctest has a bug that causes
# doctest_discover_tests to fail when tests are built with sanitizers. On the
# other hand, if tests are not built with sanitizers, then the unit tests hang
# in the middle of the test run.

- name: Unit Test (Windows)
if: matrix.os == 'windows-latest'
run: |
cmake --build "${{ env.BUILD_DIR }}" --target RUN_TESTS
# name: Build and test
# runs-on: ${{ matrix.os }}
# strategy:
# matrix:
# os: [windows-latest] #, ubuntu-latest, macos-latest]
# crypto: [openssl_1.1] #, openssl_3, boringssl]
#
# env:
# BUILD_DIR: "${RUNNER_TEMP}/build_${{ matrix.crypto }}"
# CRYPTO_DIR: "./alternatives/${{ matrix.crypto }}"
#
# steps:
# - uses: actions/checkout@v4
# with:
# submodules: recursive
# fetch-depth: 0
#
# - uses: ./.github/actions/prepare-build
# with:
# os: ${{ matrix.os }}
# crypto: ${{ matrix.crypto }}
# cache-dir: ${{ github.workspace }}/vcpkg_cache
#
# - name: Build
# run: |
# cmake -B "${{ env.BUILD_DIR }}" -DVCPKG_MANIFEST_DIR="${{ env.CRYPTO_DIR }}" -DTESTING=ON
# cmake --build "${{ env.BUILD_DIR }}"
#
# - name: Unit Test (non-Windows)
# if: matrix.os != 'windows-latest'
# run: |
# cmake --build "${{ env.BUILD_DIR }}" --target test
#
## XXX(RLB): Unit tests are currently disabled on Windows because of two
## conflicting bugs. On the one hand, doctest has a bug that causes
## doctest_discover_tests to fail when tests are built with sanitizers. On the
## other hand, if tests are not built with sanitizers, then the unit tests hang
## in the middle of the test run.
#
# - name: Unit Test (Windows)
# if: matrix.os == 'windows-latest'
# run: |
# cmake --build "${{ env.BUILD_DIR }}" --target RUN_TESTS
#
# interop-test:
# if: github.event.pull_request.draft == false
# needs: build-and-unit-test
Expand Down

0 comments on commit c354632

Please sign in to comment.