Fix build in cases where AMENT_PREFIX_PATH has more than two items (#517)

Fix cases where AMENT_PREFIX_PATH has >2 items

The regular expression transform on AMENT_PREFIX_PATH would only
replace the first ':' in many places, resulting in it not using the
remaining entries if there are more than two. '/g' makes the regular
expression 'global' so it will replace more than one occurrence per
line.

Fix #521

Author: amikhalev

README.md

@@ -177,6 +177,7 @@ Special thanks to the people who contribute.
 
 * [martins-mozeiko](https://github.com/martins-mozeiko)
 * [Teo Koon Peng](https://github.com/koonpeng)
+* [Alex Mikhalev](https://github.com/amikhalev)
 
 ## License
 

binding.gyp

@@ -52,10 +52,10 @@
               '-std=c++14'
             ],
             'include_dirs': [
-              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/, '/include/ ') + '/include/')\")",
+              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/g, '/include/ ') + '/include/')\")",
             ],
             'library_dirs': [
-              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/, '/lib/ ') + '/lib/')\")",
+              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/g, '/lib/ ') + '/lib/')\")",
             ],
           }
         ],
@@ -70,7 +70,7 @@
             ],
             'include_dirs': [
               './src/third_party/dlfcn-win32/',
-              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/;/, '\\\include ').replace(/\\\/g, '/') + '/include')\")",
+              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/;/g, '\\\include ').replace(/\\\/g, '/') + '/include')\")",
             ],
             'sources': [
               './src/third_party/dlfcn-win32/dlfcn.c',
@@ -81,7 +81,7 @@
               },
               'VCLinkerTool': {
                 'AdditionalDependencies': ['psapi.lib'],
-                'AdditionalLibraryDirectories': ["<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/;/, '\\\lib ').replace(/\\\/g, '/') + '/lib')\")",],
+                'AdditionalLibraryDirectories': ["<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/;/g, '\\\lib ').replace(/\\\/g, '/') + '/lib')\")",],
               }
             }
           }
@@ -93,10 +93,10 @@
               'OS_MACOS'
             ],
             'include_dirs': [
-              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/, '/include/ ') + '/include/')\")",
+              "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/g, '/include/ ') + '/include/')\")",
             ],
             'library_dirs': [
-                "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/, '/lib/ ') + '/lib/')\")",
+                "<!@(node -e \"console.log(process.env.AMENT_PREFIX_PATH.replace(/:/g, '/lib/ ') + '/lib/')\")",
             ],
             'xcode_settings': {
               'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',