Merge pull request gulpjs#924 from asfktz/patch-1

suggesting a cleaner way

Author: Eric Schoffstall

docs/recipes/browserify-uglify-sourcemap.md

@@ -9,8 +9,7 @@ Browserify with transforms and full sourcemaps that resolve to the original indi
 
 var browserify = require('browserify');
 var gulp = require('gulp');
-var source = require('vinyl-source-stream');
-var buffer = require('vinyl-buffer');
+var transform = require('vinyl-transform');
 var uglify = require('gulp-uglify');
 var sourcemaps = require('gulp-sourcemaps');
 
@@ -20,25 +19,19 @@ var getBundleName = function () {
   return version + '.' + name + '.' + 'min';
 };
 
-gulp.task('javascript', function() {
-
-  var bundler = browserify({
-    entries: ['./app.js'],
-    debug: true
+gulp.task('javascript', function () {
+  // transform regular node stream to gulp (buffered vinyl) stream 
+  var browserified = transform(function(filename) {
+    var b = browserify(filename);
+    return b.bundle();
   });
-
-  var bundle = function() {
-    return bundler
-      .bundle()
-      .pipe(source(getBundleName() + '.js'))
-      .pipe(buffer())
-      .pipe(sourcemaps.init({loadMaps: true}))
+  
+  return gulp.src('./app.js')
+    .pipe(browserified)
+    .pipe(sourcemaps.init({loadMaps: true}))
         // Add transformation tasks to the pipeline here.
         .pipe(uglify())
-      .pipe(sourcemaps.write('./'))
-      .pipe(gulp.dest('./dist/js/'));
-  };
-
-  return bundle();
+    .pipe(sourcemaps.write('./'))
+    .pipe(gulp.dest('./dist/js/'));
 });
 ```