diff options
| author |  Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-04-10 20:36:31 -0300 |
|---|---|---|
| committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-04-24 23:29:28 -0300 |
| commit | fd59df07de66a3e20038f2949ad760b77f9999b4 (patch) | |
| tree | f54ffb752f2db405c9fab3369ea797fef2330cb8 /.github/workflows/build-extra.yml | |
| parent | ci: simplify test steps in build.yml (diff) | |
| download | firejail-fd59df07de66a3e20038f2949ad760b77f9999b4.tar.gz firejail-fd59df07de66a3e20038f2949ad760b77f9999b4.tar.zst firejail-fd59df07de66a3e20038f2949ad760b77f9999b4.zip | |
ci: line-wrap and split/join some commands
For increased readability.
Note: `>` basically turns each newline into a space while `|` keeps
newlines as is. Both remove leading indentation.
Note2: On jobs using `apt-get install`, this commit moves package names
to their own line, to make it easier to compare which packages are being
installed across such jobs.
Diffstat (limited to '.github/workflows/build-extra.yml')
| -rw-r--r-- | .github/workflows/build-extra.yml | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/.github/workflows/build-extra.yml b/.github/workflows/build-extra.yml index 9db2ec41f..13c575c8b 100644 --- a/.github/workflows/build-extra.yml +++ b/.github/workflows/build-extra.yml | |||
| @@ -68,7 +68,9 @@ jobs: | |||
| 68 | - name: install dependencies | 68 | - name: install dependencies |
| 69 | run: sudo apt-get install libapparmor-dev libselinux1-dev | 69 | run: sudo apt-get install libapparmor-dev libselinux1-dev |
| 70 | - name: configure | 70 | - name: configure |
| 71 | run: CC=clang-14 ./configure --enable-fatal-warnings --enable-apparmor --enable-selinux | 71 | run: > |
| 72 | CC=clang-14 ./configure --enable-fatal-warnings --enable-apparmor | ||
| 73 | --enable-selinux | ||
| 72 | - name: make | 74 | - name: make |
| 73 | run: make | 75 | run: make |
| 74 | - name: make install | 76 | - name: make install |
| @@ -91,7 +93,9 @@ jobs: | |||
| 91 | - name: install clang-tools-14 and dependencies | 93 | - name: install clang-tools-14 and dependencies |
| 92 | run: sudo apt-get install clang-tools-14 libapparmor-dev libselinux1-dev | 94 | run: sudo apt-get install clang-tools-14 libapparmor-dev libselinux1-dev |
| 93 | - name: configure | 95 | - name: configure |
| 94 | run: CC=clang-14 ./configure --enable-fatal-warnings --enable-apparmor --enable-selinux | 96 | run: > |
| 97 | CC=clang-14 ./configure --enable-fatal-warnings --enable-apparmor | ||
| 98 | --enable-selinux | ||
| 95 | - name: scan-build | 99 | - name: scan-build |
| 96 | run: NO_EXTRA_CFLAGS="yes" scan-build-14 --status-bugs make | 100 | run: NO_EXTRA_CFLAGS="yes" scan-build-14 --status-bugs make |
| 97 | cppcheck: | 101 | cppcheck: |
| @@ -110,9 +114,11 @@ jobs: | |||
| 110 | - name: install cppcheck | 114 | - name: install cppcheck |
| 111 | run: sudo apt-get install cppcheck | 115 | run: sudo apt-get install cppcheck |
| 112 | - name: cppcheck | 116 | - name: cppcheck |
| 113 | run: cppcheck -q --force --error-exitcode=1 --enable=warning,performance -i src/firejail/checkcfg.c -i src/firejail/main.c . | 117 | run: > |
| 114 | # new cppcheck version currently chokes on checkcfg.c and main.c, therefore scan all files also | 118 | cppcheck -q --force --error-exitcode=1 --enable=warning,performance |
| 115 | # with older cppcheck version from ubuntu 20.04. | 119 | -i src/firejail/checkcfg.c -i src/firejail/main.c . |
| 120 | # new cppcheck version currently chokes on checkcfg.c and main.c, therefore | ||
| 121 | # scan all files also with older cppcheck version from ubuntu 20.04. | ||
| 116 | cppcheck_old: | 122 | cppcheck_old: |
| 117 | runs-on: ubuntu-20.04 | 123 | runs-on: ubuntu-20.04 |
| 118 | steps: | 124 | steps: |