Skip to content

Commit

Permalink
Githubactions: Update Linux OS's
Browse files Browse the repository at this point in the history
  • Loading branch information
aveenismail committed Dec 2, 2024
1 parent 8b03831 commit e5854b2
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 11 deletions.
19 changes: 13 additions & 6 deletions .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,16 @@ jobs:
fail-fast: false
matrix:
include:
- environment: "ubuntu:24.04"
- environment: "ubuntu:24.10"
cc: "gcc"
upload_for_test: "false"
- environment: "ubuntu:24.04"
- environment: "ubuntu:24.10"
cc: "clang"
upload_for_test: "false"
- environment: "ubuntu:23.10"
- environment: "ubuntu:24.04"
cc: "gcc"
upload_for_test: "false"
- environment: "ubuntu:23.10"
- environment: "ubuntu:24.04"
cc: "clang"
upload_for_test: "false"
- environment: "ubuntu:22.04"
Expand Down Expand Up @@ -102,6 +102,13 @@ jobs:
# append the following flags: -Wno-missing-braces -Wno-missing-field-initializers -Wno-implicit-function-declaration
sed -i 's/-Wall -Wextra -Werror/-Wall -Wextra -Werror -Wno-missing-braces -Wno-missing-field-initializers -Wno-implicit-function-declaration/' cmake/SecurityFlags.cmake
- name: apply environment specific changes to CMakeLists.txt 2
working-directory: yubihsm-shell
if: ${{ matrix.environment == 'ubuntu:24.10' }}
run: |
# ubuntu 24.10 comes with _FORTIFY_SOURCE already set
sed -i 's/add_definitions (-D_FORTIFY_SOURCE=2)/add_definitions (-D_FORTIFY_SOURCE=3)/' cmake/SecurityFlags.cmake
- name: do build
working-directory: yubihsm-shell
env:
Expand Down Expand Up @@ -172,10 +179,10 @@ jobs:
- environment: "fedora:40"
cc: "clang"
upload_for_test: "false"
- environment: "fedora:39"
- environment: "fedora:41"
cc: "gcc"
upload_for_test: "false"
- environment: "fedora:39"
- environment: "fedora:41"
cc: "clang"
upload_for_test: "false"

Expand Down
15 changes: 10 additions & 5 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,8 @@ jobs:
fail-fast: false
matrix:
environment: [
"ubuntu:24.10",
"ubuntu:24.04",
"ubuntu:23.10",
"ubuntu:22.04",
"ubuntu:20.04",
# "ubuntu:18.04",
Expand Down Expand Up @@ -211,10 +211,13 @@ jobs:
- name: apply environment specific changes to CMakeLists.txt
working-directory: yubihsm-shell
if: ${{ matrix.environment == 'ubuntu:24.04' }}
env:
DOCKER_IMAGE: ${{ matrix.environment }}
run: |
# ubuntu 24.04 comes with _FORTIFY_SOURCE already set
sed -i 's/add_definitions (-D_FORTIFY_SOURCE=2)/add_definitions (-D_FORTIFY_SOURCE=3)/' cmake/SecurityFlags.cmake
if [ "$DOCKER_IMAGE" = "ubuntu:24.04" ] || [ "$DOCKER_IMAGE" = "ubuntu:24.10" ]; then
# ubuntu 24.04 comes with _FORTIFY_SOURCE already set
sed -i 's/add_definitions (-D_FORTIFY_SOURCE=2)/add_definitions (-D_FORTIFY_SOURCE=3)/' cmake/SecurityFlags.cmake
fi
- name: extract platform name
env:
Expand All @@ -232,6 +235,8 @@ jobs:
set -x
uname -a
ls /usr/include/PCSC
# Create directory containing all output
OUTPUT=$GITHUB_WORKSPACE/$PLATFORM/yubihsm-shell
Expand Down Expand Up @@ -288,8 +293,8 @@ jobs:
fail-fast: false
matrix:
environment: [
"fedora:39",
"fedora:40",
"fedora:41",
]

name: build on ${{ matrix.environment }}
Expand Down

0 comments on commit e5854b2

Please sign in to comment.