From a4bfcb890dc97f3e0801d8d824d0653e31ab3d44 Mon Sep 17 00:00:00 2001 From: Benjamin Pasero Date: Mon, 16 Nov 2015 16:10:25 +0100 Subject: [PATCH] real fix for: gulp-symdest does not preserve links on electron #2 --- gulpfile.vscode.js | 40 +--------------------------------------- package.json | 4 ++-- 2 files changed, 3 insertions(+), 41 deletions(-) diff --git a/gulpfile.vscode.js b/gulpfile.vscode.js index b9fa49f7c2e..f8e781f6d87 100644 --- a/gulpfile.vscode.js +++ b/gulpfile.vscode.js @@ -11,8 +11,7 @@ var path = require('path'); var es = require('event-stream'); var azure = require('gulp-azure-storage'); var electron = require('gulp-atom-electron'); -var mkdirp = require('mkdirp'); -var vfs = require('vinyl-fs'); +var symdest = require('gulp-symdest'); var rename = require('gulp-rename'); var filter = require('gulp-filter'); var json = require('gulp-json-editor'); @@ -39,10 +38,6 @@ var baseModules = [ // Build -var builtInExtensions = { - // nothing yet -}; - var vscodeEntryPoints = _.flatten([ buildfile.entrypoint('vs/workbench/workbench.main'), buildfile.base, @@ -138,33 +133,6 @@ function mixinProduct() { })); } -// Writes to destination with support for symlinks as they can appear in Electron on Mac -function symdest(out) { - var pass = es.through(); - - return es.duplex(pass, - pass.pipe(es.mapSync(function (f) { - if (!f.symlink) { - return f; - } - - var dest = path.join(out, f.relative); - try { - mkdirp.sync(path.dirname(dest)); - } catch (error) { - // Folder exists - } - - try { - fs.symlinkSync(f.symlink, dest); - } catch (error) { - console.error('Problem writing symlink: ' + error); - } - })) - .pipe(vfs.dest(out)) - ); -}; - function packageTask(platform, arch, opts) { opts = opts || {}; @@ -215,11 +183,6 @@ function packageTask(platform, arch, opts) { .pipe(util.cleanNodeModule('fsevents', ['binding.gyp', 'fsevents.cc', 'build/**', 'src/**', 'test/**'], true)) .pipe(util.cleanNodeModule('oniguruma', ['binding.gyp', 'build/**', 'src/**', 'deps/**'], true)); - var extraExtensions = util.downloadExtensions(builtInExtensions) - .pipe(rename(function (p) { - p.dirname = path.posix.join('extensions', p.dirname); - })); - var resources = gulp.src('resources/*', { base: '.' }); if (platform === 'win32') { @@ -235,7 +198,6 @@ function packageTask(platform, arch, opts) { license, sources, deps, - extraExtensions, resources ).pipe(util.skipDirectories()); diff --git a/package.json b/package.json index 26718fb8eae..25bffc16805 100644 --- a/package.json +++ b/package.json @@ -54,6 +54,7 @@ "gulp-tsb": "^1.7.3", "gulp-uglify": "^1.4.1", "gulp-util": "^3.0.6", + "gulp-symdest": "^1.0.0", "gulp-vinyl-zip": "^1.1.0", "gulp-watch": "^4.2.4", "innosetup-compiler": "^5.5.60", @@ -73,8 +74,7 @@ "source-map": "^0.4.4", "uglify-js": "2.4.8", "underscore": "^1.8.2", - "vinyl": "^0.4.5", - "vinyl-fs": "^2.2.1" + "vinyl": "^0.4.5" }, "repository": { "type": "git",