Thanks Darryl for the detailed explanation. Do you mind if we would raise a new issue in <https://github.com/apache/cordova/issues> so that we can track and discuss this idea in one place?
(I think we don't really use cordova-discuss any more, and the existing cordova-common PR does not look like the right place to track and discuss.) In general I would not favor considering this idea for Cordova 9, would be happy to give my reasons on GitHub. I think it would be best if you would be willing to raise the new issue. I would also be happy to raise the issue if needed. On Tue, Nov 20, 2018 at 12:52 PM Darryl Pogue <dvpdin...@gmail.com> wrote: > It's not quite as simple as dropping config.xml. In all cases, > config.xml will still need to exist to provide configuration > information about the app. > > Historically, when installing platforms and plugins with `--save`, > they were added to config.xml. When running `cordova prepare`, it > would read config.xml and restore any that were listed. > Now that we're using npm for all of our fetching, it makes more sense > to store those platform/plugin dependencies in package.json. > > Relevant proposals: > > https://github.com/apache/cordova-discuss/blob/master/proposals/save-restore.md > https://github.com/apache/cordova-discuss/pull/53 > > Currently, Cordova will attempt to use both package.json and > config.xml, and mirror installed plugins to both locations. > Unfortunately, handling of the spec value seems to be buggy depending > on which location it reads from. > > It would be great if Cordova 9 could drop the code that mirrors > changes to config.xml and use only package.json, but nobody has really > started work on that yet. The closest is an unfinished helper class > that I was hoping to add to cordova-common to assist with package.json > changes: https://github.com/apache/cordova-common/pull/34 > > I was also kinda hoping that this could all be tackled as part of the > `cordova install` proposal, but it probably makes sense to handle it > in smaller pieces. > > > On Tue, Nov 20, 2018 at 9:38 AM Chris Brody <chris.br...@gmail.com> wrote: > > > > My understanding is that Cordova 8 is using both config.xml and > > package.json. Any plugins and platforms specified in config.xml will be > > automatically added in package.json if not already there. My > understanding > > is that Cordova will drop config.xml in a new release, forget if this was > > planned for Cordova 9 or not. > > > > Can anyone confirm the following: > > > > Do we plan to drop config.xml support in Cordova 9? If so, any pointers > to > > where this was discussed and agreed? > > > > If not, do we plan to drop config.xml support in some other future > Cordova > > release? Any pointers to the discussion? > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > >
