Repository navigation
Support Flutter SDK paths with spaces #6577
Description
Steps to Reproduce
- Install the SDK in a path containing spaces. e.g.,
/Users/someone/Google Drive/Development/dart/flutter-sdk. - Run flutter doctor
$ flutter doctor
...
/Users/someone/Google Drive/Development/dart/flutter-sdk/bin/flutter: line 50: $PROG_NAME: ambiguous redirect
Activity
cbracken commentedon Feb 9, 2017
flutter doctoris now working. What's still left to do:xcode_backend.shpath in the two build phases we call it in the project templates + all examples.xcode_backend.shitself.flutter runon both Android and iOS[-]Flutter doctor fails when Flutter SDK install path contains spaces[/-][+]Support Flutter SDK paths with spaces[/+]cbracken commentedon Feb 9, 2017
Now working:
xcode_backend.shis cleaned upflutter build iosWhat's still not working:
flutter runwith and without hot reload.cbracken commentedon Feb 10, 2017
There's also a bug in the engine (specifically in the Tonic loader) wherein
file://URIs are not URI-decoded during conversion to filesystem paths. I'll put together a patch and send upstream.Emit doctor error if Flutter SDK path contains spaces (#10477)
cbracken commentedon Jun 30, 2017
Note: dart-lang/sdk#30059 is a blocker for this.
Hixie commentedon Dec 13, 2017
@bkonyi You did this right?
cbracken commentedon Dec 13, 2017
This is done in the engine. We still need to verify nothing has regressed in the tooling, then the last remaining task here is to configure the build bot recipes with a space in the SDK path to ensure we don't regress this.
15 remaining items
cbracken commentedon Feb 10, 2018
This is now resolved. Travis/Appveyor are updated to run all tests in a flutter SDK directory named "flutter sdk".
cbracken commentedon Feb 10, 2018
As an added bonus, we now also support any path that's a valid UTF-8 string. This assumes UTF-8 encoding of paths in your filesystem, which isn't strictly-speaking a safe assumption everywhere.
Hixie commentedon Feb 12, 2018
This patch had a minor regressive impact on the aot_snapshot size metrics.Hixie commentedon Feb 12, 2018
Please disregard my most recent comment.
Add a space to the Flutter SDK path on Travis (flutter#14613)
Add a space to the Flutter SDK path on Appveyor (flutter#14615)
github-actions commentedon Sep 3, 2021
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of
flutter doctor -vand a minimal reproduction of the issue.