Skip to content

Commit

Permalink
fixup! Update linux CI to also build liboqs with -DOQS_LIBJADE_BUILD=OFF
Browse files Browse the repository at this point in the history
Signed-off-by: Pravek Sharma <[email protected]>
  • Loading branch information
praveksharma committed Sep 30, 2024
1 parent 8cee7f7 commit 380a8da
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ jobs:
fail-fast: false
matrix:
cmake-params: [ "", "-DOQS_KEM_ENCODERS=ON" ]
libjade-build: ["ON", "OFF"]
container:
image: openquantumsafe/ci-ubuntu-jammy:latest
env:
Expand All @@ -25,7 +26,7 @@ jobs:
- name: Checkout code
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # pin@v4
- name: Full build
run: OQSPROV_CMAKE_PARAMS=${{ matrix.cmake-params}} ./scripts/fullbuild.sh
run: OQSPROV_CMAKE_PARAMS=${{ matrix.cmake-params}} OQS_LIBJADE_BUILD=${{ matrix.libjade-build }} ./scripts/fullbuild.sh
- name: Enable sibling oqsprovider for testing
run: cd _build/lib && ln -s oqsprovider.so oqsprovider2.so
- name: Test
Expand All @@ -38,9 +39,9 @@ jobs:
fail-fast: false
matrix:
ossl-branch: [openssl-3.1.0, master]
cmake-params:
- -DOQS_LIBJADE_BUILD=ON -DOQS_MINIMAL_BUILD="${{ vars.LIBJADE_ALG_LIST }}"
- -DOQS_LIBJADE_BUILD=OFF
libjade-build:
- "ON"
- "OFF"
include:
- name: alpine
container: openquantumsafe/ci-alpine-amd64:latest
Expand All @@ -58,7 +59,7 @@ jobs:
- name: Checkout code
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # pin@v4
- name: Full build
run: OPENSSL_BRANCH=${{ matrix.ossl-branch }} CMAKE_PARAMS="${{ matrix.cmake-params }}" ./scripts/fullbuild.sh
run: OPENSSL_BRANCH=${{ matrix.ossl-branch }} OQS_LIBJADE_BUILD=${{ matrix.libjade-build }} ./scripts/fullbuild.sh
- name: Enable sibling oqsprovider for testing
run: cd _build/lib && ln -s oqsprovider.so oqsprovider2.so
- name: Test
Expand Down

0 comments on commit 380a8da

Please sign in to comment.