fix

Author: lucasssvaz

.github/workflows/cron.yml

@@ -15,13 +15,13 @@ on:
   workflow_dispatch:
     inputs:
       idf_branch:
-        # This input can be an escaped JSON list of strings e.g. [\"release/v5.1\"] or "all" to build all branches.
+        # This input can be a comma-separated list of strings (e.g. "release/v5.1,release/v5.3") or "all" to build all branches.
         # For IDF versions before v5.1, "all" is not supported. Use a specific branch instead.
         description: 'IDF branch to build (check workflow file for instructions on how to use this input)'
         required: true
         default: 'all'
       target:
-        # This input can be an escaped JSON list of strings e.g. [\"esp32\", \"esp32s2\"] or "all" to build all targets.
+        # This input can be a comma-separated list of strings (e.g. "esp32,esp32s2") or "all" to build all targets.
         # For IDF versions before v5.1, "all" is not supported. Use a specific target list instead.
         description: 'Target to build (check workflow file for instructions on how to use this input)'
         required: true
@@ -39,53 +39,11 @@ jobs:
       matrix: ${{ steps.gen-matrix.outputs.matrix }}
       branches: ${{ steps.gen-matrix.outputs.branches }}
     steps:
+      - uses: actions/checkout@v4
       - name: Generate matrix
         id: gen-matrix
         run: |
-          # Change this based on the IDF branches we want to build
-          all_branches="[\"release/v5.1\"]"
-
-          # Change this based on the COMMON targets for all branches we want to build.
-          common_targets="[\"esp32\", \"esp32s2\", \"esp32s3\", \"esp32c2\", \"esp32c3\", \"esp32c6\", \"esp32h2\"]"
-
-          # For additional targets per branch, add them here
-          additional_targets="[{\"idf_branch\": \"release/v5.3\", \"target\": \"esp32p4\"}]"
-
-          branches="${{ github.event.inputs.idf_branch }}"
-          targets="${{ github.event.inputs.target }}"
-
-          if [ -z "$branches" ] || [ "$branches" == "all" ]; then
-            branches=$all_branches
-          fi
-
-          if [ -z "$targets" ]; then
-            targets="all"
-          fi
-
-          echo "Inputs:"
-          echo "idf_branch: ${{ github.event.inputs.idf_branch }}"
-          echo "target: ${{ github.event.inputs.target }}"
-
-          matrix="{"
-          matrix+="\"idf_branch\": $branches,"
-
-          if [ "$targets" == "all" ]; then
-            matrix+="\"target\": $common_targets,"
-            matrix+="\"include\": "
-            # Add all additional targets that are in the selected branches
-            matrix+=$(echo $additional_targets | jq --argjson branches "$branches" '[.[] | select(.idf_branch as $branch | $branches | index($branch))]')
-          else
-            matrix+="\"target\": $targets"
-          fi
-
-          matrix+="}"
-
-          echo "Branches: $branches"
-          echo "Targets: $targets"
-          echo "Matrix:"
-          echo "$matrix" | jq .
-          echo "matrix=$matrix" >> $GITHUB_OUTPUT
-          echo "branches=$branches" >> $GITHUB_OUTPUT
+          bash ./tools/cron-gen-matrix.sh "${{ github.event.inputs.idf_branch }}" "${{ github.event.inputs.target }}"
 
   build-libs:
     name: Build with IDF ${{ matrix.idf_branch }} for ${{ matrix.target }}

tools/cron-gen-matrix.sh

@@ -0,0 +1,66 @@
+#!/bin/bash
+
+set -e
+
+if [ ! $# -eq 2 ]; then
+	echo "Bad number of arguments: $#" >&2
+	echo "Usage: $0 <branches> <targets>" >&2
+	exit 1
+fi
+
+if ! [ -x "$(command -v jq)" ]; then
+    echo "ERROR: jq is not installed! Please install jq first."
+    exit 1
+fi
+
+if ! [ -x "$(command -v sed)" ]; then
+    echo "ERROR: sed is not installed! Please install sed first."
+    exit 1
+fi
+
+echo "Inputs:"
+echo "idf_branch: $1"
+echo "target: $2"
+
+# Change this based on the IDF branches we want to build
+all_branches="[\"release/v5.1\"]"
+
+# Change this based on the COMMON targets for all branches we want to build.
+common_targets="[\"esp32\", \"esp32s2\", \"esp32s3\", \"esp32c2\", \"esp32c3\", \"esp32c6\", \"esp32h2\"]"
+
+# For additional targets per branch, add them here
+additional_targets="[{\"idf_branch\": \"release/v5.3\", \"target\": \"esp32p4\"}]"
+
+if [ -z "$1" ] || [ "$1" == "all" ]; then
+    branches=$all_branches
+else
+    branches=$(echo "$1" | sed 's/ *, */,/g' | sed 's/^/["/' | sed 's/$/"]/' | sed 's/,/","/g')
+fi
+
+if [ -z "$2" ] || [ "$2" == "all" ]; then
+    targets="all"
+else
+    targets=$(echo "$2" | sed 's/ *, */,/g' | sed 's/^/["/' | sed 's/$/"]/' | sed 's/,/","/g')
+fi
+
+matrix="{"
+matrix+="\"idf_branch\": $branches,"
+
+if [ "$targets" == "all" ]; then
+    matrix+="\"target\": $common_targets,"
+    matrix+="\"include\": "
+    # Add all additional targets that are in the selected branches
+    matrix+=$(echo $additional_targets | jq --argjson branches "$branches" '[.[] | select(.idf_branch as $branch | $branches | index($branch))]')
+else
+    matrix+="\"target\": $targets"
+fi
+
+matrix+="}"
+
+echo "Matrix:"
+echo "$matrix" | jq .
+
+if [ ! -x $GITHUB_OUTPUT ]; then
+    echo "matrix=$matrix" >> $GITHUB_OUTPUT
+    echo "branches=$branches" >> $GITHUB_OUTPUT
+fi

tools/cron.sh

@@ -4,4 +4,6 @@ if [ -z "$TARGETS" ]; then
     TARGETS="all"
 fi
 
+export IDF_CCACHE_ENABLE=1
+
 bash ./build.sh -e -t $TARGETS