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: reduce derivation size and refactor #108611
Conversation
Instead of copying the whole dart derivation to bin/cache/dart-sdk directory, just symlink it. Also, make sure that the flutter is build from dart passed as a parameter. Before: /nix/store/p3avfmvd0yrjpwwml7vvqbjin5vacx2k-flutter-stable-1.22.0 2084894968 After: /nix/store/750k4z1yj5xsw7ymmwvn7cfjjzkaygzg-flutter-stable-1.22.0 1647047080
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.
LGTM
Tested in a local repo:
|
Opening this PR for review. I will try to bump flutter again after this PR is merged on master: #105311 |
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
/status needs_merger |
@SuperSandro2000 Why the tag of blocked by PR? I will leave the bump in a separate PR. |
Result of 4 packages built:
|
Motivation for this change
This PR refactors the flutter derivation, with the main objective to reduce derivation size:
Also, bump the packages to the latest versions.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)