From 141071a4142d8dd9f7ecaa8d77018a7e591a668f Mon Sep 17 00:00:00 2001 From: rbaron Date: Sat, 21 Oct 2023 17:21:39 +0200 Subject: [PATCH] Refactor github actions Reason: https://github.com/NordicPlayground/nrf-docker/pull/75 --- .github/actions/build/Dockerfile | 5 --- .github/actions/build/action.yml | 41 ++++++++++++--------- .github/actions/build/build.sh | 21 ----------- .github/actions/build_and_upload/action.yml | 20 +++++----- .github/workflows/b-parasite.yml | 12 +++--- 5 files changed, 40 insertions(+), 59 deletions(-) delete mode 100644 .github/actions/build/Dockerfile delete mode 100755 .github/actions/build/build.sh diff --git a/.github/actions/build/Dockerfile b/.github/actions/build/Dockerfile deleted file mode 100644 index 865cbbf..0000000 --- a/.github/actions/build/Dockerfile +++ /dev/null @@ -1,5 +0,0 @@ -FROM nordicplayground/nrfconnect-sdk:v2.4-branch - -COPY build.sh /build.sh - -ENTRYPOINT ["/build.sh"] \ No newline at end of file diff --git a/.github/actions/build/action.yml b/.github/actions/build/action.yml index 1130965..bd5c3de 100644 --- a/.github/actions/build/action.yml +++ b/.github/actions/build/action.yml @@ -1,27 +1,34 @@ -name: 'Build' -description: 'Builds a nrf-connect sample for b-parasite' +name: "Build" +description: "Builds a nrf-connect sample for b-parasite" inputs: sample-dir: - description: 'Sample directory to build' + description: "Sample directory to build" required: true board: - description: 'Board definition to use' + description: "Board definition to use" default: bparasite_nrf52840 revision: - description: 'Board revision use' - default: '1.2.0' + description: "Board revision use" + default: "2.0.0" cmake-extra: - description: 'Extra CMake arguments' - default: '' + description: "Extra CMake arguments" + default: "" output-bin: - description: 'Name of the .hex output' + description: "Name of the .hex output" required: true runs: - using: 'docker' - image: 'Dockerfile' - args: - - ${{ inputs.sample-dir }} - - ${{ inputs.board }} - - ${{ inputs.revision }} - - ${{ inputs.cmake-extra }} - - ${{ inputs.output-bin }} + using: "composite" + steps: + - run: | + docker run --rm -v ${GITHUB_WORKSPACE}:/repo \ + nordicplayground/nrfconnect-sdk:v2.4-branch \ + west build \ + --build-dir /repo/${{ inputs.sample-dir }}/build \ + --pristine \ + --board ${{ inputs.board }}@${{ inputs.revision }} \ + /repo/${{ inputs.sample-dir }} -- ${{ inputs.cmake-extra }} + shell: bash + - run: | + cp "${GITHUB_WORKSPACE}"/${{ inputs.sample-dir }}/build/zephyr/zephyr.hex \ + "${GITHUB_WORKSPACE}"/${{ inputs.sample-dir }}/build/zephyr/${{ inputs.output-bin }} + shell: bash diff --git a/.github/actions/build/build.sh b/.github/actions/build/build.sh deleted file mode 100755 index b32fc87..0000000 --- a/.github/actions/build/build.sh +++ /dev/null @@ -1,21 +0,0 @@ -#!/bin/bash -set -eux -o pipefail - -SAMPLE_DIR=$1 -BOARD=$2 -REVISION=$3 -CMAKE_EXTRA=$4 -OUTPUT_BIN=$5 - -TODAY=$(date +'%Y-%m-%d') - -# Replaces occurrences of "__TODAY__" with $TODAY in $CMAKE_EXTRA. -CMAKE_EXTRA="${CMAKE_EXTRA/__TODAY__/"$TODAY"}" - -cd "${GITHUB_WORKSPACE}/${SAMPLE_DIR}" - -echo $CMAKE_EXTRA - -west build --build-dir ./build --pristine --board "${BOARD}@${REVISION}" -- $CMAKE_EXTRA - -mv build/zephyr/zephyr.hex build/zephyr/"${OUTPUT_BIN}" \ No newline at end of file diff --git a/.github/actions/build_and_upload/action.yml b/.github/actions/build_and_upload/action.yml index 7939a8f..be2b199 100644 --- a/.github/actions/build_and_upload/action.yml +++ b/.github/actions/build_and_upload/action.yml @@ -1,20 +1,20 @@ -name: 'Build and upload artifact' -description: 'Builds a nrf-connect sample for b-parasite' +name: "Build and upload artifact" +description: "Builds a nrf-connect sample for b-parasite" inputs: sample-dir: - description: 'Sample directory to build' + description: "Sample directory to build" required: true board: - description: 'Board definition to use' + description: "Board definition to use" required: true revision: - description: 'Board revision use' - default: '1.2.0' + description: "Board revision use" + default: "2.0.0" cmake-extra: - description: 'Extra CMake arguments' - default: '' + description: "Extra CMake arguments" + default: "" output-bin: - description: 'Name of the .hex output' + description: "Name of the .hex output" required: true runs: @@ -30,4 +30,4 @@ runs: - uses: actions/upload-artifact@v3 with: name: sample-binaries - path: ${{ inputs.sample-dir }}/build/zephyr/${{ inputs.output-bin }} \ No newline at end of file + path: ${{ inputs.sample-dir }}/build/zephyr/${{ inputs.output-bin }} diff --git a/.github/workflows/b-parasite.yml b/.github/workflows/b-parasite.yml index 4186cfe..8d20e2d 100644 --- a/.github/workflows/b-parasite.yml +++ b/.github/workflows/b-parasite.yml @@ -17,7 +17,7 @@ jobs: - name: Check clang-format uses: jidicula/clang-format-action@v4.9.0 with: - check-path: 'code/nrf-connect' + check-path: "code/nrf-connect" exclude-regex: '\/build\/' build-blinky: @@ -27,7 +27,7 @@ jobs: - lint steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Build blinky uses: ./.github/actions/build with: @@ -43,7 +43,7 @@ jobs: - lint steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Build input uses: ./.github/actions/build with: @@ -59,7 +59,7 @@ jobs: - lint steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Build soil_read_loop uses: ./.github/actions/build with: @@ -79,7 +79,7 @@ jobs: - lint steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Build uses: ./.github/actions/build_and_upload with: @@ -100,7 +100,7 @@ jobs: - lint steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Build uses: ./.github/actions/build_and_upload with: