Repository navigation
Add check for existence of system requirement command line tools #52262
Description
Check for the presence of all required executables as soon as possible (before artifacts are downloaded) in the tool and give a good error message. flutter doctor makes sense but the validators may be run too late to catch git or zip exceptions.
https://flutter.dev/docs/get-started/install/windows#system-requirements
- Windows PowerShell 5.0 or newer (this is pre-installed with Windows 10)
- Git for Windows 2.x, with the Use Git from the Windows Command Prompt option.
https://flutter.dev/docs/get-started/install/macos#system-requirements
- bash
- curl
- git 2.x
- mkdir
- rm
- unzip
- which
- zip
https://flutter.dev/docs/get-started/install/linux#system-requirements
(above list + xz-utils)
Activity
jmagman commentedon Mar 9, 2020
See #52241
[-]Add system requirement command line tool check[/-][+]Add check for existence of system requirement command line tools[/+]christopherfujino commentedon Mar 9, 2020
It might make sense to create an abstraction layer class for each system dependency we assume is installed, to uniformly handle them not existing.
jmagman commentedon Mar 9, 2020
May need to check some of them all the way up in https://github.com/flutter/flutter/blob/master/bin/flutter.
Also that list seems kind of random, why
mkdirandrmbut notechoortouchor the dozens of other system commands we use.jonahwilliams commentedon Mar 9, 2020
Can we go another direction entirely? How do we remove most/all of these dependencies?
christopherfujino commentedon Mar 9, 2020
I would love that.
christopherfujino commentedon Mar 9, 2020
Related #45278