Skip to content

Commit 3acf04a

Browse files
committed
.github/workflows/: Fixed environment variables not in the top scope
The variables previously expanded to nothing without error
1 parent 432c2fe commit 3acf04a

File tree

6 files changed

+25
-23
lines changed

6 files changed

+25
-23
lines changed

.github/workflows/ci.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,16 @@
22

33
name: chrony-dbus-service-ci
44
on: [push, workflow_dispatch]
5+
env:
6+
PROJECT_NAME: ${{ github.repository }}
7+
BUILD_WORKSPACE: build
58
jobs:
69
build-doxygen:
710
uses: ./.github/workflows/doxygen.yaml
811
clang-format:
912
runs-on: ubuntu-latest
1013
container:
1114
image: fedora:41
12-
env:
13-
PROJECT_NAME: ${{ github.repository }}
14-
BUILD_WORKSPACE: build
1515
steps:
1616
- name: pepare-git
1717
run: |

.github/workflows/cmake-gcc.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,15 @@ on:
88
description: Parameters passed to cmake
99
required: true
1010
type: string
11+
env:
12+
PROJECT_NAME: ${{ github.repository }}
13+
BUILD_WORKSPACE: build
14+
PARALLEL: 4
1115
jobs:
1216
build-gcc:
1317
runs-on: ubuntu-latest
1418
container:
1519
image: fedora:41
16-
env:
17-
PROJECT_NAME: ${{ github.repository }}
18-
BUILD_WORKSPACE: build
19-
PARALLEL: 4
2020
steps:
2121
- name: Install Dependencies
2222
id: install-dependencies

.github/workflows/cppcheck.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@
22

33
name: cppcheck
44
on: [workflow_call]
5+
env:
6+
PROJECT_NAME: ${{ github.repository }}
7+
BUILD_WORKSPACE: build
8+
PARALLEL: 1
59
jobs:
610
cppcheck-run:
711
runs-on: ubuntu-latest
812
container:
913
image: fedora:41
10-
env:
11-
PROJECT_NAME: ${{ github.repository }}
12-
BUILD_WORKSPACE: build
13-
PARALLEL: 1
1414
steps:
1515
- name: pepare-git
1616
run: |
@@ -47,4 +47,4 @@ jobs:
4747
uses: actions/upload-artifact@v4
4848
with:
4949
name: cppcheck
50-
path: $BUILD_WORKSPACE/cppcheck_report.json
50+
path: ${{ github.workspace }}/${{ env.BUILD_WORKSPACE }}/cppcheck_report.json

.github/workflows/doxygen.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,14 @@
22

33
name: doxygen
44
on: [workflow_call]
5+
env:
6+
PROJECT_NAME: ${{ github.repository }}
7+
BUILD_WORKSPACE: build
58
jobs:
69
build-doxygen:
710
runs-on: ubuntu-latest
811
container:
912
image: fedora:41
10-
env:
11-
PROJECT_NAME: ${{ github.repository }}
12-
BUILD_WORKSPACE: build
1313
steps:
1414
- name: pepare-git
1515
run: |
@@ -32,7 +32,7 @@ jobs:
3232
uses: actions/upload-artifact@v4
3333
with:
3434
name: doxygen
35-
path: ${{ github.workspace }}/${{ env.BUILD_WORKSPACE }}/doxygen.tar.gz
35+
path: $BUILD_WORKSPACE/doxygen.tar.gz
3636
- name: doxygen-test
3737
id: doxygen-test
3838
run: |

.github/workflows/rpm.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@
22

33
name: build-rpm
44
on: [workflow_call]
5+
env:
6+
PROJECT_NAME: ${{ github.repository }}
7+
BUILD_WORKSPACE: build
58
jobs:
69
rpm:
710
runs-on: ubuntu-latest
811
container:
912
image: fedora:41
10-
env:
11-
PROJECT_NAME: ${{ github.repository }}
12-
BUILD_WORKSPACE: build
13+
1314
steps:
1415
- name: Install Dependencies
1516
id: install-dependencies

.github/workflows/tests.yaml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,16 @@ on:
88
description: Parameters passed to ctest
99
type: string
1010
default: --parallel 16 --schedule-random
11+
env:
12+
PROJECT_NAME: ${{ github.repository }}
13+
BUILD_WORKSPACE: build
14+
CTEST: ${{ inputs.TEST_PARAMS }}
1115
jobs:
1216
ctest:
1317
runs-on: ubuntu-latest
1418
container:
1519
image: fedora:41
16-
env:
17-
PROJECT_NAME: ${{ github.repository }}
18-
BUILD_WORKSPACE: build
19-
CTEST: ${{ inputs.TEST_PARAMS }}
20+
2021
steps:
2122
- name: Install Dependencies
2223
id: install-dependencies

0 commit comments

Comments
 (0)