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 stable (1.12.13+hotfix.9 -> 1.17.3), beta (1.15.17 -> 1.19.0-4.1.pre) and dev (1.17.0-dev.5.0 -> 1.20.0-0.0.pre) update #90681
Conversation
version = "1.12.13+hotfix.9"; | ||
sha256Hash = "1ql3zvmmk5zk47y30lajxaam04q6vr373dayq15jv4vpc0fzif1y"; | ||
version = "1.17.3"; | ||
filename = "flutter_linux_${version}-${channel}.tar.xz"; |
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.
It seems like flutter changed the naming scheme everywhere, this should probably be changed in flutter.nix
to prevent boilerplate code.
@babariviere Could you review? |
I will review it ! Thank you for your contribution ! |
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.
Works great ! I've tested the 3 versions by running this script:
nix-review pr 90681
# Stable
flutter new stable
cd stable
flutter run
# check if it runs
cd ..
# Beta
flutter-beta new beta
cd beta
flutter-beta run
# check if it works
cd ..
# Dev
flutter-dev create dev
cd dev
flutter-dev run
# check if it works
I've got no issue with it and flutter web works.
It fixes #85574.
That sounds great! |
Seems like version 1.17.5 etc has been release, I guess I'll update |
I've updated stable, beta and dev versions to their newest versions respectively, I think this is ready to review again. It would be great if this could get merged before flutter releases new major versions because these might require updating the patches again... |
Result of 3 packages built:- flutter (flutterPackages.stable) - flutterPackages.beta - flutterPackages.dev I also created a new app and ran 👍 |
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.
Works great !
@babariviere @worldofpeace @kolaente Can we merge this PR? |
Yes, it's good for me but I don't have the permissions to do so. |
Thank you, I merged ✨ |
Meanwhile, Flutter updated their versions again 🙂 I guess I'll send a follow-up PR... |
PR is up: #94092 |
Already ? Well done ! |
@babariviere Was an easy one since I did not need to modify the patches again 🙃 |
Motivation for this change
Flutter packages update for stable (1.12.13+hotfix.9 -> 1.17.3), beta (1.15.17 -> 1.19.0-4.1.pre) and dev (1.17.0-dev.5.0 -> 1.20.0-0.0.pre) packages.
I fixed the patches since they stopped working. For testing, I ran
flutter doctor
for each version.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)