From 8d234a22a08b33049d6ee60672ccbabb273af5a1 Mon Sep 17 00:00:00 2001
From: Konstantin Raev <bestander@gmail.com>
Date: Fri, 23 Jun 2017 11:20:52 -0700
Subject: [PATCH] WIP

---
 .../deps/a-1/package.json                        |   7 +++++++
 .../deps/b-1/b-1.0.0.tgz                         | Bin 0 -> 4862 bytes
 .../deps/b-2/package.json                        |   4 ++++
 .../deps/d-1/package.json                        |   7 +++++++
 .../hardlink-collision-no-bundled/package.json   |   7 +++++++
 .../install/transitive-file/package.json         |   4 +---
 src/package-linker.js                            |   6 +++++-
 7 files changed, 31 insertions(+), 4 deletions(-)
 create mode 100644 __tests__/fixtures/install/hardlink-collision-no-bundled/deps/a-1/package.json
 create mode 100644 __tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-1/b-1.0.0.tgz
 create mode 100644 __tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-2/package.json
 create mode 100644 __tests__/fixtures/install/hardlink-collision-no-bundled/deps/d-1/package.json
 create mode 100644 __tests__/fixtures/install/hardlink-collision-no-bundled/package.json

diff --git a/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/a-1/package.json b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/a-1/package.json
new file mode 100644
index 0000000000..689146054d
--- /dev/null
+++ b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/a-1/package.json
@@ -0,0 +1,7 @@
+{
+  "name": "a",
+  "version": "1.0.0",
+  "dependencies": {
+    "b": "file:../b-1/b-1.0.0.tgz"
+  }
+}
diff --git a/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-1/b-1.0.0.tgz b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-1/b-1.0.0.tgz
new file mode 100644
index 0000000000000000000000000000000000000000..b5a9eb1da7b27bc625d629adee8214ea6486aaa9
GIT binary patch
literal 4862
zcmZveWmME#xQB;E8VLyzgkk6ol^Q@0hVGP*?q(Deh90_0JW|r2G=fSa-CYtgIv`Sm
z#K18Bd(OG*)>`-dv_HMSz4qGc`8}LT<e-0t;lO4;VC`Y<q_5e8eDQ*{$jW<4U+!|J
z{JjA<<ilQDb7OP0mmq^N7~H6v;bB|HZ&SVcaiwzE?wb5s^(D=Vh4S}FB3~5=j2ij3
zwm<_vw&Xcgc5|?MyfWTwErS)o-ZXCKD>#`yof;Rn)ePrNaulw_aI>s<+Uq|+{rx7c
zkxaFFutcy>W@(%AUIu@wsVO*3>6`w^N94ogaZt?*bH$N~blj=_?K06v@=&FYSLdJH
z=lrFm!765_Rb5A)@B_PVd^Aba$9CREqRIZm?#wm_u<80LXgT0DpGX!E@r^pkX*)Z;
zQ@{{+OlN<6`7@qPQN@ti$j6ONuFF+Zw^4w?!&L=zm=Y>`CQPOk;12aj)?K@F72LE*
zKl)mDFuM~VIx~#96_A&#>sIS*{3N~nskJnRG0~A;I#YoEAIu;0bAv4Q$Fo5;dCrFE
za+WP<xeXcVuClD<#*_~=0&i2-XAGS{^hO#>VA%YbUI!xdFnIS>MVug9MkQCuITg)*
zKr>nOfomitK{HWw*g>Gu*9Q5da=pL>mAorfa=-dps&Jwpj^S%lkDIM>0YgJB<*s_8
z9vyWaMc6^YaZ21b%Ny=>AK)wRxt*a5wa#9cxs&DP-~n(%c%MnSE@!B0ws!u+(?eGx
zj&&F;E0F+SfkX|w)QlP!mp0qlCxiG*jov8nk$qy=PkPivZg+_T<hB<G@u#oegx|%&
zsgeSO_Uq+WkZWkDPH>11ek_x83_65(NltmGAHsF1pTn%YG8Dd*7QlFJoVby#-f}??
zG8To%lvi-`AMhG&1iH-4UeC=tbJKGW>q*P>l?cde*aRW7cFc<q?=;X$2dPZNSH8f;
zei;55aTQlA+&7#S(F8IjkeqOxon@!o^_8IJ@!(6B5bq>ln&&+eASROY>*LR7Go&*v
z5!n#*)MG9<fFDqKpiO-?t(@7ce&(;bN?FeH9w-`@BuTUefIe(O)+;k)gp<VfwHcoi
zdO(Z%BD`+BRX-w7Q{C(=X_H7U^}lRi7%iY1I;wv*M@t&p>50mtmW?Qw*=jBpVf7Ff
z?R(+mmbtzJ);7uRa)4Jam)MlA<q2n<^UIi4SDrCdugT=W5%CWmY^r2_DbweB%x3WJ
z3=+fpX(9StLb2r=1GL>occRBL49BA4b$Q}gd-N*Np&gyO6Pdr?7UyFBE^ns6-b}u|
z?}0*hhUmUX{!MY?{hUfavpQCd$52huSnO-kC{eMKe9~u9H0xi2pY_r1*v?@zM0N++
z<s@%PL*9zVY}m+>-%+erKQ+euBBWrTx{}VYluY+xDU6hI$vWnyxa+Ijz#NmZ|8)&?
z;WfCw-;4HhB3&AUI<t_8o0+2%?B!UHV{GobX}c_<iymc1L>BoPL9Zh}R~YXG$Omej
z*yidaKGkwCifUS3?I5?ZyIw41%x);T8O5(hWqZWbW%x3AuoJZ8v5Ex#uo)_~)lB)A
zCuV!|MaSMru5eq~5fK%u>L#`!OQCkmSwXG?mUT<p(%*WC5*eqyQ;|GvTldXNcK6nh
zjBucIq%f5FnEiaViDF`Pbn-hc?#<a_K1}cI7ICB!AVGzM9RWdW<UQ(vD@{g5!;6%R
z)(-EoB8yU=5&3hl&J}O=L6_m$aRIp`DVmu?^*)tP{X$}AU0E^~xG>L7z#J+@MulY}
zLYs7kP4e@0F>5E?b)KVNfO=2y`VOEQ*@Hun*3Ix1^>Zcc(7J^}QYJ2i1!AYUdd#+j
zWz{G0eR=)#Oq~r_Pf<MLy6m*NVi*mL$3v||iQL@?m{CB!g&&ohuEDLxGdcunE*7-6
z>BL<df1=WSKC2~-n~;9jj(BbvK;fg!s8K*y@IYGk#9>L-G4O@_Py*vc-qvcAmphZN
zHjRhGX<NIjH;Q<qv^2QaD_7z@4l~owCA2@!QMy=GM_kFwbAMlXpicb_w~>gGy}Ibf
zZ){bqb?*0SC&!}+3n$gl@r>GqMCCaqKYzXuh|JwFl(k|3PsH8n(@Rzq7R=wC2tE>z
z{R_q+*)BVV$sv^spZK6XBcL;`q4>Gm;U4#~5UC%<MMD(T39mX2bAJ`pPo_oz)v^6-
zY3=HIbcwB%xSln!KYvriFF_rYwrzDeL$^`!huE2B*0M2qVHp#aoadH_JwHS5eq{zd
z{Xpgexnpk^;4Nh$T5u3j5p096-+Ccy=V*3<@TOiwSmxPd&2!q~aYclMchSH(-9umq
z3!Mb~@oA@szknSaCmLu&0rQ7)IJ;Z?3gC17MPiuNG8C)y0OsDbZ2Gkg{@}2_?g##4
z^Uwf6Z}uCAlnJB(v_BCtWS{;`svqHZia)Z;#^!w4yCUl1L0gmWUjg;{g&aV@?A;aX
z?8XY9UfaWZR^|_&V*4ErC=p%&9l+EL;t&V~imrim0F4LQ@Ca8907{lYJ9cw~5`(#~
zhFt-lYP2N-*N?sfd9CYE{2IqEvA&uMs4>vp17udTnjQj2Jg)%75#2oTZY7F5aq?UF
z6^+_Z0`hPj8rmt(574R*GBPAG>KOm|V#;x-bwu6iiT~wx7-Rkey=A{A;uT6LyWU@6
zbj19>q$9bb{DOlQr20#aeTf_H!05FM88lrmzN*vxWxY==mEt8{n^)*kWvX^r$&8sP
z(#1%3m>pb)x9Y8usn)6|SC*-g516=+gt~wvtwgiba$?HlGWN>^%>wBS1^M<3RNuy{
z3sSjuW|{!U$G>qH)XDb!j=NE;9CDGm<MwoERd+J{c88|D6t#336gGOwQKN(r@CC5g
z%`IUf0#q^hb^5@$`>rWZJO%ESgraJ64{iJ$pD`5jOyS;V<8<l<7R|Fx9qVQAo$Snu
z&9S$%t*sF&4@t}TpVJgfcSya(<~w2M+{T{8hxpG>_>v;_BPoeICx>HD%KT~3t!I9C
zcg|-vcs`=1YuSyWz&p2siYm54fbJ2{T-1Z*oCN&v$W!QFz&4H+jd%z&ivaKH`vFwv
zdGT(M&&gq4O(Od#^t2wHb6M2J4G#({<XX)J1l&6ixR3uJfawU~{$g>73ZEC0aRe;u
zyWLmZj&Zts8QpW2zWp~@cINbv27^?wZUEbTBDKeo1lm7_=p{M@BEg{QnHD{6Xh`?>
zbb97L#xZ|LE%GxMdOzue8at#8+0RcDYA1N=6`<a7GK~EpFP4e1(-TYOC!ox&Avj&f
z5S1zYg=hQ@OQu7*rzv}D;*_oferJ7rEamWfG->&hOX#L*xuklYQV7#Uu^T3^sbYQv
z0e(UW<6t;sHST|AG)CEu&FfP49goO>yk>q{-D{!`$@&TYmG-Sd;~n!}GaG64GUFoE
z1Fy{fT!WOyZV%gegld%NGe_pjiUz~!Vh4uvYx$x;&dQV)2L92VgGTSakFl9%hNFtU
zs_)gjdK9R5?R>~yO~F}=3jb|M9vi^+p7iW}|ATtHx^04&=(2?<nv0$;4HSh@eR0wX
zN!>5=bSiTLZ`?JH2@aOIO(Fesd#kswgK*B@l-h8gi|zJQQ3TzqXWm;Gq%J;?1o!jg
zEewAUDI=0nKqb63jur1gcLIu|?6IUhb#b=p7jgr-SU*p9S-{+SYo#}zFQ(&;iPjmg
zllCjyqdkyt$>+!_r*qQRV4T1=+gVF->L=4d%VW8$gs*fHRYRip25(!&CYYGy{|KX<
z;Z%04K3~=(PSMZS5gFW18Jx*3BTuBe%jY;4{8Pq13_)d{b-ymZEUs%)<*JkpOsFK}
z6TgSN`eC!PCd)8ha7M<>A$|t^exNjxy*!yhv>%>H4Iv$8S|gxM4)%^{Cr9&SJrU6_
zVpAKHGWX(yFue!wfNF1F4Tq`j*)!w|QBAN!C$JFXiR0p-;=P^sQhSu!53`jeg&xjn
z=nAt~x>Zp9DdSzv2{j_E{Tb6(y|TMEqe{_r{kmzI^^vHwK>VL=$TX99Ny;~t@idLl
zR9DjJ%Xz8>9r37ONn(^5OfMwwO<GFTO757oRUHFr%#vh|h9@Q}S~WCcufB+UWXvU<
z<V}%o)6v#TIeUvdFpu-aki+{i;6BwlGkF~fSUtrRRcYgJQ=P>Bm%IO<xZ4{G_2$5f
zkz~*&;*AIX!e_Of|I~oqbnJCA;WlmlMy}5g;0=2{1WaQS2e92?fA9Fl2_S<ezMr>f
zHJd(q(RbTlF2{H|K`|X-ulio=KxI_ebRX>c%fp4-O-#$J5cTO%oO&H(giJrND(1NY
zRz&vOYLlW*aAue_)cj@bQ)>5GFE*#lnze<%>pE)p>n|L`0|8B)vFvE)^k9g*86l69
z8N=%uI$`N=I*svK>Wv11k}k|!;7LuE%kNrY4sZE``{Zutn#}hqJj+Ku#AM(|QUs6%
z={~o9TUY0EbUpclYFjb%s``Sy7A48@)A(UJ&C6F~=s8*85^C6+=g`$yL2X$ov+gu{
zUt4bJu_k3`xY~41;1x})`GvWe<(H?G2o>8E16${mMwK|Y(!HgL`{mSyak>G5Qje1A
zhOfi>Wcm|k2RYs}$i81Ht!nz@-t3bFn~?IJ?@|9SVbziC`=-KnyzqyZE;;QW{%4k|
zooRy8onVKYvK&*ri>D7PyxD7iXn*q`#@vO_n>O-oGbHMULQD;2xqM1hg{t}YmaEu8
zEj3Ei7$3}XCyXW++8|pBw?^sO=%K7;cI@kz2FbcGyj_!Z_alxD_Pj<6HqBNEh`zxi
z6I}^U(a_JdI4ms!-(LhAqoM!nxQ53g+ZVA2lK58^QxRo2P!Ak^uLxAGaj|z104nav
z;#2<o_(t6#7+)khwT*z82jj0fU*58(CX=9w7>Y>&HBjuK9V4YR)oUBl;FuB!r-N~(
z(0T`u_fG|DpzAl@7T$!_N}62(9-(+7WG_*82d{7X{RnZy>-S&P3;pjlJ_-U{DY!Xv
zkHpZGK&v`d=_8^>u6V--Jy{9-u!2VdaO71|Lm#jt<NbI2bBP832+qjtzLQ5G*O(RF
zj#nY7wqG7yk%c_R$9)NdlS-gb3vbZSqI>k0hyb`h=1!#Wt@4AMq50Bx_o&HJPrAvu
z5U8}uNGX;lW$+--Or};LK`s|?3|w?OK7b(9olcCm-oR;d<LE@b8|R)7>r60Cwe)+}
zU<YT1`Iy}X4ob^f(iojyK7_&AV&_X_cR_|0ue092D~8xO#UEJzR_Ky5cBCeF&ZF$d
zEj*^S$+seozS;n_<d0lWxE-%B%Vd*9jCzD1;a?O&ZpKJL;BcMP$*EL*G5x72(cz@i
zCGgwu>4M5TdG-25-MwJQ<rwy4FKO=G->S<f24VL^#GbtJB(d;L!rLtYu9QO|6Y|Rr
z@|S{uUDh(rJcuKuWHcP0ZJ@XSF6fwXh@Ag4o&S06KmgH+24>~(2v#*E;oHg+Fpy==
zVIWz=#)2uVljoMfV!+g;@AIl|av>bTnE=>10!~hWi`y<!!qy5cbuVrlPl_9`Sv@|A
zKlamTPp!}cA6BF~5{Vj=9p7cVjmVit!phNJVf2ZPIp;yevqi#QC}7Uu<plHL#CWe3
zfY`=8?-2R;x09#!{2v$l_lDi?f$qgqfP@a)pcw&IA=F%b5I{Houm53zQ5^QDY1ynD
z=*~NKJp-EiTagn^0<cu4$yD8+8nkunvKU}4)^YBikB8|y>UOtZ{a5|}pO|YjV7W&-
zdbTv$P4K+lmq5XOle7Pyn7^6VR57`OfVujw1Dj#_XTV9i15g*F=HEKfhA^2UN)8ls
zGXtkhqUs)!AK%?=GHH&j7RH9ORppXka%AslU&bb`H2>BLZI^7K6gM61c3Mp0Sn7&@
z8=qN#)@aeYvbErI5hrIT=))+2!W){l(jG>F<W)2FTwhatpMe>cLyW&Zj62<^pHK8#
zq3|p|1+ABC4Yb6yMlq_a75PGYl65V3EWNO(ah)jpg$>7FNJwF#;`@-s{fc1S@F%}0
zuXXScim-aK){0zhSDKxeoW3N<&L8qR;(D1hnqZdcU9N@KRTvG&yeG}8xdJs^;*a6>
ztqhrh*o@AHGr#41)S|6gi^@!C?N{zD)x6MNi2m}cT=x#lF&rkxczCaXJ*lS2@Qj6&
z{W|AU7G+8q&#u43u*IyQWHy%)hsp3Exq9VNO-ki&zUJfKBY%!(Q3Lj`)`&Ect#pwC
z*Mc;>a5<NN7yHO)7rK`+RVPMap0z~YLJPr5ixqVx)V^>t`T@ooHTsK|R+-MZ&ZAl!
vA?+H|*VfgO{Ut`O-d_c9PF3}<hfmeJ&nbF|Xccg^aRAx<kPRcyAPDpy2guA@

literal 0
HcmV?d00001

diff --git a/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-2/package.json b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-2/package.json
new file mode 100644
index 0000000000..57549744f3
--- /dev/null
+++ b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/b-2/package.json
@@ -0,0 +1,4 @@
+{
+  "name": "b",
+  "version": "2.0.0"
+}
diff --git a/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/d-1/package.json b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/d-1/package.json
new file mode 100644
index 0000000000..397ab11fc6
--- /dev/null
+++ b/__tests__/fixtures/install/hardlink-collision-no-bundled/deps/d-1/package.json
@@ -0,0 +1,7 @@
+{
+  "name": "d-1",
+  "version": "2.0.0",
+  "dependencies": {
+    "b": "file:../b-1/b-1.0.0.tgz"
+  }
+}
diff --git a/__tests__/fixtures/install/hardlink-collision-no-bundled/package.json b/__tests__/fixtures/install/hardlink-collision-no-bundled/package.json
new file mode 100644
index 0000000000..8ab2fe880e
--- /dev/null
+++ b/__tests__/fixtures/install/hardlink-collision-no-bundled/package.json
@@ -0,0 +1,7 @@
+{
+  "dependencies": {
+    "a": "file:deps/a-1",
+    "b": "file:deps/b-2",
+    "d": "file:deps/d-1"
+  }
+}
diff --git a/__tests__/fixtures/install/transitive-file/package.json b/__tests__/fixtures/install/transitive-file/package.json
index 8ab2fe880e..bf8183b7b7 100644
--- a/__tests__/fixtures/install/transitive-file/package.json
+++ b/__tests__/fixtures/install/transitive-file/package.json
@@ -1,7 +1,5 @@
 {
   "dependencies": {
-    "a": "file:deps/a-1",
-    "b": "file:deps/b-2",
-    "d": "file:deps/d-1"
+    "a": "file:deps/a-1"
   }
 }
diff --git a/src/package-linker.js b/src/package-linker.js
index 0540579093..3926f52107 100644
--- a/src/package-linker.js
+++ b/src/package-linker.js
@@ -96,12 +96,16 @@ export default class PackageLinker {
     }
 
     // link up the `bin` scripts in bundled dependencies
-    if (pkg.bundleDependencies) {
+    if (false && pkg.bundleDependencies) {
       for (const depName of pkg.bundleDependencies) {
         const loc = path.join(this.config.generateHardModulePath(ref), this.config.getFolder(pkg), depName);
 
+        console.log("!!!", loc)
+
         const dep = await this.config.readManifest(loc, remote.registry);
 
+        console.log("???")
+
         if (dep.bin && Object.keys(dep.bin).length) {
           deps.push({dep, loc});
         }