Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Commit

Permalink
scripts/build.js: line 21, col 21, 'options' is defined but never used.
Browse files Browse the repository at this point in the history
  • Loading branch information
saper committed Aug 6, 2015
1 parent 290edc5 commit 40d903a
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions scripts/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ require('../lib/extensions');
* @api private
*/

function afterBuild(options) {
function afterBuild() {
var install = process.sass.binaryPath;
var target = path.join(__dirname, '..', 'build', process.config.target_defaults.default_configuration ? 'Debug' : 'Release', 'binding.node');
var target = path.join(__dirname, '..', 'build', process.config.target_defaults.default_configuration, 'binding.node');

mkdir(path.dirname(install), function(err) {
if (err && err.code !== 'EEXIST') {
Expand All @@ -30,6 +30,7 @@ function afterBuild(options) {

fs.stat(target, function(err) {
if (err) {
console.log('process.config.target_defaults.default_configuration: ', process.config.target_defaults.default_configuration);
console.error('Build succeeded but target not found');
return;
}
Expand Down Expand Up @@ -141,7 +142,7 @@ function build(options) {

proc.on('exit', function(errorCode) {
if (!errorCode) {
afterBuild(options);
afterBuild();

return;
}
Expand Down

0 comments on commit 40d903a

Please sign in to comment.