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

dart: 2.19.6 -> 3.0.0 #231083

Merged
merged 2 commits into from
May 13, 2023
Merged

dart: 2.19.6 -> 3.0.0 #231083

merged 2 commits into from
May 13, 2023

Conversation

hacker1024
Copy link
Member

@hacker1024 hacker1024 commented May 10, 2023

Description of changes

This upgrades Dart from 2.19.6 to 3.0.0.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 23.05 Release Notes (or backporting 22.11 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.

@hacker1024
Copy link
Member Author

For some reason, this seems to break dart-sass-embedded. @kira-bruneau?

error: builder for '/nix/store/naphmf1vki47v42cbivmmj7lk9rh4lj4-dart-sass-embedded-1.62.1.drv' failed with exit code 69;
       last 7 log lines:
       > unpacking sources
       > unpacking source archive /nix/store/1q84s8r7a16mw1l1gwimqb6vrlzicq8p-source
       > source root is source
       > patching sources
       > configuring
       > Resolving dependencies...
       > Package not available (Could not find git ref '8dddcb7b7db13984fea69fa85438acf30b56b4bb' (fatal: bad object 8dddcb7b7db13984fea69fa85438acf30b56b4bb)).
       For full logs, run 'nix log /nix/store/naphmf1vki47v42cbivmmj7lk9rh4lj4-dart-sass-embedded-1.62.1.drv'.
error: 1 dependencies of derivation '/nix/store/fakjjf4ld7mm3sk89i558w35x1j4fmpx-ruby3.2.2-sass-embedded-1.62.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/v9iiqmqyw6zqmwx43hicny7mn7n5ywpp-ruby3.2.2-dartsass-ruby-3.0.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/l63ik4agp3mvads40a3fm9x34h8nra9b-ruby3.2.2-dartsass-sprockets-3.0.0.drv' failed to build
error: 3 dependencies of derivation '/nix/store/n0lmg5hyy41ipbkp9rajrqd5i475x9xx-discourse-ruby-env-3.1.0.beta4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/jj30k47l41vw5lr8aiknvn1kyf7km35w-wrapped-ruby-discourse-ruby-env-3.1.0.beta4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/3ywkdq506lam0vcp9hrj3l46n7m8zkms-discourse-assets-3.1.0.beta4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/mgx6brp614kpk71ihjv07df8lgxzbsww-discourse-assets-3.1.0.beta4.drv' failed to build
error: 3 dependencies of derivation '/nix/store/hk6nfsnqdfazq36fp3nvxa30nv9w0qgd-discourse-3.1.0.beta4.drv' failed to build
error: 3 dependencies of derivation '/nix/store/lxdpb32wfn6r41r3ic2rary0m4qwm76s-discourse-3.1.0.beta4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/m8d3k2a32rwb91azlib1583y4856pl76-shopware-cli-0.1.62.drv' failed to build
error: 4 dependencies of derivation '/nix/store/abhqdp78dgspjafjvzs9zwiccbsaalhr-review-shell.drv' failed to build
4 packages failed to build:
dart-sass-embedded discourse discourseAllPlugins shopware-cli

@hacker1024
Copy link
Member Author

@hacker1024 hacker1024 changed the title dart: 2.19.6 -> 3.0.0 [WIP] dart: 2.19.6 -> 3.0.0 May 11, 2023
@hacker1024 hacker1024 marked this pull request as draft May 11, 2023 01:36
@hacker1024 hacker1024 marked this pull request as ready for review May 11, 2023 14:45
@hacker1024 hacker1024 changed the title [WIP] dart: 2.19.6 -> 3.0.0 dart: 2.19.6 -> 3.0.0 May 11, 2023
@hacker1024
Copy link
Member Author

hacker1024 commented May 11, 2023

@NixOS/flutter

I have added a wrapper script around git that is used when calling pub get to spoof revision results.

This solution, however, has the following drawbacks:

  • Users must remember to use the doPubGet function to make pub get calls (though this will be done for them when using provided builders like buildFlutterApplication)
  • The wrapper logic is coupled tightly with Pub's internal revision checking method. If the command Pub runs changes even slightly, this could break.

Does anyone have any ideas for other ways to fix this issue? If not, this may be the best workaround. I don't think it's possible to modify the files in .git to fake a commit hash.

@hacker1024
Copy link
Member Author

hacker1024 commented May 11, 2023

Note: All rebuilds now succeed other than yubioath-flutter, which has been broken since 293ca68. (#230674)

@ofborg ofborg bot requested review from lelgenio and shyim May 11, 2023 15:25
Copy link
Member

@shyim shyim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@lelgenio
Copy link
Contributor

How does this fit with #229770? If this was merged later, you could just update buildDartApplication.

@hacker1024
Copy link
Member Author

How does this fit with #229770? If this was merged later, you could just update buildDartApplication.

buildDartApplication will need to be updated to use the doPubGet function.

@hacker1024 hacker1024 mentioned this pull request May 11, 2023
12 tasks
@mkg20001 mkg20001 merged commit 49ce83d into NixOS:master May 13, 2023
This pull request was closed.
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.

6 participants