Use gyp targets to link in X11 libraries instead of setting -lXfoo

build/linux/system.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'conditions': [
       ['sysroot!=""', {
         'pkg-config': './pkg-config-wrapper "<(sysroot)" "<(target_arch)"',
       }, {
@@ skipping 312 matching lines @@
           'link_settings': {
             'ldflags': [
               '<!@(pkg-config --libs-only-L --libs-only-other <(glib_packages))',
             ],
             'libraries': [
               '<!@(pkg-config --libs-only-l <(glib_packages))',
             ],
           },
         }],
         ['use_x11==1', {
-          'link_settings': {
-            'libraries': [ '-lXtst' ]
-          }
+          'dependencies': [ 'xtst' ],
         }],
       ],
     },
     {
       'target_name': 'gnome_keyring',
       'type': 'none',
       'conditions': [
         ['use_gnome_keyring==1', {
           'direct_dependent_settings': {
             'cflags': [
@@ skipping 444 matching lines @@
               '<!@(pkg-config --libs-only-L --libs-only-other x11 xi)',
             ],
             'libraries': [
               '<!@(pkg-config --libs-only-l x11 xi)',
             ],
           },
         }],
       ],
     },
     {
+      'target_name': 'xcursor',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {

[piman, 2014/02/04 00:25:14]

nit: you don't need that conditional if you don't declare multiple toolsets.
(same with other ones).

+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xcursor)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xcursor)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xcursor)',
+            ],
+          },
+        }],
+      ],
+    },
+    {
+      'target_name': 'xcomposite',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {
+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xcomposite)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xcomposite)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xcomposite)',
+            ],
+          },
+        }],
+      ],
+    },
+    {
+      'target_name': 'xdamage',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {
+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xdamage)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xdamage)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xdamage)',
+            ],
+          },
+        }],
+      ],
+    },
+    {
       'target_name': 'xext',
       'type': 'none',
       'conditions': [
         ['_toolset=="target"', {
           'direct_dependent_settings': {
             'cflags': [
               '<!@(<(pkg-config) --cflags xext)',
             ],
           },
           'link_settings': {
@@ skipping 22 matching lines @@
               '<!@(<(pkg-config) --libs-only-L --libs-only-other xfixes)',
             ],
             'libraries': [
               '<!@(<(pkg-config) --libs-only-l xfixes)',
             ],
           },
         }],
       ],
     },
     {
+      'target_name': 'xi',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {
+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xi)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xi)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xi)',
+            ],
+          },
+        }],
+      ],
+    },
+    {
       'target_name': 'xrandr',
       'type': 'none',
       'toolsets': ['host', 'target'],
       'conditions': [
         ['_toolset=="target"', {
           'direct_dependent_settings': {
             'cflags': [
               '<!@(<(pkg-config) --cflags xrandr)',
             ],
           },
@@ skipping 15 matching lines @@
             'ldflags': [
               '<!@(pkg-config --libs-only-L --libs-only-other xrandr)',
             ],
             'libraries': [
               '<!@(pkg-config --libs-only-l xrandr)',
             ],
           },
         }],
       ],
     },
+    {
+      'target_name': 'xrender',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {
+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xrender)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xrender)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xrender)',
+            ],
+          },
+        }],
+      ],
+    },
+    {
+      'target_name': 'xtst',
+      'type': 'none',
+      'conditions': [
+        ['_toolset=="target"', {
+          'direct_dependent_settings': {
+            'cflags': [
+              '<!@(<(pkg-config) --cflags xtst)',
+            ],
+          },
+          'link_settings': {
+            'ldflags': [
+              '<!@(<(pkg-config) --libs-only-L --libs-only-other xtst)',
+            ],
+            'libraries': [
+              '<!@(<(pkg-config) --libs-only-l xtst)',
+            ],
+          },
+        }],
+      ],
+    }
   ],
 }