CI: check-kernel-patches: use buildbot user on git diff check
authorChristian Marangi <ansuelsmth@gmail.com>
Mon, 22 May 2023 13:27:48 +0000 (15:27 +0200)
committerChristian Marangi <ansuelsmth@gmail.com>
Mon, 22 May 2023 14:05:13 +0000 (16:05 +0200)
Use buildbot user on git diff check instead of using git config
safe directory.

This should accomplish the same result but should be a better approach
following safe practice enforced by git.

Fixes: a7747e8670cb ("ci: fix check kernel patches job")
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
.github/workflows/check-kernel-patches.yml

index 4ad35e625075e5abc6bd5a9be1fcac08ece294f8..274bae367ded566683afc5f6d377ef9192de36e0 100644 (file)
@@ -85,10 +85,6 @@ jobs:
         run: |
           chown -R buildbot:buildbot openwrt
 
-      - name: Opt-out from Git stricter repository ownership checks
-        run: |
-          git config --global --add safe.directory '*'
-
       - name: Initialization environment
         run: |
           TARGET=$(echo ${{ inputs.target }} | cut -d "/" -f 1)
@@ -138,6 +134,7 @@ jobs:
         run: make target/linux/refresh V=s
 
       - name: Validate Refreshed Kernel Patches
+        shell: su buildbot -c "sh -e {0}"
         working-directory: openwrt
         run: |
           . .github/workflows/scripts/ci_helpers.sh