Try and fix mpy-cross variants

tannewt · tannewt · commit 22a44c6003d9 · 2023-10-11T10:03:46.000-07:00
diff --git a/.github/actions/mpy_cross/action.yml b/.github/actions/mpy_cross/action.yml
@@ -5,6 +5,9 @@ inputs:
     required: false
     default: true
     type: boolean
+  cp-version:
+    required: true
+    type: string
 
 runs:
   using: composite
@@ -27,11 +30,13 @@ runs:
       if: inputs.download == 'false' || steps.download-mpy-cross.outcome == 'failure'
       run: make -C mpy-cross -j2
       shell: bash
+      env:
+        CP_VERSION: ${{ inputs.cp-version }}
 
     - name: Upload mpy-cross
       if: inputs.download == 'false' || steps.download-mpy-cross.outcome == 'failure'
       continue-on-error: true
       uses: actions/upload-artifact@v3
       with:
         name: mpy-cross
-        path: mpy-cross/mpy-cross
+        path: mpy-cross/build/mpy-cross
diff --git a/.github/workflows/build-boards.yml b/.github/workflows/build-boards.yml
@@ -49,6 +49,8 @@ jobs:
     - name: Set up mpy-cross
       if: steps.set-up-submodules.outputs.frozen == 'True'
       uses: ./.github/actions/mpy_cross
+      with:
+        cp-version: ${{ inputs.cp-version }}
 
     - name: Versions
       run: |
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -51,6 +51,7 @@ jobs:
     - name: Set up mpy-cross
       uses: ./.github/actions/mpy_cross
       with:
+        cp-version: ${{ steps.set-up-submodules.outputs.version }}
         download: false
     - name: Get last commit with checks
       id: get-last-commit-with-checks
diff --git a/.github/workflows/custom-board-build.yml b/.github/workflows/custom-board-build.yml
@@ -65,6 +65,7 @@ jobs:
       if: steps.set-up-submodules.outputs.frozen == 'True'
       uses: ./.github/actions/mpy_cross
       with:
+        cp-version: ${{ steps.set-up-submodules.outputs.version }}
         download: false
     - name: Versions
       run: |
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -17,7 +17,7 @@ jobs:
     env:
       CP_VERSION: ${{ inputs.cp-version }}
       MICROPY_CPYTHON3: python3.8
-      MICROPY_MICROPYTHON: ../ports/unix/micropython-coverage
+      MICROPY_MICROPYTHON: ../ports/unix/build-coverage/micropython
       TEST_all:
       TEST_mpy: --via-mpy -d basics float micropython
       TEST_native: --emit native
@@ -41,6 +41,8 @@ jobs:
       uses: ./.github/actions/deps/external
     - name: Set up mpy-cross
       uses: ./.github/actions/mpy_cross
+      with:
+        cp-version: ${{ inputs.cp-version }}
     - name: Build unix port
       run: make -C ports/unix VARIANT=coverage -j2
     - name: Run tests
diff --git a/mpy-cross/Makefile b/mpy-cross/Makefile
@@ -1,7 +1,7 @@
 include ../py/mkenv.mk
 
 # define main target
-PROG = mpy-cross
+PROG ?= mpy-cross
 
 # qstr definitions (must come before including py.mk)
 QSTR_DEFS = qstrdefsport.h
diff --git a/mpy-cross/Makefile.fuzz b/mpy-cross/Makefile.fuzz
@@ -6,4 +6,4 @@ PROG=mpy-cross.fuzz
 BUILD=build-static
 STATIC_BUILD=1
 CC=afl-clang-fast
-include mpy-cross.mk
+include Makefile
diff --git a/mpy-cross/Makefile.m1 b/mpy-cross/Makefile.m1
@@ -5,6 +5,6 @@
 PROG=mpy-cross-arm64
 BUILD=build-arm64
 
-include mpy-cross.mk
+include Makefile
 # Because mpy-cross.mk unconditionally overwrites CC for Darwin, we must set it BELOW the inclusion
 CC := $(shell xcrun --find clang) -isysroot $(shell xcrun --show-sdk-path) -target arm64-apple-macos11
diff --git a/mpy-cross/Makefile.static b/mpy-cross/Makefile.static
@@ -6,4 +6,4 @@ PROG=mpy-cross.static
 BUILD=build-static
 STATIC_BUILD=1
 
-include mpy-cross.mk
+include Makefile
diff --git a/mpy-cross/Makefile.static-aarch64 b/mpy-cross/Makefile.static-aarch64
@@ -7,4 +7,4 @@ BUILD=build-static-aarch64
 STATIC_BUILD=1
 
 CROSS_COMPILE = aarch64-linux-gnu-
-include mpy-cross.mk
+include Makefile
diff --git a/mpy-cross/Makefile.static-mingw b/mpy-cross/Makefile.static-mingw
@@ -7,4 +7,4 @@ CROSS_COMPILE = x86_64-w64-mingw32-
 BUILD=build-static-mingw
 STATIC_BUILD=1
 
-include mpy-cross.mk
+include Makefile
diff --git a/mpy-cross/Makefile.static-raspbian b/mpy-cross/Makefile.static-raspbian
@@ -8,4 +8,4 @@ STATIC_BUILD=1
 
 $(shell if ! [ -d pitools ]; then echo 1>&2 "Fetching pi build tools.  This may take awhile."; git clone -q https://github.com/raspberrypi/tools.git --depth=1 pitools; fi)
 CROSS_COMPILE = pitools/arm-bcm2708/gcc-linaro-arm-linux-gnueabihf-raspbian-x64/bin/arm-linux-gnueabihf-
-include mpy-cross.mk
+include Makefile
diff --git a/mpy-cross/mpy-cross.mk b/mpy-cross/mpy-cross.mk
diff --git a/py/makeversionhdr.py b/py/makeversionhdr.py
@@ -13,8 +13,9 @@
 import datetime
 import subprocess
 
-# CIRCUITPY: use external script
-tools_describe = str(pathlib.Path(__file__).parent.parent / "tools/describe")
+# CIRCUITPY: use external script that can override git describe output with an
+# environment variable.
+tools_describe = str(pathlib.Path(__file__).resolve().parent.parent / "tools/describe")
 
 
 def get_version_info_from_git(repo_path):
@@ -27,6 +28,7 @@ def get_version_info_from_git(repo_path):
 
     # Note: git describe doesn't work if no tag is available
     try:
+        print(tools_describe)
         git_tag = subprocess.check_output(
             [tools_describe],
             cwd=repo_path,
@@ -93,8 +95,10 @@ def make_version_header(repo_path, filename):
     # Get version info using git (required)
     info = get_version_info_from_git(repo_path)
     if info is None:
+        print(info)
         cannot_determine_version()
     git_tag, git_hash, ver = info
+    print(git_tag, git_hash, ver)
     if len(ver) < 3:
         cannot_determine_version()
     else:
