I understood and read it too Gorkem. I was (poorly) suggesting we look at the issue of configuration in a complete view. Due to backwards compatibility we will be adding a new file and the code to support the old file will be around a while.
We can probably roll a whole lot more into a single file. What I"m not sure about is what should and should not be. On Thu, Jan 2, 2014 at 12:31 PM, Gorkem Ercan <gorkem.er...@gmail.com>wrote: > > Reducing the number of configuration files is actually the goal here. > > The abstraction is not a new one. It already exists and it is part of the > $PROJECT/.cordova/config.json. > I am suggesting to move it to $HOME/.cordova/config.json so that we no > longer need the > $PROJECT/.cordova/config.json and have only the cordova.xml to carry > project specific > properties. > -- > Gorkem > > > On Thu, Jan 02, 2014 at 12:16:57PM -0800, Brian LeRoux wrote: > > Considering http://wiki.apache.org/cordova/ConfigurationFiles I'm not > sure > > we want more config either. > > > > Perhaps we need to think more comprehensively rather than proposing more > > abstractions. > > > > > > On Thu, Jan 2, 2014 at 11:15 AM, Gorkem Ercan <gorkem.er...@gmail.com > >wrote: > > > > > > > > I think what I will describe here is more that what CLI provides today. > > > > > > An engine/lib has a version, id and a uri. On most cases, you only care > > > about the > > > id and uri and assume that the tools that you work with already knows > how > > > to > > > resolve the id and version to a location. In the case of CLI an engine > > > with id: cordova > > > and version:3.1.0 should be resolved to > ~/.cordova/lib/ios/cordova/3.1.0 . > > > If we have a > > > uri defined that actually means we want to overwrite the default > location > > > for the engine. > > > I think this redirection should be per engine not per project and > should > > > be located as part of > > > CLI's configuration > > > -- > > > Gorkem > > > > > > On Thu, Jan 02, 2014 at 01:28:12PM -0500, Andrew Grieve wrote: > > > > Hmm, good point about absolute paths. > > > > > > > > I think if you're using an override there though, that you could set > it > > > to > > > > a relative path for shared projects. Same thing with plugin search > paths. > > > > > > > > I think it'll be confusing to have a "cordova.xml" as well as a > > > > "cordova.json" file right in the root. > > > > > > > > WDYT? Other options? > > > > > > > > > > > > On Thu, Jan 2, 2014 at 1:06 PM, Ian Clelland <iclell...@chromium.org > > > > > wrote: > > > > > > > > > On Thu, Jan 2, 2014 at 10:22 AM, Andrew Grieve < > agri...@chromium.org> > > > > > wrote: > > > > > > > > > > > What cordova.json has that config.xml doesn't, is that you can > set > > > the > > > > > > location of platforms with it through: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > That said, I like your idea of having one top-level config file > > > instead > > > > > of > > > > > > two. I don't see why we couldn't just put these same settings > into a > > > > > > "cordova.xml". > > > > > > > > > > > > > > > > Wouldn't this make it impossible to share project configuration > between > > > > > developers? If your /Users/agrieve/.../ paths are in your config > xml, > > > > > you're going to have a bad time putting that in version control. > > > > > > > > > > -1 on having a single file to manage both application config and > > > > > build-environment config. > > > > > > > > > > +1 to the way I read Gorkem's original suggestion, which was to > > > coordinate > > > > > the move of the two files into a single issue and take care of it > all > > > at > > > > > once. > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Jan 1, 2014 at 5:05 PM, Gorkem Ercan < > gorkem.er...@gmail.com > > > > > > > > > > wrote: > > > > > > > > > > > > > There is also another proposal to move config.xml out of www. > Can > > > we > > > > > > merge > > > > > > > this two moves and > > > > > > > 1. remove .cordova > > > > > > > 2. remove config.json > > > > > > > 3. move config.xml to root > > > > > > > 4. rename config.xml to cordova.xml > > > > > > > > > > > > > > AFAIK config,json does not carry any information that is not > > > already > > > > > > > available on the config.xml. Since .cordova is basically a > marker > > > for > > > > > CLI > > > > > > > for the root of an app I think renaming config.xml should > provide > > > the > > > > > > same > > > > > > > functionality. > > > > > > > -- > > > > > > > Gorkem > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
