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

Reply via email to