+1 and using real git urls (should?) lets you install from branches/tags+other git repos and not just the mystical magical one that lives behind the flag.
@purplecabbage risingj.com On Wed, Jun 3, 2015 at 12:05 PM, Steven Gill <[email protected]> wrote: > Sounds like a good candidate for deprecation to me. I used --usegit in the > old nightly builds I had running. But the custom git urls would work just > as well. > > -Steve > > On Wed, Jun 3, 2015 at 11:44 AM, Mefire O. <[email protected]> wrote: > > > Hi list, > > I think it's high time we deprecate the -usegit flag, whose intent was to > > provide apache git repos as an alternative source to install platforms > (the > > default being npm). > > > > A little bit of background : > > In the past, there were two main sources from which to install a > platform. > > You could fetch the platform to be installed from npm (the default). > > Or you could specify that the platform to be installed be fetched from > the > > Apache git repo using -usegit : `cordova platform add android -usegit`. > > > > There are certain shortcomings to the -useflag : > > > > - It only works with Apache git repos. It doesn't support custom > > git repos. > > > > e.g: It passes if the following is found in platformsConfig.json : > > `cordova platform add > > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git` > > > > e.g: the following would fail : `cordova platform add > > https://github.com/apache/cordova-android.git`< > > https://github.com/apache/cordova-android.git%60> > > > > - The code that implements it is complex and prone to bugs. e.g: > > https://github.com/apache/cordova-lib/pull/232 > > > > - The git repo we want to fetch the platform from has to be > > manually added to the file platformsConfig.json. > > > > > > Suggested replacement: > > We've been supporting custom git-urls for a while now : > > > > - > > > https://github.com/apache/cordova-lib/commit/811f6f72d74dfd2ca054553ae248f0831235f912 > > > > - > > > https://github.com/apache/cordova-lib/commit/4d2059452fa36dd510d9a949e150de840d336e04 > > > > Using this feature, we can add platforms from any git repo using the > > following : `cordova platform add > > https://github.com/apache/cordova-android.git`< > > https://github.com/apache/cordova-android.git%60> > > It also supports specifying a custom branch : `cordova platform add > > https://github.com/apache/cordova-android.git#my_branch` > > > > So, I think this second feature can handle all the scenarios handled by > > -usegit and much more ... > > > > What I propose is phasing out the -usegit flag, over the next 6 months. > > > > Please, let me know your thoughts. > > > > Thanks, > > Mefire > > > > >
