-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Directory assets: symlinks #731
Comments
I think this would be welcomed upgrade. It could be as easy as switching out the current archiver library with a different one. There seems to be some chatter on the issues in archiver about supporting materializing symlinks however, it doesn't seem like anything concrete was released yet. For now I'm just leveraging my underlying |
+1 from here because I can't deploy my Node.js Lambda function which has some node_modules symlinked using |
This is a pretty important required feature. In nodejs its possible to create either links as @kennu said for testing or to actually put a hard dependency to a relative module such as:
Which will make a symlink to the I don't think it's really necessary to have options here, it needs to always do it. |
When building a nodejs lambda it is really common for dependencies under the `node_modules` folder to actually be symlink'd to other folders on your system. The symlinks will now be followed and the indicated files packaged (instead of the symlinks themselves). Fixes #731
Allow configuring how symlinks are treated:
The text was updated successfully, but these errors were encountered: