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: 1.24.3 -> 2.0.0 #46526

Closed
wants to merge 3 commits into from
Closed

dart: 1.24.3 -> 2.0.0 #46526

wants to merge 3 commits into from

Conversation

bitmappergit
Copy link
Contributor

@bitmappergit bitmappergit commented Sep 11, 2018

Motivation for this change

The Dart VM supplied by nixpkgs has been outdated for quite a while, and needed to be updated.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

Added the link to the new dart sdk stable. Forgot semicolons.
@andir
Copy link
Member

andir commented Sep 11, 2018

Thank you for the contribution!

Could you please adjust the commit message to match our usual format?

The CONTRIBUTING.md describes that in detail: https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes

@bitmappergit bitmappergit changed the title Update default.nix dart: 1.24.3 -> 2.0.0 Sep 11, 2018
@andir
Copy link
Member

andir commented Sep 11, 2018

all-packages.nix has a few references to dart version:

  dart = callPackage ../development/interpreters/dart { };
  dart_stable = dart.override { version = "1.24.3"; };
  dart_old = dart.override { version = "1.16.1"; };
  dart_dev = dart.override { version = "2.0.0-dev.26.0"; };

Could you please update them according the version change?

@@ -25,6 +25,14 @@ let
url = "${stable}/${version}/sdk/dartsdk-linux-ia32-release.zip";
sha256 = "d67b8f8f9186e7d460320e6bce25ab343c014b6af4b2f61369ee83755d4da528";
};
"2.0.0-x86_64-linux" = fetchurl {
url = "{$stable}/${version}/sdk/dartsdk-linux-x64-release.zip";
Copy link
Member

Choose a reason for hiding this comment

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

You have a small syntax error there ;)

The $ of the stable part isn't where it should be.

sha256 = "4014a1e8755d2d32cc1573b731a4a53acdf6dfca3e26ee437f63fe768501d336";
};
"2.0.0-i686-linux" = fetchurl {
url = "{$stable}/${version}/sdk/dartsdk-linux-ia32-release.zip";
Copy link
Member

Choose a reason for hiding this comment

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

same issue as before.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sorry for causing so many problems with this, it's been a little hectic not being able to fix things because my internet is spotty right now

Copy link
Member

Choose a reason for hiding this comment

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

Thais all right. just take your time.

moved dollar sign
changed dart version numbers
@bitmappergit
Copy link
Contributor Author

Done.

@bitmappergit
Copy link
Contributor Author

Is there anything else that needs to be done?

@andir
Copy link
Member

andir commented Sep 11, 2018

The commits still have the wrong messages.

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.

None yet

4 participants