Linux: create a library loader code generator for dlopen and use it for libpci.

content/content_gpu.gypi

 # 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': {
     'linux_link_libpci%': 0,
   },
   'dependencies': [
     '../base/base.gyp:base',
@@ skipping 30 matching lines @@
         '../third_party/angle/src/build_angle.gyp:libEGL',
         '../third_party/angle/src/build_angle.gyp:libGLESv2',
         '../third_party/libxml/libxml.gyp:libxml',
       ],
       'link_settings': {
         'libraries': [
           '-lsetupapi.lib',
         ],
       },
     }],
+    ['OS=="linux"', {
+      'cflags': [
+        '<!@(pkg-config --cflags libpci)',

[piman, 2012/11/27 00:09:18]

You need to go through the pkg-config indirection for sysroots (see
build/linux/system.gyp)

[Paweł Hajdan Jr., 2012/11/27 19:52:18]

Done.

+      ],
+      'sources': [
+        '<(INTERMEDIATE_DIR)/libpci_loader.cc',
+      ],
+      'include_dirs': [
+        '<(INTERMEDIATE_DIR)',
+      ],
+      'hard_dependency': 1,

[piman, 2012/11/27 00:09:18]

Is that really needed? That prevents a ton of build parallelization (essentially
forces a build barrier between content's dependencies and content, as well as
between content and content's dependent).
Could we separate this into its own target? This would maximize parallelization
(this can happen in parallel with content's dependencies building)

[Paweł Hajdan Jr., 2012/11/27 19:52:18]

Done.

+      'actions': [
+        {
+          'action_name': 'generate_libpci_loader',
+          'inputs': [
+            '../tools/generate_library_loader/generate_library_loader.py',
+          ],
+          'outputs': [
+            '<(INTERMEDIATE_DIR)/libpci_loader.cc',
+            '<(INTERMEDIATE_DIR)/libpci_loader.h',
+          ],
+          'action': ['python',
+                     '../tools/generate_library_loader/generate_library_loader.py',
+                     '--name', 'LibPciLoader',
+                     '--output', '<(INTERMEDIATE_DIR)/libpci_loader',
+                     '--header', '<pci/pci.h>',
+                     '--use-extern-c',
+                     '--link-directly=<(linux_link_libpci)',
+                     'pci_alloc',
+                     'pci_init',
+                     'pci_cleanup',
+                     'pci_scan_bus',
+                     'pci_fill_info',
+                     'pci_lookup_name',
+          ],
+          'message': 'Generating libpci library loader.',
+        },
+      ],
+    }],
     ['OS=="win" and directxsdk_exists=="True"', {
       'actions': [
         {
           'action_name': 'extract_d3dx9',
           'variables': {
             'input': 'Jun2010_d3dx9_43_x86.cab',
             'output': 'd3dx9_43.dll',
           },
           'inputs': [
             '../third_party/directxsdk/files/Redist/<(input)',
@@ skipping 45 matching lines @@
     ['target_arch=="arm" and chromeos == 1', {
       'include_dirs': [
         '<(DEPTH)/third_party/openmax/il',
       ],
     }],
     ['target_arch!="arm" and chromeos == 1', {
       'include_dirs': [
         '<(DEPTH)/third_party/libva',
       ],
     }],
-    ['linux_link_libpci==0', {
-      'defines': [
-        'DLOPEN_LIBPCI',
-      ],
-    }, { # linux_link_libpci==1
-      'cflags': [
-        '<!@(pkg-config --cflags libpci)',
-      ],
+    ['linux_link_libpci==1', {
       'link_settings': {
         'ldflags': [
           '<!@(pkg-config --libs-only-L --libs-only-other libpci)',
         ],
         'libraries': [
           '<!@(pkg-config --libs-only-l libpci)',
         ],
       }
     }],
   ],
 }