update readme

Fix temp folder handling on windows

.github/workflows/workflow.yml

@@ -1,12 +1,19 @@
 name: Main workflow
-on: [push]
+on: 
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
 jobs:
   test_channel:
     runs-on: ${{ matrix.operating-system }}
     strategy:
       matrix:
         operating-system: [ubuntu-latest, windows-latest, macos-latest]
-        channel: [stable, beta, dev]
+        channel: [stable, beta]
     steps:
     - name: Checkout
       uses: actions/checkout@v2
@@ -72,3 +79,24 @@ jobs:
     - name: Run flutter --version
       shell: bash
       run: flutter --version
+  test_cache:
+    runs-on: ${{ matrix.operating-system }}
+    strategy:
+      matrix:
+        operating-system: [ubuntu-latest, windows-latest, macos-latest]
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+    - uses: ./
+      with:
+        channel: stable
+        flutter-version: 2.5.0
+        cache: true
+        cache-key: key-20220113
+        cache-path: ${{ runner.tool_cache }}/flutter/2.5.0-stable
+    - name: Run dart --version
+      shell: bash
+      run: dart --version
+    - name: Run flutter --version
+      shell: bash
+      run: flutter --version

README.md

@@ -1,8 +1,8 @@
 # flutter-action
 
-This action sets up a flutter environment for use in actions. It works on Linux, Windows, and macOS.
+Flutter environment for use in GitHub Actions. It works on Linux, Windows, and macOS.
 
-# Usage
+## Usage
 
 Use specific version and channel:
 
@@ -23,7 +23,7 @@ steps:
 - uses: actions/checkout@v2
 - uses: subosito/flutter-action@v2
   with:
-    channel: 'stable' # or: 'beta', 'dev' or 'master'
+    channel: 'stable' # or: 'beta' or 'master'
 - run: flutter --version
 ```
 
@@ -108,27 +108,55 @@ jobs:
      - uses: actions/checkout@v2
      - uses: subosito/flutter-action@v2
        with:
-         channel: beta
+         channel: 'beta'
      - run: flutter config --enable-windows-desktop
      - run: flutter build windows
 ```
 
-Matrix Testing:
+Build for Linux desktop:
 
 ```yaml
 jobs:
-  test:
-    name: Test on ${{ matrix.os }}
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
-    steps:
-      - uses: actions/checkout@v2
-      - uses: subosito/flutter-action@v2
-        with:
-          flutter-version: '1.20.2'
-          channel: 'beta'
-      - run: dart --version
-      - run: flutter --version
+ build:
+   runs-on: ubuntu-latest
+   steps:
+     - uses: actions/checkout@v2
+     - uses: subosito/flutter-action@v2
+       with:
+         channel: 'beta'
+     - run: |
+        sudo apt-get update -y
+        sudo apt-get install -y ninja-build libgtk-3-dev
+     - run: flutter config --enable-linux-desktop
+     - run: flutter build linux
+```
+
+Build for macOS desktop:
+
+```yaml
+jobs:
+ build:
+   runs-on: macos-latest
+   steps:
+     - uses: actions/checkout@v2
+     - uses: subosito/flutter-action@v2
+       with:
+         channel: 'beta'
+     - run: flutter config --enable-macos-desktop
+     - run: flutter build macos
+```
+
+Integration with `actions/cache`:
+
+```yaml
+steps:
+- uses: actions/checkout@v2
+- uses: subosito/flutter-action@v2
+  with:
+    flutter-version: '2.5.0'
+    channel: 'stable'
+    cache: true
+    cache-key: flutter # optional, change this to force refresh cache
+    cache-path: ${{ runner.tool_cache }}/flutter # optional, change this to specify the cache path
+- run: flutter --version
 ```

action.yml

@@ -12,8 +12,24 @@ inputs:
     description: 'The Flutter build release channel'
     required: false
     default: 'stable'
+  cache:
+    description: 'Cache the Flutter SDK'
+    required: false
+  cache-key:
+    description: 'Identifier for the Flutter SDK cache'
+    required: false
+    default: 'flutter'
+  cache-path:
+    description: 'Flutter SDK cache path'
+    required: false
+    default: ${{ runner.tool_cache }}/flutter
 runs:
   using: 'composite'
   steps:
-    - run: $GITHUB_ACTION_PATH/setup.sh ${{ inputs.channel }} ${{ inputs.flutter-version }}
+    - if: ${{ inputs.cache == 'true' }}
+      uses: actions/cache@v2
+      with:
+        path: ${{ inputs.cache-path }}
+        key: ${{ inputs.cache-key }}-${{ inputs.channel }}-${{ inputs.flutter-version }}
+    - run: $GITHUB_ACTION_PATH/setup.sh -c "${{ inputs.cache-path }}" ${{ inputs.channel }} ${{ inputs.flutter-version }}
       shell: bash

setup.sh

@@ -48,47 +48,73 @@ get_version_manifest() {
 download_archive() {
   archive_url="$MANIFEST_BASE_URL/$1"
   archive_name=$(basename $1)
-  archive_local="$HOME/$archive_name"
+  archive_local="$RUNNER_TEMP/$archive_name"
 
   curl --connect-timeout 15 --retry 5 $archive_url >$archive_local
 
+  # Create the target folder
+  mkdir -p "$2"
+
   if [[ $archive_name == *zip ]]; then
-    unzip -o "$archive_local" -d "$2"
+    unzip -q -o "$archive_local" -d "$RUNNER_TEMP"
+    # Remove the folder again so that the move command can do a simple rename
+    # instead of moving the content into the target folder.
+    # This is a little bit of a hack since the "mv --no-target-directory"
+    # linux option is not available here
+    rm -r "$2"
+    mv ${RUNNER_TEMP}/flutter "$2"
   else
-    tar xf "$archive_local" -C "$2"
+    tar xf "$archive_local" -C "$2" --strip-components=1
   fi
 
   rm $archive_local
 }
 
-CHANNEL="$1"
-VERSION="$2"
-
-if [[ $CHANNEL == master ]]; then
-  git clone -b master https://github.com/flutter/flutter.git "$RUNNER_TOOL_CACHE/flutter"
-else
-  VERSION_MANIFEST=$(get_version_manifest $CHANNEL $VERSION)
-
-  if [[ $VERSION_MANIFEST == null ]]; then
-    echo "Unable to determine Flutter version for $CHANNEL $VERSION"
-    exit 1
+transform_path() {
+  if [[ $OS_NAME == windows ]]; then
+    echo $1 | sed -e 's/^\///' -e 's/\//\\/g'
+  else
+    echo $1
   fi
+}
 
-  ARCHIVE_PATH=$(echo $VERSION_MANIFEST | jq -r '.archive')
-  download_archive "$ARCHIVE_PATH" "$RUNNER_TOOL_CACHE"
-fi
+SDK_CACHE=""
+
+while getopts 'c:' flag; do
+  case "${flag}" in
+  c) SDK_CACHE="$(transform_path $OPTARG)" ;;
+  ?) exit 2 ;;
+  esac
+done
+
+CHANNEL="${@:$OPTIND:1}"
+VERSION="${@:$OPTIND+1:1}"
 
 if [[ $OS_NAME == windows ]]; then
-  FLUTTER_ROOT="${RUNNER_TOOL_CACHE}\\flutter"
   PUB_CACHE="${USERPROFILE}\\.pub-cache"
 else
-  FLUTTER_ROOT="${RUNNER_TOOL_CACHE}/flutter"
   PUB_CACHE="${HOME}/.pub-cache"
 fi
 
-echo "FLUTTER_ROOT=${FLUTTER_ROOT}" >>$GITHUB_ENV
+if [[ ! -x "${SDK_CACHE}/bin/flutter" ]]; then
+  if [[ $CHANNEL == master ]]; then
+    git clone -b master https://github.com/flutter/flutter.git "$SDK_CACHE"
+  else
+    VERSION_MANIFEST=$(get_version_manifest $CHANNEL $VERSION)
+
+    if [[ $VERSION_MANIFEST == null ]]; then
+      echo "Unable to determine Flutter version for $CHANNEL $VERSION"
+      exit 1
+    fi
+
+    ARCHIVE_PATH=$(echo $VERSION_MANIFEST | jq -r '.archive')
+    download_archive "$ARCHIVE_PATH" "$SDK_CACHE"
+  fi
+fi
+
+echo "FLUTTER_ROOT=${SDK_CACHE}" >>$GITHUB_ENV
 echo "PUB_CACHE=${PUB_CACHE}" >>$GITHUB_ENV
 
-echo "${FLUTTER_ROOT}/bin" >>$GITHUB_PATH
-echo "${FLUTTER_ROOT}/bin/cache/dart-sdk/bin" >>$GITHUB_PATH
+echo "${SDK_CACHE}/bin" >>$GITHUB_PATH
+echo "${SDK_CACHE}/bin/cache/dart-sdk/bin" >>$GITHUB_PATH
 echo "${PUB_CACHE}/bin" >>$GITHUB_PATH