From a96905ef1a23d5cc25b2ea5cf78c60e625100a43 Mon Sep 17 00:00:00 2001
From: Marcin Cieslak <saper@saper.info>
Date: Sun, 15 Mar 2015 01:13:20 +0000
Subject: [PATCH] Build with the unbundled libsass

Use environment variables to control
how libsass is linked:

LIBSASS_EXT variable empty or unset

Build a bundled libsass source
(from src/libsass).
This is the default.

LIBSASS_EXT=auto

libsass is located using pkg-config
and linked dynamically

LIBSASS_EXT=yes
LIBSASS_CFLAGS=
LIBSASS_LDFLAGS=

Link libsass manually, by adding
compiler flags and linker flags
in LIBSASS_CFLAGS and LIBSASS_LDFLAGS,
respectively.

For example this way libsass can
be linked in statically (on Unix):

LIBSASS_EXT=yes
LIBSASS_CFLAGS=-I/usr/local/include
LIBSASS_LDFLAGS=/usr/local/lib/libsass.a

When using LIBSASS_EXT there is no need
to checkout the libsass source into src/libsass submodule
and the library from the package management
system can be used.

PR: #139
PR: #389
PR: #744
---
 binding.gyp                | 75 ++++++++++++------------------
 libsass.gyp                | 94 ++++++++++++++++++++++++++++++++++++++
 scripts/build.js           | 14 +++++-
 src/sass_context_wrapper.h |  2 +-
 4 files changed, 136 insertions(+), 49 deletions(-)
 create mode 100644 libsass.gyp

diff --git a/binding.gyp b/binding.gyp
index 80b9e4cc6..2ef8cb5f9 100644
--- a/binding.gyp
+++ b/binding.gyp
@@ -4,56 +4,37 @@
       'target_name': 'binding',
       'sources': [
         'src/binding.cpp',
-        'src/sass_context_wrapper.cpp',
-        'src/libsass/ast.cpp',
-        'src/libsass/base64vlq.cpp',
-        'src/libsass/bind.cpp',
-        'src/libsass/cencode.c',
-        'src/libsass/constants.cpp',
-        'src/libsass/context.cpp',
-        'src/libsass/contextualize.cpp',
-        'src/libsass/copy_c_str.cpp',
-        'src/libsass/error_handling.cpp',
-        'src/libsass/eval.cpp',
-        'src/libsass/expand.cpp',
-        'src/libsass/extend.cpp',
-        'src/libsass/file.cpp',
-        'src/libsass/functions.cpp',
-        'src/libsass/inspect.cpp',
-        'src/libsass/json.cpp',
-        'src/libsass/node.cpp',
-        'src/libsass/output_compressed.cpp',
-        'src/libsass/output_nested.cpp',
-        'src/libsass/parser.cpp',
-        'src/libsass/prelexer.cpp',
-        'src/libsass/remove_placeholders.cpp',
-        'src/libsass/sass.cpp',
-        'src/libsass/sass2scss.cpp',
-        'src/libsass/sass_context.cpp',
-        'src/libsass/sass_functions.cpp',
-        'src/libsass/sass_util.cpp',
-        'src/libsass/sass_values.cpp',
-        'src/libsass/source_map.cpp',
-        'src/libsass/to_c.cpp',
-        'src/libsass/to_string.cpp',
-        'src/libsass/units.cpp',
-        'src/libsass/utf8_string.cpp',
-        'src/libsass/util.cpp'
+        'src/sass_context_wrapper.cpp'
       ],
       'include_dirs': [
-        '<!(node -e "require(\'nan\')")'
-      ],
-      'cflags!': [
-        '-fno-exceptions'
-      ],
-      'cflags_cc!': [
-        '-fno-exceptions'
-      ],
-      'cflags_cc': [
-        '-fexceptions',
-        '-frtti'
+        '<!(node -e "require(\'nan\')")',
       ],
       'conditions': [
+        ['libsass_ext == ""', {
+            'dependencies': [
+              'libsass.gyp:libsass',
+            ]
+        }],
+	['libsass_ext == "auto"', {
+	  'cflags_cc': [
+	    '<!(pkg-config --cflags libsass)',
+	  ],
+	  'link_settings': {
+	    'ldflags': [
+	      '<!(pkg-config --libs libsass)',
+	    ],
+	  }
+	}],
+	['libsass_ext == "yes"', {
+	  'cflags_cc': [
+	    '<(libsass_cflags)',
+	  ],
+	  'link_settings': {
+	    'ldflags': [
+	      '<(libsass_ldflags)',
+	    ],
+	  }
+        }],
         ['OS=="mac"', {
           'xcode_settings': {
             'OTHER_CPLUSPLUSFLAGS': [
@@ -67,7 +48,7 @@
             'GCC_ENABLE_CPP_RTTI': 'YES',
             'MACOSX_DEPLOYMENT_TARGET': '10.7'
           }
-         }],
+        }],
         ['OS=="win"', {
           'msvs_settings': {
             'VCCLCompilerTool': {
diff --git a/libsass.gyp b/libsass.gyp
new file mode 100644
index 000000000..3d113d69e
--- /dev/null
+++ b/libsass.gyp
@@ -0,0 +1,94 @@
+{
+  'targets': [
+    {
+      'target_name': 'libsass',
+      'type': 'static_library',
+      'sources': [
+        'src/libsass/ast.cpp',
+        'src/libsass/base64vlq.cpp',
+        'src/libsass/bind.cpp',
+        'src/libsass/cencode.c',
+        'src/libsass/constants.cpp',
+        'src/libsass/context.cpp',
+        'src/libsass/contextualize.cpp',
+        'src/libsass/copy_c_str.cpp',
+        'src/libsass/error_handling.cpp',
+        'src/libsass/eval.cpp',
+        'src/libsass/expand.cpp',
+        'src/libsass/extend.cpp',
+        'src/libsass/file.cpp',
+        'src/libsass/functions.cpp',
+        'src/libsass/inspect.cpp',
+        'src/libsass/json.cpp',
+        'src/libsass/node.cpp',
+        'src/libsass/output_compressed.cpp',
+        'src/libsass/output_nested.cpp',
+        'src/libsass/parser.cpp',
+        'src/libsass/prelexer.cpp',
+        'src/libsass/remove_placeholders.cpp',
+        'src/libsass/sass.cpp',
+        'src/libsass/sass2scss.cpp',
+        'src/libsass/sass_context.cpp',
+        'src/libsass/sass_functions.cpp',
+        'src/libsass/sass_util.cpp',
+        'src/libsass/sass_values.cpp',
+        'src/libsass/source_map.cpp',
+        'src/libsass/to_c.cpp',
+        'src/libsass/to_string.cpp',
+        'src/libsass/units.cpp',
+        'src/libsass/utf8_string.cpp',
+        'src/libsass/util.cpp'
+      ],
+      'cflags!': [
+        '-fno-exceptions'
+      ],
+      'cflags_cc!': [
+        '-fno-exceptions'
+      ],
+      'cflags_cc': [
+        '-fexceptions',
+        '-frtti'
+      ],
+      'direct_dependent_settings': {
+        'include_dirs': [ 'src/libsass' ],
+      },
+      'conditions': [
+        ['OS=="mac"', {
+          'xcode_settings': {
+            'OTHER_CPLUSPLUSFLAGS': [
+              '-std=c++11',
+              '-stdlib=libc++'
+            ],
+            'OTHER_LDFLAGS': [
+              '-stdlib=libc++'
+            ],
+            'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
+            'GCC_ENABLE_CPP_RTTI': 'YES',
+            'MACOSX_DEPLOYMENT_TARGET': '10.7'
+          }
+         }],
+        ['OS=="win"', {
+          'msvs_settings': {
+            'VCCLCompilerTool': {
+              'AdditionalOptions': [
+                '/GR',
+                '/EHsc'
+              ]
+            }
+          },
+          'msvs_disabled_warnings': [
+            # conversion from `double` to `size_t`, possible loss of data
+            4244,
+            # decorated name length exceeded
+            4503
+          ]
+        }],
+        ['OS!="win"', {
+          'cflags_cc+': [
+            '-std=c++0x'
+          ]
+        }]
+      ]
+    }
+  ]
+}
diff --git a/scripts/build.js b/scripts/build.js
index 03f2e14b3..6e9f5013a 100644
--- a/scripts/build.js
+++ b/scripts/build.js
@@ -50,7 +50,19 @@ function afterBuild(options) {
  */
 
 function build(options) {
-  var proc = spawn(process.execPath, ['node_modules/pangyp/bin/node-gyp', 'rebuild'].concat(options.args), {
+  var arguments = [
+    path.join('node_modules', 'pangyp', 'bin', 'node-gyp'),
+    'rebuild',
+  ].concat(
+    [ 'libsass_ext', 'libsass_cflags',
+      'libsass_ldflags' ].map(function(optvar) {
+      return ["--", optvar, "=", process.env[optvar.toUpperCase() || '']].join('');
+    })
+  ).concat(options.args);
+
+  console.log(['Building:', process.execPath].concat(arguments).join(' '));
+
+  var proc = spawn(process.execPath, arguments, {
     stdio: [0, 1, 2]
   });
 
diff --git a/src/sass_context_wrapper.h b/src/sass_context_wrapper.h
index b9120eb17..ba412fcb4 100644
--- a/src/sass_context_wrapper.h
+++ b/src/sass_context_wrapper.h
@@ -1,7 +1,7 @@
 #include <stdlib.h>
 #include <nan.h>
 #include <condition_variable>
-#include "libsass/sass_context.h"
+#include <sass_context.h>
 
 #ifdef __cplusplus
 extern "C" {