mikeseven / node-opencl

Low-level OpenCL 1.x and 2.x bindgings for node.js
156 stars 33 forks source link

binding.gyp in linux #37

Closed milhidaka closed 8 years ago

milhidaka commented 8 years ago

In linux, environment variable should be used with "$". Please consider fixing it.

diff --git a/binding.gyp b/binding.gyp
index 912d172..6a4f120 100644
--- a/binding.gyp
+++ b/binding.gyp
@@ -44,7 +44,7 @@
         ['OS in "linux freebsd openbsd solaris android"', {
           'variables' : {
             # AMD APP SDK
-            'OPENCL_SDK' : '<!(echo %AMDAPPSDKROOT%)',
+            'OPENCL_SDK' : '<!(echo $AMDAPPSDKROOT)',
             'OPENCL_SDK_INCLUDE' : '<(OPENCL_SDK)/include',
             'OPENCL_SDK_LIB' : '<(OPENCL_SDK)/lib/x86_64',
           },
mikeseven commented 8 years ago

fixed, thanks!