Skip to content
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

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

phlip9
Copy link
Contributor

@phlip9 phlip9 commented Sep 11, 2024

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

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@phlip9
Copy link
Contributor Author

phlip9 commented Sep 11, 2024

cc @RossComputerGuy

@RossComputerGuy
Copy link
Member

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 getVersionFile which I implemented in LLVM?

@hatch01
Copy link
Contributor

hatch01 commented Sep 13, 2024

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?

@kjxbyz
Copy link

kjxbyz commented Sep 13, 2024

Does this PR include flutter 3.24.3?

@RossComputerGuy
Copy link
Member

Does this PR include flutter 3.24.3?

Yes

@Aleksanaa
Copy link
Member

Aleksanaa commented Sep 14, 2024

x86_64-linux

3 packages failed to build:
quickgui quickgui.debug quickgui.pubcache

16 packages built:
flet-client-flutter flet-client-flutter.debug flet-client-flutter.pubcache flutter flutter323 flutterPackages-source.beta flutterPackages-source.v3_24 hover python311Packages.flet python311Packages.flet-runtime python311Packages.flet-runtime.dist python311Packages.flet.dist python312Packages.flet python312Packages.flet-runtime python312Packages.flet-runtime.dist python312Packages.flet.dist

quickgui log: https://paste.aleksana.moe/--+Cid

@Aleksanaa
Copy link
Member

@mkg20001 mkg20001 merged commit 962f168 into NixOS:master Sep 14, 2024
26 of 28 checks passed
@kjxbyz
Copy link

kjxbyz commented Sep 16, 2024

Is flutter 3.24.3 released? No flutter 3.24.3 was found in the search results.

https://search.nixos.org/packages?channel=unstable&from=0&size=50&sort=relevance&type=packages&query=flutter

@RossComputerGuy
Copy link
Member

Is flutter 3.24.3 released?

Yes, https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/compilers/flutter/versions

No flutter 3.24.3 was found in the search results.

Likely hasn't rolled through the pipeline fully yet.

@phlip9 phlip9 deleted the phlip9/flutter324-patch-fix branch September 16, 2024 19:25
byronogis added a commit to byronogis/nix-config that referenced this pull request Sep 18, 2024
@kjxbyz
Copy link

kjxbyz commented Sep 21, 2024

Thanks for your reply, how long does it take through the pipeline?

@byronogis
Copy link

Thanks for your reply, how long does it take through the pipeline?

unstable already

@kjxbyz
Copy link

kjxbyz commented Sep 21, 2024

@Lucas-Haux
Copy link

How long is it expected to take for 3.24.3 to come out?

@RossComputerGuy
Copy link
Member

It should already be out, even if it isn't in the search.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants