dpogue commented on PR #939: URL: https://github.com/apache/cordova-plugin-camera/pull/939#issuecomment-3625264252
The actual requirements that are enforced are as defined in the plugin.xml file `engine` elements. The idea with the list of cordovaDependencies in package.json is that if you did `cordova plugin add cordova-plugin-camera`, the CLI tooling would look at the cordovaDependencies of the latest version of cordova-plugin-camera and check your local platform/tooling versions and then figure out which version of the plugin was actually compatible and install that one, rather than defaulting to the latest. I don't think that ever got fully implemented in the tooling through. The platform versions should **not** be listed as `dependencies` because that would cause other platforms to potentially be downloaded when they aren't needed (i.e. an app that only targets Android could end up getting cordova-ios pulled in as an unnecessary dependency). -- 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]
