Re: [meta-raspberrypi][PATCH] firmware.inc: fetch from SVN instead of Git

["Petter Mabäcker" <petter@technux.se>]

On 07/04/2015 07:29 AM, Nikolay Dimitrov wrote:
> Hi Jonathan,
>
> On 07/04/2015 06:48 AM, Jonathan Liu wrote:
>> Reduces the download time significantly.
>>
>> Signed-off-by: Jonathan Liu <net147@gmail.com>
>> ---
>>   recipes-bsp/common/firmware.inc | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/recipes-bsp/common/firmware.inc 
>> b/recipes-bsp/common/firmware.inc
>> index ad3176a..100df3a 100644
>> --- a/recipes-bsp/common/firmware.inc
>> +++ b/recipes-bsp/common/firmware.inc
>> @@ -1,7 +1,7 @@
>> -RPIFW_SRCREV ?= "e42a747e8d5c4a2fb3e837d0924c7cc39999936a"
>> +RPIFW_SRCREV ?= "461"
>>   RPIFW_DATE ?= "20150206"
>> -RPIFW_SRC_URI ?= 
>> "git://github.com/raspberrypi/firmware.git;protocol=git;branch=master"
>> -RPIFW_S ?= "${WORKDIR}/git"
>> +RPIFW_SRC_URI ?= 
>> "svn://github.com/raspberrypi/firmware;protocol=https;module=trunk"
>> +RPIFW_S ?= "${WORKDIR}/trunk"
>>
>>   SRC_URI = "${RPIFW_SRC_URI}"
>>   SRCREV = "${RPIFW_SRCREV}"
>>
>
> Just tested the download times for this repo:
>
> git (deep clone): 24m
> svn: 3m8s
> git (shallow clone): 40s
>
> Isn't it possible to use "--depth 1" for git clone? I would rather
> prefer seeing this instead of just giving up and going back to svn...
>
> Regards,
> Nikolay
Jonathan, I understand your problem, I have pointed out 
"https://github.com/raspberrypi/firmware/archive/${RPIFW_SRCREV}.zip" 
ontop of HEAD when building in order to get rid of this problem. But I 
agree with Nikolay that going back to svn feels like a step in the wrong 
direction and instead we should push for a changle like the one 
discussed in the thread: '[yocto] [meta-raspberrypi][PATCH] 
firmware.inc: Fetch a zip instead of cloning a git repo'

or by using "--depth=1" (which isn't supported in the bitbake git 
fetcher for the moment..).

I will try to push for that one of the solutions (--depth=1 or .zip) is 
merged soon so we once for all can get rid of this annoying problem.

BR,
Petter