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

Unified Diff: build/config/BUILD.gn

Issue 1902593003: [Mac/GN] Do not specify any default_libs to link. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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 | « base/BUILD.gn ('k') | ui/base/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/BUILD.gn
diff --git a/build/config/BUILD.gn b/build/config/BUILD.gn
index b6d0b71fa0edeaa9b20b0742821c8be9732e6ff8..73c7d831d4b30167708e6785414fdb68e09b2e0a 100644
--- a/build/config/BUILD.gn
+++ b/build/config/BUILD.gn
@@ -361,16 +361,9 @@ config("default_libs") {
"m",
]
} else if (is_mac) {
- libs = [
- "AppKit.framework",
- "ApplicationServices.framework",
- "Carbon.framework",
- "CoreFoundation.framework",
- "Foundation.framework",
- "IOKit.framework",
- "OpenGL.framework",
- "Security.framework",
- ]
+ # Targets should choose to explicitly link frameworks they require. Since
+ # linking can have run-time side effects, nothing should be listed here.
+ libs = []
} else if (is_ios) {
# The libraries listed here will be specified for both the target and the
# host. Only the common ones should be listed here.
« no previous file with comments | « base/BUILD.gn ('k') | ui/base/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698