commit 7799afe248112b689625135db0d1fb290f50d4c8
author Christopher Fujino <christopherfujino@gmail.com> Wed Apr 03 12:30:34 2019 -0700
committer GitHub <noreply@github.com> Wed Apr 03 12:30:34 2019 -0700
tree d41f5aa77039795f8a1d1707ac839ea80558ab80
parent 194babfb4233dda73690dbb065c0ea20cf14bdc4

make shellcheck (linter) recommended changes to bin/flutter (#30456)

bin/flutter

diff --git a/bin/flutter b/bin/flutter
index bd33fc4..18bad48 100755
--- a/bin/flutter
+++ b/bin/flutter
@@ -41,11 +41,11 @@
 
 function retry_upgrade {
   local total_tries="10"
-  local remaining_tries=$(($total_tries - 1))
-  while [[ "$remaining_tries" > 0 ]]; do
+  local remaining_tries=$((total_tries - 1))
+  while [[ "$remaining_tries" -gt 0 ]]; do
     (cd "$FLUTTER_TOOLS_DIR" && "$PUB" upgrade "$VERBOSITY") && break
     echo "Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... ($remaining_tries tries left)"
-    remaining_tries=$(($remaining_tries - 1))
+    remaining_tries=$((remaining_tries - 1))
     sleep 5
   done
 
@@ -102,6 +102,12 @@
   fi
 
   local revision="$(cd "$FLUTTER_ROOT"; git rev-parse HEAD)"
+
+  # Invalidate cache if:
+  #  * SNAPSHOT_PATH is not a file, or
+  #  * STAMP_PATH is not a file with nonzero size, or
+  #  * Contents of STAMP_PATH is not our local git HEAD revision, or
+  #  * pubspec.yaml last modified after pubspec.lock
   if [[ ! -f "$SNAPSHOT_PATH" || ! -s "$STAMP_PATH" || "$(cat "$STAMP_PATH")" != "$revision" || "$FLUTTER_TOOLS_DIR/pubspec.yaml" -nt "$FLUTTER_TOOLS_DIR/pubspec.lock" ]]; then
     rm -f "$FLUTTER_ROOT/version"
     touch "$FLUTTER_ROOT/bin/cache/.dartignore"