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

Unified Diff: third_party/libevent/libevent.gyp

Issue 14203006: Extract use_system_foo logic out of gyp files, patch 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: third_party/libevent/libevent.gyp
diff --git a/third_party/libevent/libevent.gyp b/third_party/libevent/libevent.gyp
index e0d8304517274a855596331a5470894d7c6db4dd..27ac1f9c583bbbf33c071549e17b8996be124497 100644
--- a/third_party/libevent/libevent.gyp
+++ b/third_party/libevent/libevent.gyp
@@ -3,90 +3,61 @@
# found in the LICENSE file.
{
- 'variables': {
- 'use_system_libevent%': 0,
- },
- 'conditions': [
- ['use_system_libevent==0', {
- 'targets': [
- {
- 'target_name': 'libevent',
- 'product_name': 'event',
- 'type': 'static_library',
- 'toolsets': ['host', 'target'],
- 'sources': [
- 'buffer.c',
- 'evbuffer.c',
- 'evdns.c',
- 'event.c',
- 'event_tagging.c',
- 'evrpc.c',
- 'evutil.c',
- 'http.c',
- 'log.c',
- 'poll.c',
- 'select.c',
- 'signal.c',
- 'strlcpy.c',
- ],
- 'defines': [
- 'HAVE_CONFIG_H',
- ],
- 'conditions': [
- # libevent has platform-specific implementation files. Since its
- # native build uses autoconf, platform-specific config.h files are
- # provided and live in platform-specific directories.
- [ 'OS == "linux" or (OS == "android" and _toolset == "host")', {
- 'sources': [ 'epoll.c', 'epoll_sub.c' ],
- 'include_dirs': [ 'linux' ],
- 'link_settings': {
- 'libraries': [
- # We need rt for clock_gettime().
- # TODO(port) Maybe on FreeBSD as well?
- '-lrt',
- ],
- },
- }],
- [ 'OS == "android" and _toolset == "target"', {
- # On android, epoll_create(), epoll_ctl(), epoll_wait() and
- # clock_gettime() are all in libc.so, so no need to add
- # epoll_sub.c and link librt.
- 'sources': [ 'epoll.c' ],
- 'include_dirs': [ 'android' ],
- }],
- [ 'OS == "mac" or OS == "ios" or os_bsd==1', {
- 'sources': [ 'kqueue.c' ],
- 'include_dirs': [ 'mac' ]
- }],
- [ 'OS == "solaris"', {
- 'sources': [ 'devpoll.c', 'evport.c' ],
- 'include_dirs': [ 'solaris' ]
- }],
- ],
- },
+ 'targets': [
+ {
+ 'target_name': 'libevent',
+ 'product_name': 'event',
+ 'type': 'static_library',
+ 'toolsets': ['host', 'target'],
+ 'sources': [
+ 'buffer.c',
+ 'evbuffer.c',
+ 'evdns.c',
+ 'event.c',
+ 'event_tagging.c',
+ 'evrpc.c',
+ 'evutil.c',
+ 'http.c',
+ 'log.c',
+ 'poll.c',
+ 'select.c',
+ 'signal.c',
+ 'strlcpy.c',
],
- }, { # use_system_libevent != 0
- 'targets': [
- {
- 'target_name': 'libevent',
- 'type': 'none',
- 'toolsets': ['host', 'target'],
- 'variables': {
- 'headers_root_path': '.',
- 'header_filenames': [
- 'event.h',
- ],
- },
- 'includes': [
- '../../build/shim_headers.gypi',
- ],
+ 'defines': [
+ 'HAVE_CONFIG_H',
+ ],
+ 'conditions': [
+ # libevent has platform-specific implementation files. Since its
+ # native build uses autoconf, platform-specific config.h files are
+ # provided and live in platform-specific directories.
+ [ 'OS == "linux" or (OS == "android" and _toolset == "host")', {
+ 'sources': [ 'epoll.c', 'epoll_sub.c' ],
+ 'include_dirs': [ 'linux' ],
'link_settings': {
'libraries': [
- '-levent',
+ # We need rt for clock_gettime().
+ # TODO(port) Maybe on FreeBSD as well?
+ '-lrt',
],
},
- }
+ }],
+ [ 'OS == "android" and _toolset == "target"', {
+ # On android, epoll_create(), epoll_ctl(), epoll_wait() and
+ # clock_gettime() are all in libc.so, so no need to add
+ # epoll_sub.c and link librt.
+ 'sources': [ 'epoll.c' ],
+ 'include_dirs': [ 'android' ],
+ }],
+ [ 'OS == "mac" or OS == "ios" or os_bsd==1', {
+ 'sources': [ 'kqueue.c' ],
+ 'include_dirs': [ 'mac' ]
+ }],
+ [ 'OS == "solaris"', {
+ 'sources': [ 'devpoll.c', 'evport.c' ],
+ 'include_dirs': [ 'solaris' ]
+ }],
],
- }],
+ },
],
}

Powered by Google App Engine
This is Rietveld 408576698