ninja: Add target_rpath generator flag

pylib/gyp/generator/ninja.py

Index: pylib/gyp/generator/ninja.py
diff --git a/pylib/gyp/generator/ninja.py b/pylib/gyp/generator/ninja.py
index ffb3a8567fa4ee16fdc43777dcd8bb44844fd6e0..e5ae14b78f5e13573547bbb22fe942de7af57494 100644
--- a/pylib/gyp/generator/ninja.py
+++ b/pylib/gyp/generator/ninja.py
@@ -379,6 +379,8 @@ class NinjaWriter(object):
     # should be used for linking.
     self.uses_cpp = False
 
+    self.target_rpath = generator_flags.get('target_rpath', r'\$$ORIGIN/lib/')

[sdefresne, 2016/03/18 14:12:40]

Instead of passing this as a generator flag, why not allow this to be
configurable in the .gyp files instead?

[Nico, 2016/03/18 14:17:49]

I feel that if you can get away with a less general thing, you should always do
that. Flexibility enables complexity. So if nobody needs this at a per-target
level, I wouldn't support it there.

[jbriance, 2016/03/18 14:20:37]

"output_dir" is a generator flag, so I thought this new "target_rpath" could be
a generator flag as well.
I also think that rpath is more a matter that should be up to the gyp caller. I
mean from same .gyp file, you might want two different rpath for two different
target systems.

[sdefresne, 2016/03/18 15:41:38]

Acknowdledged.

+
     self.is_mac_bundle = gyp.xcode_emulation.IsMacBundle(self.flavor, spec)
     self.xcode_settings = self.msvs_settings = None
     if self.flavor == 'mac':
@@ -1193,7 +1195,9 @@ class NinjaWriter(object):
         rpath = 'lib/'
         if self.toolset != 'target':
           rpath += self.toolset
-        ldflags.append(r'-Wl,-rpath=\$$ORIGIN/%s' % rpath)
+          ldflags.append(r'-Wl,-rpath=\$$ORIGIN/%s' % rpath)
+        else:
+          ldflags.append('-Wl,-rpath=%s' % self.target_rpath)
         ldflags.append('-Wl,-rpath-link=%s' % rpath)
     self.WriteVariableList(ninja_file, 'ldflags',
                            map(self.ExpandSpecial, ldflags))