Merge pull request #1801 from browserify/resolve-initial-link

Always resolve initial link
browserify · Feb 7, 2018 · 2fc049e · 2fc049e
2 parents 2b9b319 + 59e4642
commit 2fc049e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -503,7 +503,7 @@ Browserify.prototype._createDeps = function (opts) {
             }
             if (err) cb(err, file, pkg)
             else if (file) {
-                if (opts.preserveSymlinks && parent) {
+                if (opts.preserveSymlinks && parent.id !== self._mdeps.top.id) {
                     return cb(err, path.resolve(file), pkg, file)
                 }
 

diff --git a/test/preserve-symlinks.js b/test/preserve-symlinks.js
@@ -13,3 +13,15 @@ test('optionally preserves symlinks', function (t) {
         vm.runInNewContext(src, {});
     });
 });
+
+test('always resolve entry point symlink', function (t) {
+    t.plan(2);
+
+    var b = browserify(__dirname + '/preserve_symlinks/linked-entry.js', {preserveSymlinks: true});
+    b.bundle(function (err, buf) {
+        t.ok(!err);
+        t.ok(buf);
+        var src = buf.toString('utf8');
+        vm.runInNewContext(src, {});
+    });
+})
diff --git a/test/preserve_symlinks/linked-entry.js b/test/preserve_symlinks/linked-entry.js
@@ -0,0 +1 @@
+a/index.js