Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Unified Diff: build/config/ios/BUILD.gn

Issue 1250913002: patch from chinmaygarde@ to make progress on mac, ios. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tweaks needed to get base_unittests to compile Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/config/ios/BUILD.gn
diff --git a/build/config/ios/BUILD.gn b/build/config/ios/BUILD.gn
index 0539810f818dd46564a57cff05e4ca1a5723bd48..43bcdb3a781cfa15dc8d75badb2cfb780a4e96a9 100644
--- a/build/config/ios/BUILD.gn
+++ b/build/config/ios/BUILD.gn
@@ -9,13 +9,10 @@ config("sdk") {
common_flags = [
"-isysroot",
sysroot,
- ]
- if (use_ios_simulator) {
- common_flags += [ "-mios-simulator-version-min=$ios_deployment_target" ]
- } else {
- common_flags += [ "-miphoneos-version-min=$ios_deployment_target" ]
- }
+ # TODO(dpranke): Figure out why chinmay had this set. Needed on ios?
sdefresne 2015/07/24 08:47:31 Yes, iOS builds requires libc++ and c++11 and uses
Dirk Pranke 2015/07/31 21:27:40 Hm. Seems to work now. Will fix.
+ #"-stdlib=libc++",
+ ]
cflags = common_flags
ldflags = common_flags

Powered by Google App Engine
This is Rietveld 408576698