Thanks a lot!! On Thu, Jul 23, 2015 at 12:23 PM Jesse <purplecabb...@gmail.com> wrote:
> CB-9384 > https://github.com/apache/cordova-lib/pull/265 > > > > On Jul 23, 2015, at 10:10 AM, Victor Sosa <sosah.vic...@gmail.com> > wrote: > > > > Now I understand. Thanks. > > Jesse, do you know the JIRA in which you worked this problem? > > > >> On Wed, Jul 22, 2015 at 5:27 PM Jesse <purplecabb...@gmail.com> wrote: > >> > >> Yes, we absolutely need to be able to install plugins via github url and > >> support branches/tags/commit-sha as part of the url. Also, sub-folders > >> within github repos. > >> > >> The feature I removed was the 'git version checking bs' [1] > >> > >> Carlos, npm supports installing from github too[2]. I'm not sure why > you > >> recommend dropping that as it is great for trying different combos. > >> ie. platformA+pluginB@unpublished-version. > >> > >> > >> [1] > >> > >> > https://github.com/apache/cordova-lib/commit/9af173a65281e9ea87fc6da3ee0b9f10bb8fec1c > >> [2] https://docs.npmjs.com/cli/install > >> > >> > >> > >> > >> > >> My team is hiring! > >> @purplecabbage > >> risingj.com > >> > >>> On Wed, Jul 22, 2015 at 3:00 PM, Darryl Pogue <dar...@dpogue.ca> > wrote: > >>> > >>> Thanks for clarifying :) > >>> > >>>> On 22 July 2015 at 14:53, Steven Gill <stevengil...@gmail.com> wrote: > >>>> > >>>> To clarify, we aren't removing installing from git. It is totally > >>>> important. > >>>> > >>>> Jesse fixed a problem with not being able to install from git when you > >>>> point to specific commit. This was do the --depth flag being used. He > >>>> recently removed the use of --depth and this may fix the git 1 vs 2 > >>> issues > >>>> Victor is seeing. > >>>> > >>>> On Wed, Jul 22, 2015 at 2:30 PM, Darryl Pogue <dar...@dpogue.ca> > >> wrote: > >>>> > >>>>> Support for installing from git URLs for plugins is a pretty > >> important > >>>>> requirement for us as end users. > >>>>> > >>>>> On 22 July 2015 at 14:21, Carlos Santana <csantan...@gmail.com> > >> wrote: > >>>>> > >>>>>> +1 only npm id or local path should be supported > >>>>>> On Wed, Jul 22, 2015 at 5:16 PM Victor Sosa < > >> sosah.vic...@gmail.com> > >>>>>> wrote: > >>>>>> > >>>>>>> Wait a minute, Jesse? what you mean by "I removed this > >> 'feature'"? > >>>> What > >>>>>>> feature are you referring to? > >>>>>>> > >>>>>>> On Wed, Jul 22, 2015 at 3:50 PM Jesse <purplecabb...@gmail.com> > >>>> wrote: > >>>>>>> > >>>>>>>> No idea, but, I just removed this 'feature' as there were > >> errors > >>>> with > >>>>>>>> checking out a particular tag or commit. > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>>> On Jul 22, 2015, at 1:29 PM, Victor Sosa < > >>> sosah.vic...@gmail.com > >>>>> > >>>>>>> wrote: > >>>>>>>>> > >>>>>>>>> Hello community. > >>>>>>>>> > >>>>>>>>> I've been playing a little with Cordova on simple scenarios, > >>> and > >>>>>>> noticed > >>>>>>>>> that when using Git 2.x I get this error when installing a > >>> plugin > >>>>>> from > >>>>>>> a > >>>>>>>>> Git repo from GitHub > >>>>>>>>> https://github.com/apache/cordova-plugin-contacts.git > >>>>>>>>> > >>>>>>>>> The error is: > >>>>>>>>> *Error: git: Command failed with exit code 128 Error output:* > >>>>>>>>> *Cloning into '/tmp/git/1436905601736'...* > >>>>>>>>> *fatal: dumb http transport does not support --depth * > >>>>>>>>> > >>>>>>>>> However, when using 1.9.x I have no problems installing the > >>>> plugin. > >>>>>>>>> > >>>>>>>>> I've been looking at the docs but I can't find a single > >> mention > >>>> of > >>>>>>>>> supported Git version. Does anyone happen to know the answer > >> of > >>>>> this? > >>>>>>>>> > >>>>>>>>> BTW, using Cordova 5.1.1 > >>>> --------------------------------------------------------------------- > >>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > >>>>>>>> For additional commands, e-mail: dev-h...@cordova.apache.org > >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > >