-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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: roll forward missed patches to v3.23 and v3.24 #341133
Conversation
Sweet, this looks good. CC @NixOS/flutter, we should probably work on improving the patch system so we don't have to copy a bunch of patches. Maybe something similar to |
Will it be possible to merge this PR first to be able quickly to build apk using nixpkgs flutter and then look at how to improve the build system? |
Does this PR include flutter 3.24.3? |
Yes |
quickgui log: https://paste.aleksana.moe/--+Cid |
It's also failing on master: https://hydra.nixos.org/job/nixos/trunk-combined/nixpkgs.quickgui.x86_64-linux |
Is flutter 3.24.3 released? No flutter 3.24.3 was found in the search results. |
Yes, https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/compilers/flutter/versions
Likely hasn't rolled through the pipeline fully yet. |
Thanks for your reply, how long does it take through the pipeline? |
|
No flutter 3.24.3 was found in the search results. |
How long is it expected to take for 3.24.3 to come out? |
It should already be out, even if it isn't in the search. |
Description of changes
This
gradle-flutter-tools-wrapper.patch
is necessary for android builds using nixpkgs flutter. The patch file is present for prior package versions (3.16, 3.19, 3.22), but it looks like the previous update PR missed it (#336650).Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.