New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[flutter_tools] require cmdline-tools for android licenses #82560
[flutter_tools] require cmdline-tools for android licenses #82560
Conversation
In order for this change to land, all of our infra recipes would need to be updated to download the cmdline-tools |
…ine-tools-sdkmanager
…lliams/flutter into require-cmdline-tools-sdkmanager
@@ -262,7 +268,7 @@ class AndroidSdk { | |||
return null; | |||
} | |||
|
|||
String? getCmdlineToolsPath(String binaryName) { | |||
String? getCmdlineToolsPath(String binaryName, {bool skipTools = false}) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a more specific name than skipTools
for this flag? Maybe skipOldTools
(supposing that I'm reading the comments in the function correctly)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also does the skipTools = true
case have test coverage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done!
Made the doctor check fatal, now waiting on an update to LUCI |
…ine-tools-sdkmanager
…lliams/flutter into require-cmdline-tools-sdkmanager
Fixes #80273
The latest android studio can only run the cmdline tools version of the sdk manager. Running the platform tools version will lead to an unresolvable
java.lang.NoClassDefFoundError