Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (athena.apache.org: domain of gorkem.ercan@gmail.com
 designates 209.85.223.174 as permitted sender)
From: "Gorkem Ercan" <gorkem.ercan@gmail.com>
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Subject: Re: platforms/plugins save and restore from config.xml
Date: Fri, 09 Jan 2015 14:55:57 -0500
Message-ID: <1B0F504C-6C20-4673-984A-6A1A31BBD55C@gmail.com>
In-Reply-To: 
 <85A3E123BABF314D9D3656D0B418125643F056B6@FMSMSX103.amr.corp.intel.com>
References: <20150109044831.32391250.99529.4050@blackberry.com>
 <BY2PR0301MB05971490A7F5FF311DA853AFCD440@BY2PR0301MB0597.namprd03.prod.outlook.com>
 <BY2PR0301MB0597D1B28DA3F39802675A7ECD440@BY2PR0301MB0597.namprd03.prod.outlook.com>
 <CABiQX1UvC-k9BoqNd7=YonL_bNh2t0vt8NhxVZyENcX-Jbc-MQ@mail.gmail.com>
 <C2BFD5F4-C4E0-477E-AD3C-250DE1E84A9E@gmail.com>
 <CAEeF2TfEBS_ghDN1hSMxNTQU25wuy9OYzH6CLXXyn14Bz-0Qsw@mail.gmail.com>
 <85A3E123BABF314D9D3656D0B418125643F056B6@FMSMSX103.amr.corp.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit


On 9 Jan 2015, at 14:13, Treggiari, Leo wrote:

> I had asked some questions about save and restore a while back, but 
> have not been following the progress in detail - so ignore me if you 
> feel it is appropriate.
>
> One of my biggest questions was why would these commands be an option?

Ironically, the very first implementation did not introduce commands but 
rather hooked itself for plugin/platform add/rm and prepare, sort of 
like what [1] introduces with config.json preferences. However during 
the review it was suggested that these needs to be commands at least for 
a while.

[1] https://github.com/apache/cordova-lib/pull/143

> What I'm looking for, as soon as possible, is that Cordova 'project' 
> metadata is stored logically and consistently so that the CLI and 
> multiple IDEs could simultaneously work on the same project and know 
> about what each other have done wrt. adding/removing 
> plugins/platforms/etc.
>

That is one of my motivations, right now too much depends on plugins and 
platforms folders being present.

> Does removing experimental advance that goal or does it, as Michal 
> says, put obstacles in the path of getting there by requiring long 
> term support of an incomplete and possibly confusing (more options?) 
> solution?
>

Removing experimental matters only if we want to keep save and restore 
as separate commands. Otherwise, it is a matter of agreeing on a goal.

> Leo
>
> -----Original Message-----
> From: mmocny@google.com [mailto:mmocny@google.com] On Behalf Of Michal 
> Mocny
> Sent: Friday, January 09, 2015 10:35 AM
> To: dev
> Subject: Re: platforms/plugins save and restore from config.xml
>
> -1 on removing experimental.
>
> I love the concept behind this feature, and I applaud Gorkem for 
> actually
> working on pushing it forward, but I'm still concerned the current 
> design
> is not perfect.  Just today we were discussing storing the list of 
> plugins
> into a package.json if plugins move to npm.  The current 
> implementation
> still saves all installed plugins including dependencies and not just 
> what
> was explicitly added.
>
> If we add this outside experimental, and document & broadcast it, we 
> will
> have to support it going forward.  That will certainly influence 
> future cli
> designs.
>
> -Michal
>
> On Fri, Jan 9, 2015 at 12:08 PM, Gorkem Ercan <gorkem.ercan@gmail.com>
> wrote:
>
>>
>>
>> On 9 Jan 2015, at 10:41, Andrew Grieve wrote:
>>
>> Questions: Would ever not want to use --save? Why not just always 
>> update
>>> config.xml with what plugins you have?
>>>
>>> Eclipse Thym always updates the plugin & platform information to
>> config.xml, and no one complained about it so far.
>>
>> Likewise, would you ever not want to have --shrinkwrap? I think you'd
>>> always want the plugin/platform version listed in there.
>>>
>>>
>> I do not set the shrinkwrap for plugins usually, because without
>> shrinkwrap, the latest version is restored. I usually prefer the 
>> latest
>> with the stable plugins, such as the core plugins.
>> As a reference, Eclipse Thym does not shrinkwrap by default but has a
>> preference you can turn on.
>>
>> With platforms shrinkwrap as default makes sense.
>>
>> On Fri, Jan 9, 2015 at 1:46 AM, Mefire O. <ommenjik@microsoft.com> 
>> wrote:
>>>
>>> Also, I have Pull Requests that implements the --save flag as 
>>> mentioned
>>>> earlier :
>>>>
>>>> - https://github.com/apache/cordova-cli/pull/203
>>>> - https://github.com/apache/cordova-lib/pull/144
>>>>
>>>>
>>>> Thanks,
>>>> Mefire
>>>>
>>>> -----Original Message-----
>>>> From: Mefire O. [mailto:ommenjik@microsoft.com]
>>>> Sent: Thursday, January 8, 2015 10:27 PM
>>>> To: Cordova Dev
>>>> Subject: RE: platforms/plugins save and restore from config.xml
>>>>
>>>> +1 on removing the --experimental flag after fixing the 'variables 
>>>> not
>>>> being saved' bug.
>>>>
>>>> Thanks,
>>>> Mefire
>>>>
>>>> -----Original Message-----
>>>> From: Josh Soref [mailto:jsoref@blackberry.com]
>>>> Sent: Thursday, January 8, 2015 8:49 PM
>>>> To: Cordova Dev
>>>> Subject: Re: platforms/plugins save and restore from config.xml
>>>>
>>>> Until adding plugins saves the variables provided, we really 
>>>> shouldn't /
>>>> can't make this non experimental.
>>>>
>>>> Sent from my BlackBerry 10 smartphone.
>>>> ‎
>>>>
>>>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
>> For additional commands, e-mail: dev-help@cordova.apache.org
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> For additional commands, e-mail: dev-help@cordova.apache.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org