Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

chinmaygarde
Copy link
Member

No description provided.

@chinmaygarde
Copy link
Member Author

cc @abarth @tvolkert

fi
else
RunCommand touch ${derived_dir}/app.so
RunCommand eval "$(echo \"static const int Moo = 88;\" | xcrun clang -x c --shared -o ${derived_dir}/app.so -)"
Copy link
Contributor

Choose a reason for hiding this comment

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

42

@abarth
Copy link
Contributor

abarth commented May 27, 2016

LGTM

@tvolkert
Copy link
Contributor

lgtm

@chinmaygarde chinmaygarde merged commit 63310aa into flutter:master May 27, 2016
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
New commits:

git log --format="%C(auto) %h %s" 900e796..4ca4767
 4ca4767 Added a dart pub outdated --transitive option (flutter#2731)
 6b145bd Deprecate --server argument to `pub publish` and `pub uploader`. (flutter#2697)
 7737023 don't warn if previous prerelease was null safe (flutter#2730)
 62f9283 Improve outdated --mode=null-safety (flutter#2724)
 cc589ec Change message for no Latest resolution (flutter#2729)
 656803e Require sdk constraint (flutter#2718)
 8309d87 Added test that dev_dependency does not trigger null-safety warnings when publishing (flutter#2727)
 332ea04 Remove warning about mixed mode. (flutter#2723)
 a98a1f2 Simplify null-safety analysis in `pub outdated --mode=null-safety` (flutter#2721)
 5fba201 Outdated null safety implies prereleases (flutter#2722)
 fb9ec4a Fixed bug in yaml_edit (flutter#2703)

Change-Id: I22a084aee06542e04a272269fb0134f0ac62f779
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/170690
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
Reviewed-by: Michael Thomsen <mit@google.com>
Reviewed-by: Jonas Jensen <jonasfj@google.com>
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
* nit: use d.file instead of YamlDescriptor

* make SDK constraint required

* migrated tests

* Update message

* Updated message and added test

* Fixed embedding tests

* Moved to separate method
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants