Re-enable plugin tests again partially

Author: mgreter

.travis.yml

@@ -61,6 +61,11 @@ matrix:
 
 script:
   - ./script/ci-build-libsass
+  - ./script/ci-build-plugin math
+  - ./script/ci-build-plugin glob
+  - ./script/ci-build-plugin digest
+  - ./script/ci-build-plugin img-size
+  - ./script/ci-build-plugin tests
 before_install: ./script/ci-install-deps
 install: ./script/ci-install-compiler
 after_success: ./script/ci-report-coverage

script/ci-build-plugin

@@ -34,13 +34,7 @@ fi
 
 mkdir -p plugins
 if [ ! -d plugins/libsass-${PLUGIN} ] ; then
-  if [ "$PLUGIN" == "math" ]; then
-    git clone https://github.com/mgreter/libsass-${PLUGIN} plugins/libsass-${PLUGIN} --branch feature/libsass-3.7
-  elif [ "$PLUGIN" == "tests" ]; then
-    git clone https://github.com/mgreter/libsass-${PLUGIN} plugins/libsass-${PLUGIN} --branch feature/libsass-4.0
-  else
-    git clone https://github.com/mgreter/libsass-${PLUGIN} plugins/libsass-${PLUGIN}
-  fi
+  git clone https://github.com/mgreter/libsass-${PLUGIN} plugins/libsass-${PLUGIN} --branch feature/libsass-4.0
 fi
 if [ ! -d plugins/libsass-${PLUGIN}/build ] ; then
   mkdir plugins/libsass-${PLUGIN}/build
@@ -56,11 +50,11 @@ cd ../../..
 
 # glob only works on paths relative to imports
 if [ "x$PLUGIN" == "xglob" ]; then
-  ${SASSC_BIN} --precision 5 --plugin-path plugins/libsass-${PLUGIN}/build ${SASS_SPEC_SPEC_DIR}/basic/input.scss > ${SASS_SPEC_SPEC_DIR}/basic/result.css
-  ${SASSC_BIN} --precision 5 --plugin-path plugins/libsass-${PLUGIN}/build ${SASS_SPEC_SPEC_DIR}/basic/input.scss --sourcemap > /dev/null
+  ${SASSC_BIN} --precision 10 --plugin-path plugins/libsass-${PLUGIN}/build ${SASS_SPEC_SPEC_DIR}/basic/input.scss > ${SASS_SPEC_SPEC_DIR}/basic/result.css
+  # ${SASSC_BIN} --precision 10 --plugin-path plugins/libsass-${PLUGIN}/build ${SASS_SPEC_SPEC_DIR}/basic/input.scss --sourcemap > /dev/null
 else
   cat ${SASS_SPEC_SPEC_DIR}/basic/input.scss | ${SASSC_BIN} --precision 5 --plugin-path plugins/libsass-${PLUGIN}/build -I ${SASS_SPEC_SPEC_DIR}/basic > ${SASS_SPEC_SPEC_DIR}/basic/result.css
-  cat ${SASS_SPEC_SPEC_DIR}/basic/input.scss | ${SASSC_BIN} --precision 5 --plugin-path plugins/libsass-${PLUGIN}/build -I ${SASS_SPEC_SPEC_DIR}/basic --sourcemap > /dev/null
+  # cat ${SASS_SPEC_SPEC_DIR}/basic/input.scss | ${SASSC_BIN} --precision 5 --plugin-path plugins/libsass-${PLUGIN}/build -I ${SASS_SPEC_SPEC_DIR}/basic --sourcemap > /dev/null
 fi
 RETVAL=$?; if [ "$RETVAL" != "0" ]; then exit $RETVAL; fi
 

src/compiler.cpp

@@ -667,7 +667,7 @@ namespace Sass {
       else if (endsWithIgnoreCase(abs_path, ".scss", 5)) {
         format = SASS_IMPORT_SCSS;
       }
-      else {
+      else if (abs_path != "stream://stdin") {
         throw Exception::UnknwonImport(callStack);
       }
     }

src/eval.cpp

@@ -1272,11 +1272,9 @@ namespace Sass {
     ExternalCallable* def = static_cast<ExternalCallable*>(fn);
     SassFunctionLambda lambda = def->lambda();
     struct SassValue* c_args = sass_make_list(SASS_COMMA, false);
-    std::cerr << "Create c_args with " << Value::unwrap(c_args).refcount << "\n";
     sass_list_push(c_args, Value::wrap(message));
     struct SassValue* c_val = lambda(
       c_args, compiler.wrap());
-    std::cerr << "After c_args with " << Value::unwrap(c_args).refcount << "\n";
     sass_delete_value(c_args);
     sass_delete_value(c_val);
   }
@@ -1833,10 +1831,8 @@ namespace Sass {
       return toCss(interpolation->value(), false);
     }
     else if (SupportsDeclaration* declaration = condition->isaSupportsDeclaration()) {
-      sass::string strm("(");
-      strm += toCss(declaration->feature()); strm += ": ";
-      strm += toCss(declaration->value()); strm += ")";
-      return strm;
+      return "(" + toCss(declaration->feature()) + ": "
+             + toCss(declaration->value()) + ")";
     }
     else if (SupportsFunction* function = condition->isaSupportsFunction()) {
       return acceptInterpolation(function->name(), false)