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
dart: 2.7.2 -> 2.10.4 (stable) + 2.9.0-4.0.dev -> 2.12.0-223.0.dev (dev) #96174
Conversation
@Mic92 could you help us merging this? thanks! |
Dart got a 2.9.2 update, should I do that in this PR or make a new one once this is merged? |
I'd rather bump in this PR :) |
Sorry, it looks like this conflicts with #105109, though I think that won't be too hard to fix. However, it also conflicts with #105311, which might be a neater solution here. I like what #105311 is going for, but it's going to require some adjustment for derivations that call dart/default.nix. What do you think? |
@Sorixelle please fix the merge conflict. |
cef7e5f
to
0864e8e
Compare
Merge conflict fixed, rebased, and versions updated to most recent versions. I agree with @jtacoma here - #105311 seems like a much better solution going forward. It seems a bit stagnant at the moment, so for now, I think we can merge this so we have up-to-date versions now, and look at getting the aforementioned PR going. |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 3 packages built:
|
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.
diff LGTM
I marked this as stale due to inactivity. → More info |
This PR is outdated as dart is already updated |
Motivation for this change
dartaotruntime
binary not having it's interpreter and rpath patchedThings 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)