breautek commented on issue #661: URL: https://github.com/apache/cordova-cli/issues/661#issuecomment-4184172081
Yah I'm not sure if I'd consider this a bug. `www` directory should be reserved for files that are intended to be packages in your app for your runtime. node packages are typically not consumable by a traditional browser/webview environment and often require additional tooling to bundle any content. Additionally node_modules may have several other dependencies related to development but not necessary for your runtime. But that could be subjective. The root issue is following symlinks. If we copied the symlinks as is, it may or may not work during the runtime, honestly not quite sure if webviews will follow symlinks. Even if it does, the symlink reference would need to be relative. If we resolved symlinks ahead of time and copied the canonical path, if your directory structure makes heavy use of symlinks it would be creating a lot of duplicate files. This would also fail if there is a circular reference in your symlinks. Lastly symlinks can make it easy to bypass security mechanisms, so adding support for them isn't as easy as simply resolving symlinks. So for those reasons I think we should consider symlinks as unsupported, and I don't think there was ever a time where symlinks was actually supported in full or even tested against. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
