Re: [Xen-devel] [OSSTEST PATCH 15/26] cr*: Support xen-unstable-smoke

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [OSSTEST PATCH 15/26] cr*: Support 
xen-unstable-smoke"):
> On Mon, 2015-09-21 at 11:21 +0100, Ian Jackson wrote:
> >     if [ "x$branch" != "xxen-unstable" ]; then
> >             export REVISION_LINUX_OLD=disable
> >     fi
> 
> That's the one which didn't need adjusting I think.

Yes.

> > I'm not sure what the qemu revisions thing is you're thinking of but I
> > think all the places where the source code matches `xen-unstable[^-]',
> > the actual effect is to treat `xen-unstable-smoke' the same way as
> > `xen-unstable'.
> 
> "qemu revisions thing" is:
> 
> @@ -340,7 +340,7 @@ case "$NEW_REVISION/$OLD_REVISION" in
>                          "$treeurl#$OLD_REVISION-$NEW_REVISION" \
>  
>                  case "$realtree" in
> -                xen-4*|xen-unstable)
> +                xen-4*|xen-unstable*)
>                      oldqemu=`./ap-qemu-revision $realtree $OLD_REVISION`
>                      newqemu=`./ap-qemu-revision $realtree $NEW_REVISION`
>                      if [ "$oldqemu" ] && [ "$newqemu" ]; then

Ah, OK.  Yes.  We agree.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel