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

Unified Diff: testing/iossim/BUILD.gn

Issue 2351613006: Remove old iossim code (pre-Xcode 8) and class-dump. (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « DEPS ('k') | testing/iossim/iossim.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/iossim/BUILD.gn
diff --git a/testing/iossim/BUILD.gn b/testing/iossim/BUILD.gn
index 97fa2c8797b0a0d30af55ca1f114ae2de54bf45e..4d2882695e931dead8b0b9cecb525f5cc109f654 100644
--- a/testing/iossim/BUILD.gn
+++ b/testing/iossim/BUILD.gn
@@ -3,7 +3,6 @@
# found in the LICENSE file.
import("//build/config/mac/mac_sdk.gni")
-import("//third_party/class-dump/class-dump.gni")
if (current_toolchain == host_toolchain) {
config("config") {
@@ -14,7 +13,7 @@ if (current_toolchain == host_toolchain) {
# TODO(crbug.com/595295): Building class-dump tools requires OS X SDK
# version 10.9 or higher. Remove this override once the global version
# of the SDK is raised.
- common_flags = [ "-mmacosx-version-min=10.9" ]
+ common_flags = [ "-mmacosx-version-min=10.8" ]
sdefresne 2016/09/20 08:09:53 I guess that you tried to compile with 10.7 and th
justincohen 2016/09/20 20:55:21 Done.
ldflags = common_flags
cflags_objcc = common_flags
}
@@ -24,36 +23,7 @@ if (current_toolchain == host_toolchain) {
"iossim.mm",
]
libs = [ "Foundation.framework" ]
- deps = [
- ":generate_dvt_core_simulator",
- ":generate_dvt_foundation_header",
- ":generate_dvt_iphone_sim_header",
- ]
configs += [ ":config" ]
-
- # TODO(crbug.com/595295): Building class-dump tools requires OS X SDK
- # version 10.9 or higher. To prevent mixing code using different value
- # of minimum SDK supported, forbids dependencies on any Chromium target.
- assert_no_deps = [ "//base/*" ]
- }
-
- class_dump("generate_dvt_core_simulator") {
- framework_name = "CoreSimulator"
- framework_path = "$mac_sdk_path/../../../../../Library/PrivateFrameworks"
- class_dump_filter = "Sim"
- }
-
- class_dump("generate_dvt_foundation_header") {
- framework_name = "DVTFoundation"
- framework_path = "$mac_sdk_path/../../../../../../SharedFrameworks"
- class_dump_filter = "DVTStackBacktrace|DVTInvalidation|DVTMixIn"
- }
-
- class_dump("generate_dvt_iphone_sim_header") {
- topological_sort = true
- framework_name = "DVTiPhoneSimulatorRemoteClient"
- framework_path = "$mac_sdk_path/../../../../../../SharedFrameworks"
- class_dump_filter = "iPhoneSimulator"
}
} else if (current_toolchain == default_toolchain) {
copy("iossim") {
« no previous file with comments | « DEPS ('k') | testing/iossim/iossim.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698