All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Alexander Graf <agraf@suse.de>
Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 3/4] .travis.yml: pre-seed sub-modules for speed
Date: Tue, 23 Sep 2014 15:50:46 +0100	[thread overview]
Message-ID: <8738bi2xx5.fsf@linaro.org> (raw)
In-Reply-To: <54214D56.1090809@suse.de>


Alexander Graf writes:

> On 15.09.14 18:48, Alex Bennée wrote:
>> A significant portion of the build time is spent initialising all the
>> sub-modules we use in the source tree. Often this is almost as long as
>> the build itself. By pre-seeding the .git/modules tree this will
>> hopefully improve things.
>> 
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> 
>> diff --git a/.travis.yml b/.travis.yml
>> index f113339..8df02a4 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -37,7 +37,12 @@ env:
>>      - TARGETS=unicore32-softmmu,unicore32-linux-user
>>      # Group remaining softmmu only targets into one build
>>      - TARGETS=lm32-softmmu,moxie-softmmu,tricore-softmmu,xtensa-softmmu,xtensaeb-softmmu
>> +git:
>> +  # we want to do this ourselves
>> +  submodules: false
>>  before_install:
>> +  - wget http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz
>> +  - tar -xvf qemu-submodule-git-seed.tar.xz
>
> wget -O - | tar?

Sure I can do this.

>>    - git submodule update --init --recursive
>
> Doesn't this overwrite the code you just downloaded?

It didn't in my testing. However I need to do the "--init" to ensure all
the modules are correctly checked out. When manually deleting one of the
packs in my testing git did the right thing and fetched the missing pack
data. Once the submodules move forward I'll double check the incremental
update works as well.

>
>
> Alex
>
>>    - sudo apt-get update -qq
>>    - sudo apt-get install -qq ${CORE_PKGS} ${NET_PKGS} ${GUI_PKGS} ${EXTRA_PKGS}
>> 

-- 
Alex Bennée

  reply	other threads:[~2014-09-23 14:51 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-15 16:48 [Qemu-devel] [PATCH 0/4] A number of Travis CI tweaks Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 1/4] .travis.yml: add more linux-user to the build matrix Alex Bennée
2014-09-17 18:36   ` Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 2/4] .travis.yml: make the make slightly more parallel Alex Bennée
2014-09-15 16:48 ` [Qemu-devel] [PATCH 3/4] .travis.yml: pre-seed sub-modules for speed Alex Bennée
2014-09-23 10:37   ` Alexander Graf
2014-09-23 14:50     ` Alex Bennée [this message]
2014-09-23 15:37       ` Alexander Graf
2014-09-15 16:48 ` [Qemu-devel] [PATCH 4/4] .travis.yml: remove "make check" from main matrix Alex Bennée
2014-09-23 10:38 ` [Qemu-devel] [PATCH 0/4] A number of Travis CI tweaks Alexander Graf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8738bi2xx5.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=agraf@suse.de \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.