[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing |
Date: |
Mon, 6 Jun 2016 08:54:23 -0300 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Mon, Jun 06, 2016 at 08:47:24AM -0300, Eduardo Habkost wrote:
> On Mon, Jun 06, 2016 at 02:58:24PM +0800, Fam Zheng wrote:
> > On Fri, 06/03 15:26, Eduardo Habkost wrote:
> > > On Fri, Jun 03, 2016 at 09:23:28AM +0800, Fam Zheng wrote:
> > > > On Thu, 06/02 16:13, Eduardo Habkost wrote:
> > > > > On Wed, Jun 01, 2016 at 05:45:12PM +0800, Fam Zheng wrote:
> > > > > [...]
> > > > > > +
> > > > > > +CUR_TIME := $(shell date +%Y-%m-%d-%H.%M.%S.$$$$)
> > > > > > +# Makes the definition constant after the first expansion
> > > > > > +DOCKER_SRC_COPY = $(eval DOCKER_SRC_COPY :=
> > > > > > docker-src.$(CUR_TIME))$(DOCKER_SRC_COPY)
> > > > > > +
> > > > > > +$(DOCKER_SRC_COPY):
> > > > >
> > > > > This seems to be confusing make in travis-ci, and I don't
> > > > > understand why:
> > > > > https://travis-ci.org/ehabkost/qemu/jobs/134820251#L1266
> > > >
> > > > Indeed, but qemu.git master doesn't have that. Wondering why.
> > >
> > > I have pushed master to a temporary branch on my tree, and the
> > > issue persists:
> > >
> > > https://travis-ci.org/ehabkost/qemu/builds/135090745
> >
> > I pushed the same commit 6b3532b20b787cbd697a68b383232f5c3b39bd1e to my
> > github
> > repo and the triggered travis build doesn't error at the same location. I
> > have
> > no idea what is happening there. Wild guesses: do you have any non-default
> > travis per-repo or per-branch settings? Have you tried deleting all caches
> > at:
> >
> > https://docs.travis-ci.com/user/caching/
> >
> > ?
>
> No non-default settings, as far as I can see. I have:
> * Build only if .travis.yml is present: OFF
> * Limit concurrent jobs: OFF
> * Build pushes: ON
> * Build pull requests: ON
> * Environment variables: none
>
> No .travis.yml changes on any of my branches.
>
> I have just deleted all caches and I will send a new push. I will
> let you know the results soon.
Same: https://travis-ci.org/ehabkost/qemu/jobs/135569978
I will make some changes to test if this really has anything to
do with the Makefile code, or if there's something wrong on
Travis side.
--
Eduardo
- [Qemu-devel] [PULL v2 00/16] docker queue, Fam Zheng, 2016/06/01
- [Qemu-devel] [PULL 03/16] Makefile: Always include rules.mak, Fam Zheng, 2016/06/01
- [Qemu-devel] [PULL 01/16] tests: Add utilities for docker testing, Fam Zheng, 2016/06/01
- [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Fam Zheng, 2016/06/01
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/02
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Fam Zheng, 2016/06/02
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/03
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/03
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Fam Zheng, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing,
Eduardo Habkost <=
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Paolo Bonzini, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Paolo Bonzini, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Eduardo Habkost, 2016/06/06
- Re: [Qemu-devel] [PULL 04/16] Makefile: Rules for docker testing, Paolo Bonzini, 2016/06/06
[Qemu-devel] [PULL 05/16] docker: Add images, Fam Zheng, 2016/06/01
[Qemu-devel] [PULL 02/16] rules.mak: Add "COMMA" constant, Fam Zheng, 2016/06/01
[Qemu-devel] [PULL 09/16] docker: Add full test, Fam Zheng, 2016/06/01
[Qemu-devel] [PULL 07/16] docker: Add common.rc, Fam Zheng, 2016/06/01