mbox

[PULL,0/3] Travis updates

Message ID 1459791945-4985-1-git-send-email-alex.bennee@linaro.org
State New
Headers show

Pull-request

https://github.com/stsquad/qemu.git tags/travis-pull-04042016

Message

Alex Bennée April 4, 2016, 5:45 p.m. UTC
The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:

  net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)

are available in the git repository at:

  https://github.com/stsquad/qemu.git tags/travis-pull-04042016

for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:

  .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)

----------------------------------------------------------------
This pull request includes:
  - further collapse of the build matrix
  - enabling MacOSX in the build
  - make -j3 change

Other pending updates are deferred for later in the cycle.

----------------------------------------------------------------
Alex Bennée (3):
      .travis.yml: collapse the test matrix
      .travis.yml: enable OSX builds
      .travis.yml: make -j3

 .travis.yml | 82 ++++++++++++++++++-------------------------------------------
 1 file changed, 24 insertions(+), 58 deletions(-)

Comments

Peter Maydell April 5, 2016, 8:06 a.m. UTC | #1
On 4 April 2016 at 18:45, Alex Bennée <alex.bennee@linaro.org> wrote:
> The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:
>
>   net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)
>
> are available in the git repository at:
>
>   https://github.com/stsquad/qemu.git tags/travis-pull-04042016
>
> for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:
>
>   .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)
>
> ----------------------------------------------------------------
> This pull request includes:
>   - further collapse of the build matrix
>   - enabling MacOSX in the build
>   - make -j3 change
>
> Other pending updates are deferred for later in the cycle.

This looks good, but you've left the '---' and v2 changes notes
in the commit message for patch 2 by mistake. Can I get you to
delete those and resend? (You can just send the new cover letter,
no need to resend the patches themselves.)

thanks
-- PMM
Alex Bennée April 5, 2016, 8:45 a.m. UTC | #2
Peter Maydell <peter.maydell@linaro.org> writes:

> On 4 April 2016 at 18:45, Alex Bennée <alex.bennee@linaro.org> wrote:
>> The following changes since commit e31f0451876aa7ada74d60304aa241506a383797:
>>
>>   net: fix missing include of qapi/error.h in netmap.c (2016-04-04 15:01:14 +0100)
>>
>> are available in the git repository at:
>>
>>   https://github.com/stsquad/qemu.git tags/travis-pull-04042016
>>
>> for you to fetch changes up to 5bdc914c1008292f3fbb5ad05026aaa5d3184435:
>>
>>   .travis.yml: make -j3 (2016-04-04 16:22:37 +0100)
>>
>> ----------------------------------------------------------------
>> This pull request includes:
>>   - further collapse of the build matrix
>>   - enabling MacOSX in the build
>>   - make -j3 change
>>
>> Other pending updates are deferred for later in the cycle.
>
> This looks good, but you've left the '---' and v2 changes notes
> in the commit message for patch 2 by mistake. Can I get you to
> delete those and resend? (You can just send the new cover letter,
> no need to resend the patches themselves.)

Doh, I'll clean that up.

I guess I need to script up my "prepare pull" step to remove those in
future.

>
> thanks
> -- PMM


--
Alex Bennée