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

Side by Side Diff: ui/ui.gyp

Issue 153223002: Use gyp targets to link in X11 libraries instead of setting -lXfoo (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address feedback, try to fix cros_x86 gyp error Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « ui/gl/gl.gyp ('k') | ui/views/views.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 }, 8 },
9 'targets': [ 9 'targets': [
10 { 10 {
(...skipping 509 matching lines...) Expand 10 before | Expand all | Expand 10 after
520 '$(SDKROOT)/System/Library/Frameworks/AudioUnit.framework', 520 '$(SDKROOT)/System/Library/Frameworks/AudioUnit.framework',
521 '$(SDKROOT)/System/Library/Frameworks/CoreVideo.framework', 521 '$(SDKROOT)/System/Library/Frameworks/CoreVideo.framework',
522 ], 522 ],
523 }, 523 },
524 }], 524 }],
525 ['use_x11==1', { 525 ['use_x11==1', {
526 'all_dependent_settings': { 526 'all_dependent_settings': {
527 'ldflags': [ 527 'ldflags': [
528 '-L<(PRODUCT_DIR)', 528 '-L<(PRODUCT_DIR)',
529 ], 529 ],
530 # TODO(jamesr): Targets using X11 should declare dependencies on the
531 # appropriate build/linux/system.gyp targets themselves instead of
532 # independently picking it up from ui.
530 'link_settings': { 533 'link_settings': {
531 'libraries': [ 534 'libraries': [
532 '-lX11', 535 '-lX11',
533 '-lXcursor', 536 '-lXcursor',
534 '-lXrender', # For XRender* function calls in x11_util.cc. 537 '-lXrender',
535 ], 538 ],
536 }, 539 },
537 }, 540 },
538 'link_settings': {
539 'libraries': [
540 '-lX11',
541 '-lXcursor',
542 '-lXrender', # For XRender* function calls in x11_util.cc.
543 ],
544 },
545 'dependencies': [ 541 'dependencies': [
546 '../build/linux/system.gyp:x11', 542 '../build/linux/system.gyp:x11',
543 '../build/linux/system.gyp:xcursor',
547 '../build/linux/system.gyp:xext', 544 '../build/linux/system.gyp:xext',
548 '../build/linux/system.gyp:xfixes', 545 '../build/linux/system.gyp:xfixes',
546 '../build/linux/system.gyp:xrender', # For XRender* function calls in x11_util.cc.
549 ], 547 ],
550 }], 548 }],
551 ['use_ozone==0', { 549 ['use_ozone==0', {
552 'sources!': [ 550 'sources!': [
553 'base/cursor/cursor_null.cc', 551 'base/cursor/cursor_null.cc',
554 'base/cursor/cursor_loader_null.cc', 552 'base/cursor/cursor_loader_null.cc',
555 'base/cursor/cursor_loader_null.h', 553 'base/cursor/cursor_loader_null.h',
556 ], 554 ],
557 }], 555 }],
558 ['toolkit_views==0', { 556 ['toolkit_views==0', {
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 'variables': { 642 'variables': {
645 'jni_gen_package': 'ui', 643 'jni_gen_package': 'ui',
646 'jni_generator_ptr_type': 'long', 644 'jni_generator_ptr_type': 'long',
647 }, 645 },
648 'includes': [ '../build/jni_generator.gypi' ], 646 'includes': [ '../build/jni_generator.gypi' ],
649 }, 647 },
650 ], 648 ],
651 }], 649 }],
652 ], 650 ],
653 } 651 }
OLDNEW
« no previous file with comments | « ui/gl/gl.gyp ('k') | ui/views/views.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698