Adds support for the library_dirs key, which appears in the documentation but was never actually im…

pylib/gyp/generator/make.py

Index: pylib/gyp/generator/make.py
===================================================================
--- pylib/gyp/generator/make.py	(revision 1429)
+++ pylib/gyp/generator/make.py	(working copy)
@@ -1418,6 +1418,9 @@
             ldflags.append(r'-Wl,-rpath=\$$ORIGIN/lib.%s/' % self.toolset)
             ldflags.append(r'-Wl,-rpath-link=\$(builddir)/lib.%s/' %
                            self.toolset)
+        library_dirs = spec.get('library_dirs', [])

[Nico, 2012/07/21 19:14:48]

Do you need to grab this from |spec|? Isn't just |config| enough?

[Brian Ellis, 2012/07/23 23:30:26]

Probably, yeah.  I wasn't sure whether dictionary merging had been performed at
this point, but apparently so.

+        library_dirs += config.get('library_dirs', [])
+        ldflags += [('-L%s' % library_dir) for library_dir in library_dirs]
         self.WriteList(ldflags, 'LDFLAGS_%s' % configname)
       libraries = spec.get('libraries')
       if libraries: