-
-
Notifications
You must be signed in to change notification settings - Fork 11.2k
flutter: add Linux support #236074
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
base: main
Are you sure you want to change the base?
flutter: add Linux support #236074
Conversation
28c32d0 to
af0ee5d
Compare
|
Adding an x86_64 dependency on Linux doesn't seem to stop CI from trying other test on arm64 Linux. We might need changes in Homebrew/brew to handle this. Thoughts @Homebrew/cask? |
Casks/f/flutter.rb
Outdated
|
|
||
| auto_updates true | ||
|
|
||
| suite "flutter", target: "#{HOMEBREW_PREFIX}/share/flutter" |
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.
The suite stanza does not work on Linux, so I replaced it with artifact.
This change provides similar behavior but works cross-platform, including Linux.
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.
Can you elaborate on it 'does not work'?
af0ee5d to
b16ba01
Compare
|
Just trying something here that may work, ultimately we do need some changes in Homebrew/Brew to handle a couple of cases;
Note: I have temporarily changed the |
b16ba01 to
73519c0
Compare
|
Yeah, part of 2 is already part of Homebrew/brew#20334. So you can copy from there. |
89edc10 to
dba32f4
Compare
|
I’m not sure why Flutter doesn’t provide a precache bundle for the Linux ARM64 platform on its official site. However, Flutter does support Linux ARM64 on the stable channel through the stable branch by cloning the official Flutter repository and running This approach works for all supported platforms and architectures, so I modified this Homebrew cask to automate that process using the source from the stable branch. I’ve tested this cask on Linux x64, Linux ARM64, and macOS ARM64 — all work correctly. Should this be implemented as a formula instead of a cask? Also, why doesn’t the CI workflow detect support for Linux x64 and Linux ARM64? |
Signed-off-by: Patrick Linnane <[email protected]>
b9b3ec6 to
4672400
Compare
Linux casks are a relatively new concept, so things like this are still identified as we add them. |
Add Linux support to Flutter cask
Summary:
This pull request extends the existing
fluttercask to support Linux platforms.It uses the official Flutter SDK binaries distributed by Google, ensuring parity with the macOS version.
Changes:
sha256andurlfor the official Linux archive.flutter,dart) work consistently on both macOS and Linux.Checklist:
brew audit --cask --online flutterpasses (expectedlipowarning due to wrapper scripts).brew style --fix flutterreports no offenses.brew audit --cask --new flutterpasses successfully on Linux.HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask flutterworks correctly on both macOS and Linux.brew uninstall --cask flutterworks correctly on both macOS and Linux.Notes:
flutteranddartexecutables are shell wrappers that invoke platform binaries;lipowarnings during audit are expected and do not indicate an issue.$(brew --prefix)/share/flutterfor consistent path usage across systems.