Revert "major refactor"

This reverts commit b577f579.

README.md

@@ -52,7 +52,7 @@ The API for using node-sass has changed, so that now there is only one variable
 [Important: currently the argument `outputStyle` has some problem which may cause the output css becomes nothing because of the libsass, so you should not use it now!]
 
 #### sourceComments
-`sourceComments` is a `String` to determine what debug information is included in the output file. Its value should be one of `'none', 'normal', 'map'`. The default is `'none'`.
+`sourceComments` is a `String` to determine what debug information is included in the output file. Its value should be one of `'none', 'normal'`. The default is `'none'`.
 [Important: `souceComments` is only supported when using the `file` option, and does nothing when using `data` flag.]
 
 ### Examples
@@ -92,13 +92,18 @@ var server = connect.createServer(
     , dest: __dirname + '/public'
     , debug: true
     , outputStyle: 'compressed'
+    , prefix:  '/prefix'
   }),
-  connect.static(__dirname + '/public')
+  connect.static('/prefix', __dirname + '/public')
 );
 ```
 
 Heavily inspired by <https://github.com/LearnBoost/stylus>
 
+## DocPad Plugin
+
+[@jking90](https://github.com/jking90) wrote a [DocPad](http://docpad.org/) plugin that compiles `.scss` files using node-sass: <https://github.com/jking90/docpad-plugin-nodesass>
+
 ## Grunt extension
 
 [@sindresorhus](https://github.com/sindresorhus/) has created a set of grunt tasks based on node-sass: <https://github.com/sindresorhus/grunt-sass>
@@ -115,10 +120,10 @@ Check out the project:
 
     git clone https://github.com/andrew/node-sass.git
     cd node-sass
-    npm install
-    npm install -g node-gyp
     git submodule init
     git submodule update
+    npm install
+    npm install -g node-gyp
     node-gyp rebuild
 
 Replace the prebuild binary with your newly generated one
@@ -141,6 +146,14 @@ Output will be saved with the same name as input SASS file into the current work
       --include-path     Path to look for @import-ed files                      [default: cwd]
       --help, -h         Print usage info
 
+## Post-install Build
+
+Install runs a series of Mocha tests to see if your machine can use the pre-built `libsass` which will save some time during install. If any tests fail it will build from source.
+
+If you know the pre-built version will work and do not want to wait for the tests to run you can skip the tests by setting the environment variable `SKIP_NODE_SASS_TESTS` to true.
+
+      SKIP_NODE_SASS_TESTS=true npm install
+
 ## Contributors
 Special thanks to the following people for submitting patches:
 
@@ -148,6 +161,7 @@ Dean Mao
 Brett Wilkins
 litek
 gonghao
+Dylan Greene
 
 ### Note on Patches/Pull Requests
 

binding.gyp

@@ -6,6 +6,7 @@
         'binding.cpp',
         'sass_context_wrapper.cpp',
         'libsass/ast.cpp',
+        'libsass/base64vlq.cpp',
         'libsass/bind.cpp',
         'libsass/constants.cpp',
         'libsass/context.cpp',
@@ -24,13 +25,17 @@
         'libsass/prelexer.cpp',
         'libsass/sass.cpp',
         'libsass/sass_interface.cpp',
+        'libsass/source_map.cpp',
         'libsass/to_c.cpp',
         'libsass/to_string.cpp',
-        'libsass/source_map.cpp',
         'libsass/units.cpp'
       ],
+      'include_dirs': [
+        '<!(node -e "require(\'nan\')")'
+      ],
       'cflags!'   : [ '-fno-exceptions' ],
       'cflags_cc!': [ '-fno-exceptions' ],
+      'cflags_cc' : [ '-fexceptions', '-frtti' ],
       'conditions': [
         ['OS=="mac"', {
           'xcode_settings': {
@@ -38,12 +43,6 @@
             'GCC_ENABLE_CPP_RTTI': 'YES',
             'MACOSX_DEPLOYMENT_TARGET': '10.7'
           }
-         }],
-        ['OS=="linux"', {
-          'cflags_cc': [
-            '-fexceptions',
-            '-frtti'
-          ]
          }]
       ]
     }

package.json

@@ -2,7 +2,7 @@
   "author": "Andrew Nesbitt <andrewnez@gmail.com> (http://andrew.github.com)",
   "name": "node-sass",
   "description": "wrapper around libsass",
-  "version": "0.5.4",
+  "version": "0.7.0",
   "homepage": "https://github.com/andrew/node-sass",
   "keywords": [
     "sass",
@@ -24,8 +24,9 @@
     "url": "git://github.com/andrew/node-sass.git"
   },
   "scripts": {
-    "install": "node rebuild.js",
-    "test": "mocha test"
+    "install": "node build.js",
+    "test": "mocha test",
+    "prepublish": "bash scripts/prepublish.sh"
   },
   "bin": {
     "node-sass": "bin/node-sass"
@@ -36,10 +37,10 @@
   },
   "dependencies": {
     "mkdirp": "0.3.x",
-    "colors": "0.6.0-1",
-    "optimist": "0.4.x"
-  },
-  "devDependencies": {
-    "mocha": "1.7.x"
+    "optimist": "0.6.x",
+    "node-watch": "0.3.x",
+    "mocha": "1.13.x",
+    "chalk": "~0.3.0",
+    "nan": "~0.6.0"
   }
 }

sass.js

 var binding;
 var fs = require('fs');
+var path = require('path');
+
+var v8 = 'v8-' + /[0-9]+\.[0-9]+/.exec(process.versions.v8)[0];
+var modPath = path.join(__dirname, 'bin', process.platform + '-' + process.arch + '-' + v8, 'binding');
 try {
   if (fs.realpathSync(__dirname + '/build')) {
     // use the build version if it exists
     binding = require(__dirname + '/build/Release/binding');
   }
 } catch (e) {
-  // default to a precompiled binary if no build exists
-  var platform_full = process.platform+'-'+process.arch;
-  binding = require(__dirname + '/precompiled/'+platform_full+'/binding');
-}
-if (binding === null) {
-  throw new Error('Cannot find appropriate binary library for node-sass');
+  try {
+    fs.realpathSync(modPath + '.node');
+    binding = require(modPath);
+  } catch (ex) {
+    // No binary!
+    throw new Error('`'+ modPath + '.node` is missing. Try reinstalling `node-sass`?');
+  }
 }
 
 var SASS_OUTPUT_STYLE = {
@@ -29,11 +34,11 @@ var SASS_SOURCE_COMMENTS = {
 };
 
 var prepareOptions = function(options) {
-  var paths, style, comments;
-  options = typeof options !== 'object' ? {} : options;
-  paths = options.include_paths || options.includePaths || [];
-  style = SASS_OUTPUT_STYLE[options.output_style || options.outputStyle] || 0;
-  comments = SASS_SOURCE_COMMENTS[options.source_comments || options.sourceComments] || 0;
+  var paths, style;
+  var options = typeof options !== 'object' ? {} : options;
+  var paths = options.include_paths || options.includePaths || [];
+  var style = SASS_OUTPUT_STYLE[options.output_style || options.outputStyle] || 0;
+  var comments = SASS_SOURCE_COMMENTS[options.source_comments || options.sourceComments] || 0;
 
   return {
     paths: paths,
@@ -44,13 +49,7 @@ var prepareOptions = function(options) {
 
 var deprecatedRender = function(css, callback, options) {
   options = prepareOptions(options);
-  var errCallback = function(err) {
-    callback(err);
-  };
-  var oldCallback = function(css, err) {
-    callback(null, css);
-  };
-  return binding.render(css, oldCallback, errCallback, options.paths.join(':'), options.style, options.comments);
+  return binding.oldRender(css, callback, options.paths.join(':'), options.style, options.comments);
 };
 
 var deprecatedRenderSync = function(css, options) {
@@ -69,11 +68,11 @@ exports.render = function(options) {
   options.error = options.error || function(){};
 
   if (options.file !== undefined && options.file !== null) {
-    return binding.renderFile(options.file, options.success, options.error, newOptions.paths.join(':'), newOptions.style, newOptions.comments);
+    return binding.renderFile(options.file, options.success, options.error, newOptions.paths.join(path.delimiter), newOptions.style, newOptions.comments);
   }
 
   //Assume data is present if file is not. binding/libsass will tell the user otherwise!
-  return binding.render(options.data, options.success, options.error, newOptions.paths.join(":"), newOptions.style);
+  return binding.render(options.data, options.success, options.error, newOptions.paths.join(path.delimiter), newOptions.style);
 };
 
 exports.renderSync = function(options) {
@@ -86,11 +85,11 @@ exports.renderSync = function(options) {
   newOptions = prepareOptions(options);
 
   if (options.file !== undefined && options.file !== null) {
-    return binding.renderFileSync(options.file, newOptions.paths.join(':'), newOptions.style, newOptions.comments);
+    return binding.renderFileSync(options.file, newOptions.paths.join(path.delimiter), newOptions.style, newOptions.comments);
   }
 
   //Assume data is present if file is not. binding/libsass will tell the user otherwise!
-  return binding.renderSync(options.data, newOptions.paths.join(":"), newOptions.style);
+  return binding.renderSync(options.data, newOptions.paths.join(path.delimiter), newOptions.style);
 };
 
 exports.middleware = require('./lib/middleware');

sass_context_wrapper.cpp

@@ -12,7 +12,24 @@ extern "C" {
   void sass_free_context_wrapper(sass_context_wrapper* ctx_w)
   {
     if (ctx_w->ctx) sass_free_context(ctx_w->ctx);
-    if (ctx_w->fctx) sass_free_file_context(ctx_w->fctx);
+
+    delete ctx_w->callback;
+    delete ctx_w->errorCallback;
+
+    free(ctx_w);
+  }
+
+  sass_file_context_wrapper* sass_new_file_context_wrapper()
+  {
+    return (sass_file_context_wrapper*) calloc(1, sizeof(sass_file_context_wrapper));
+  }
+
+  void sass_free_file_context_wrapper(sass_file_context_wrapper* ctx_w)
+  {
+    if (ctx_w->ctx) sass_free_file_context(ctx_w->ctx);
+
+    delete ctx_w->callback;
+    delete ctx_w->errorCallback;
 
     free(ctx_w);
   }

sass_context_wrapper.h

 #include "libsass/sass_interface.h"
-#include <node.h>
+#include <nan.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -7,15 +7,24 @@ extern "C" {
 
 struct sass_context_wrapper {
   sass_context* ctx;
-  sass_file_context* fctx;
   uv_work_t request;
-  v8::Persistent<v8::Function> callback;
-  v8::Persistent<v8::Function> errorCallback;
+  NanCallback* callback;
+  NanCallback* errorCallback;
 };
 
 struct sass_context_wrapper*      sass_new_context_wrapper(void);
 void sass_free_context_wrapper(struct sass_context_wrapper* ctx);
 
+struct sass_file_context_wrapper {
+  sass_file_context* ctx;
+  uv_work_t request;
+  NanCallback* callback;
+  NanCallback* errorCallback;
+};
+
+struct sass_file_context_wrapper*      sass_new_file_context_wrapper(void);
+void sass_free_file_context_wrapper(struct sass_file_context_wrapper* ctx);
+
 #ifdef __cplusplus
 }
 #endif

test.js

@@ -10,20 +10,7 @@ var scssStr = '#navbar {\
     a {\
       font-weight: bold; }}';
 
-//sass.render(scssStr, function(err, css){
-  //console.log(css)
-//})
+sass.render(scssStr, function(err, css){
+  console.log(css)
+})
 
-//console.log(sass.renderSync(scssStr));
-console.log(sass.renderSync(scssStr));
-
-//console.log(sass.renderSync({
-      //file: './test/sample.scss'
-//}));
-
-
-sass.render({
-  file: './test/sample.scss'
-}, function(output) {
-  console.log(output);
-});