Hi! Thanks for the patch. Out of curiosity- Why is junction the default for Windows?

PS: The patch is not passing the tests because it's missing require('os').

Hi there,

I defaulted to junction because it's the only link that node.js supports for windows. The current behavior under windows is that if a link is detected while copying it will error when it reaches the link, and skip copying it entirely. Making it default to junction causes will likely make it closer to what the user would expect.

My specific use case is using cordova (Mobile app framework) which leverages your code to synchronize the file tree from the main set of files to the directories for each platform (android, ios, windows phone, etc). It works fine for most scenarios, but I wanted to have the source directory have links to other locations (think like project dependencies). I was thinking I was going to have to add symlink support to shelljs entirely, but was pleasantly surprised to see it already supported it, and it just needed a quick patch to make it work on windows.

I'm not super proficient with github's pull request system, but I'll make an attempt at updating it to pass tests and re-submit.

Thanks,
Jamon