Skip to content

Commit 5b3c462

Browse files
Emmanuel Garciaadsonpleal
Emmanuel Garcia
authored andcommitted
Add plugin tools as a git submodule and depend on it directly (flutter#3527)
1 parent ba51bd4 commit 5b3c462

File tree

5 files changed

+23
-15
lines changed

5 files changed

+23
-15
lines changed

.cirrus.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ task:
1414
- flutter channel master
1515
- flutter upgrade
1616
- git fetch origin master
17+
submodules_script:
18+
- git submodule init
19+
- git submodule update
1720
matrix:
1821
- name: publishable
1922
script:
@@ -121,6 +124,9 @@ task:
121124
- flutter channel master
122125
- flutter upgrade
123126
- git fetch origin master
127+
submodules_script:
128+
- git submodule init
129+
- git submodule update
124130
matrix:
125131
- name: build-linux+drive-examples
126132
install_script:
@@ -145,6 +151,9 @@ task:
145151
- flutter channel master
146152
- flutter upgrade
147153
- git fetch origin master
154+
submodules_script:
155+
- git submodule init
156+
- git submodule update
148157
create_simulator_script:
149158
- xcrun simctl list
150159
- xcrun simctl create Flutter-iPhone com.apple.CoreSimulator.SimDeviceType.iPhone-11 com.apple.CoreSimulator.SimRuntime.iOS-14-3 | xargs xcrun simctl boot
@@ -198,6 +207,9 @@ task:
198207
- flutter channel master
199208
- flutter upgrade
200209
- git fetch origin master
210+
submodules_script:
211+
- git submodule init
212+
- git submodule update
201213
create_simulator_script:
202214
- xcrun simctl list
203215
- xcrun simctl create Flutter-iPhone com.apple.CoreSimulator.SimDeviceType.iPhone-X com.apple.CoreSimulator.SimRuntime.iOS-13-3 | xargs xcrun simctl boot
@@ -227,6 +239,9 @@ task:
227239
- flutter channel master
228240
- flutter upgrade
229241
- git fetch origin master
242+
submodules_script:
243+
- git submodule init
244+
- git submodule update
230245
matrix:
231246
- name: build_all_plugins_app
232247
script:

.gitmodules

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
[submodule "script/plugin_tools"]
2+
path = script/plugin_tools
3+
url = https://github.com/flutter/plugin_tools.git

script/check_publish.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ source "$SCRIPT_DIR/common.sh"
1212

1313
function check_publish() {
1414
local failures=()
15-
pub_command global activate flutter_plugin_tools
16-
for dir in $(pub_command global run flutter_plugin_tools list --plugins="$1"); do
15+
for dir in $(plugin_tools list --plugins="$1"); do
1716
local package_name=$(basename "$dir")
1817

1918
echo "Checking that $package_name can be published."

script/common.sh

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,8 @@ function check_changed_packages() {
4646
return 0
4747
}
4848

49-
# Normalizes the call to the pub command.
50-
function pub_command() {
51-
if [ "$(expr substr $(uname -s) 1 5)" == "MINGW" ]; then
52-
pub.bat "$@"
53-
else
54-
pub "$@"
55-
fi
56-
}
57-
58-
# Activates the Flutter plugin tool to ensures that the plugin tools dependencies
59-
# are resolved using the current Dart SDK.
60-
# Finally, runs the tool with the parameters.
49+
# Runs the plugin tools from the plugin_tools git submodule.
6150
function plugin_tools() {
62-
pub_command global activate flutter_plugin_tools && pub_command global run flutter_plugin_tools "$@"
51+
(pushd "$REPO_DIR/script/plugin_tools" && dart pub get && popd) >/dev/null
52+
dart run "$REPO_DIR/script/plugin_tools/lib/src/main.dart" "$@"
6353
}

script/plugin_tools

Submodule plugin_tools added at 432c56d

0 commit comments

Comments
 (0)