name: test size on: [pull_request] # paths: # - "*" # - "!README.md" <-- don't rebuild on doc change concurrency: group: ci-${{github.workflow}}-${{ github.ref }} cancel-in-progress: true jobs: build: runs-on: ubuntu-20.04 container: ardupilot/ardupilot-dev-${{ matrix.toolchain }}:latest strategy: fail-fast: false # don't cancel if a job from the matrix fails matrix: toolchain: [ chibios, ] config: [ Durandal, MatekF405, Pixhawk1-1M, Hitec-Airspeed, # see special code for Periph below (3 places!) f103-GPS # see special code for Periph below (3 places!) ] include: - config: disco toolchain: armhf exclude: - config: disco toolchain: chibios steps: - uses: actions/checkout@v3 with: ref: ${{ github.event.pull_request.base.ref }} path: base_branch submodules: 'recursive' # Put ccache into github cache for faster build - name: Prepare ccache timestamp id: ccache_cache_timestamp run: | NOW=$(date -u +"%F-%T") echo "{timestamp}=${NOW}" >> $GITHUB_OUTPUT - name: ccache cache files uses: actions/cache@v3 with: path: ~/.ccache key: ${{github.workflow}}-ccache-${{ matrix.toolchain }}-${{steps.ccache_cache_timestamp.outputs.timestamp}} restore-keys: ${{github.workflow}}-ccache-${{ matrix.toolchain }}- # restore ccache from either previous build on this branch or on base branch - name: setup ccache run: | . base_branch/.github/workflows/ccache.env - name: Build ${{ github.event.pull_request.base.ref }} ${{matrix.config}} ${{ matrix.toolchain }} env: CI_BUILD_TARGET: ${{matrix.config}} shell: bash run: | git config --global --add safe.directory ${GITHUB_WORKSPACE} PATH="/github/home/.local/bin:$PATH" cd base_branch ./waf configure --board ${{matrix.config}} if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then ./waf AP_Periph else ./waf fi mkdir -p $GITHUB_WORKSPACE/base_branch_bin cp -r build/${{matrix.config}}/bin/* $GITHUB_WORKSPACE/base_branch_bin/ # build a set of binaries without symbols so we can check if # the binaries have changed. echo [`date`] Building ${{ github.event.pull_request.base.ref }} with no versions NO_VERSIONS_DIR="$GITHUB_WORKSPACE/base_branch_bin_no_versions" mkdir "$NO_VERSIONS_DIR" # export some environment variables designed to get # repeatable builds from the same source: export CHIBIOS_GIT_VERSION="12345678" export GIT_VERSION="abcdef" export GIT_VERSION_INT="15" if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then ./waf AP_Periph else ./waf fi cp -r build/${{matrix.config}}/bin/* "$NO_VERSIONS_DIR" echo [`date`] Built ${{ github.event.pull_request.base.ref }} with no versions - uses: actions/checkout@v3 with: fetch-depth: 0 path: 'pr' - name: Build PR rebased ${{matrix.config}} ${{ matrix.toolchain }} env: CI_BUILD_TARGET: ${{matrix.config}} shell: bash run: | git config --global --add safe.directory ${GITHUB_WORKSPACE} PATH="/github/home/.local/bin:$PATH" cd pr/ git config user.email "ardupilot-ci@ardupilot.org" git config user.name "ArduPilot CI" git remote add ardupilot https://github.com/ArduPilot/ardupilot.git git fetch --no-tags --prune --progress ardupilot ${{ github.event.pull_request.base.ref }} git rebase ardupilot/${{ github.event.pull_request.base.ref }} git submodule update --init --recursive --depth=1 ./waf configure --board ${{matrix.config}} if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then ./waf AP_Periph else ./waf fi mkdir $GITHUB_WORKSPACE/pr_bin cp -r build/${{matrix.config}}/bin/* $GITHUB_WORKSPACE/pr_bin/ # build a set of binaries without symbols so we can check if # the binaries have changed. echo [`date`] Building PR with no versions NO_VERSIONS_DIR="$GITHUB_WORKSPACE/pr_bin_no_versions" mkdir "$NO_VERSIONS_DIR" if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" GIT_VERSION_INT="15" ./waf AP_Periph else CHIBIOS_GIT_VERSION="12345678" GIT_VERSION="abcdef" GIT_VERSION_INT="15" ./waf fi cp -r build/${{matrix.config}}/bin/* "$NO_VERSIONS_DIR" echo [`date`] Built PR with no versions # build MatekF405 Plane without quadplane if [ "${{matrix.config}}" = "MatekF405" ]; then PLANE_BINARY="build/MatekF405/bin/arduplane.bin" echo "normal size" ls -l "$PLANE_BINARY" EXTRA_HWDEF="/tmp/extra-options.def" echo "define HAL_QUADPLANE_ENABLED 0" >"$EXTRA_HWDEF" ./waf configure --board ${{matrix.config}} --extra-hwdef="$EXTRA_HWDEF" ./waf plane rm "$EXTRA_HWDEF" echo "non-quadplane size:" ls -l "$PLANE_BINARY" fi - name: Full size compare with base branch shell: bash run: | cd pr/ Tools/scripts/pretty_diff_size.py -m $GITHUB_WORKSPACE/base_branch_bin -s $GITHUB_WORKSPACE/pr_bin - name: Feature compare with ${{ github.event.pull_request.base.ref }} shell: bash run: | set -ex cd pr/ BIN_PREFIX="arm-none-eabi-" if [ "${{matrix.toolchain}}" = "armhf" ]; then BIN_PREFIX="arm-linux-gnueabihf-" fi if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then EF_BINARY_NAME="AP_Periph" else EF_BINARY_NAME="arduplane" fi EF_BASE_BRANCH_BINARY="$GITHUB_WORKSPACE/base_branch_bin/$EF_BINARY_NAME" EF_PR_BRANCH_BINARY="$GITHUB_WORKSPACE/pr_bin/$EF_BINARY_NAME" Tools/scripts/extract_features.py "$EF_BASE_BRANCH_BINARY" -nm "${BIN_PREFIX}nm" >features-base_branch.txt Tools/scripts/extract_features.py "$EF_PR_BRANCH_BINARY" -nm "${BIN_PREFIX}nm" >features-pr.txt diff -u features-base_branch.txt features-pr.txt || true - name: Checksum compare with ${{ github.event.pull_request.base.ref }} shell: bash run: | diff -r $GITHUB_WORKSPACE/base_branch_bin_no_versions $GITHUB_WORKSPACE/pr_bin_no_versions --exclude=*.elf --exclude=*.apj || true - name: elf_diff compare with ${{ github.event.pull_request.base.ref }} shell: bash run: | # we don't use weasyprint so manually pull the elf_diff deps reduce install size and time python3 -m pip install -U --no-deps elf_diff GitPython Jinja2 MarkupSafe PyYAML anytree dict2xml gitdb progressbar2 python-utils setuptools-git smmap mkdir elf_diff BIN_PREFIX="arm-none-eabi-" if [ "${{ matrix.toolchain }}" = "armhf" ]; then BIN_PREFIX="arm-linux-gnueabihf-" fi if [ "${{matrix.config}}" = "Hitec-Airspeed" ] || [ "${{matrix.config}}" = "f103-GPS" ]; then python3 -m elf_diff --bin_prefix="$BIN_PREFIX" --html_dir=elf_diff/AP_Periph $GITHUB_WORKSPACE/base_branch_bin/AP_Periph $GITHUB_WORKSPACE/pr_bin/AP_Periph else python3 -m elf_diff --bin_prefix="$BIN_PREFIX" --html_dir=elf_diff/plane $GITHUB_WORKSPACE/base_branch_bin/arduplane $GITHUB_WORKSPACE/pr_bin/arduplane python3 -m elf_diff --bin_prefix="$BIN_PREFIX" --html_dir=elf_diff/copter $GITHUB_WORKSPACE/base_branch_bin/arducopter $GITHUB_WORKSPACE/pr_bin/arducopter fi zip -r elf_diff.zip elf_diff - name: Archive elf_diff output uses: actions/upload-artifact@v3 with: name: ELF_DIFF_${{matrix.config}} path: elf_diff.zip retention-days: 14