Re: [Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-06-17 Thread Stefano Stabellini
On Wed, 13 May 2015, Ian Jackson wrote: > Stefano Stabellini writes ("Re: [PATCH v5] OSSTEST: introduce a raisin build > test"): > > On Tue, 12 May 2015, Ian Jackson wrote: > > > I don't understand what the \\ are doing here. Perhaps you should use > > > '' like in ts-xen-build ? > > > > I need

Re: [Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-05-13 Thread Ian Jackson
Stefano Stabellini writes ("Re: [PATCH v5] OSSTEST: introduce a raisin build test"): > On Tue, 12 May 2015, Ian Jackson wrote: > > I don't understand what the \\ are doing here. Perhaps you should use > > '' like in ts-xen-build ? > > I need to retain the " in the output raisin doesn't cope wit

Re: [Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-05-13 Thread Ian Campbell
On Tue, 2015-05-12 at 19:04 +0100, Stefano Stabellini wrote: > On Tue, 12 May 2015, Ian Jackson wrote: > > Stefano Stabellini writes ("[PATCH v5] OSSTEST: introduce a raisin build > > test"): > > > Signed-off-by: Stefano Stabellini > > ... > > > +echo >>config XEN_URL=\\"$r{tree_xen}\\" > > >

Re: [Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-05-12 Thread Stefano Stabellini
On Tue, 12 May 2015, Ian Jackson wrote: > Stefano Stabellini writes ("[PATCH v5] OSSTEST: introduce a raisin build > test"): > > Signed-off-by: Stefano Stabellini > ... > > +echo >>config XEN_URL=\\"$r{tree_xen}\\" > > +echo >>config XEN_REVISION=\\"$r{revision_xen}\\" > > This is very r

Re: [Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-05-12 Thread Ian Jackson
Stefano Stabellini writes ("[PATCH v5] OSSTEST: introduce a raisin build test"): > Signed-off-by: Stefano Stabellini ... > +echo >>config XEN_URL=\\"$r{tree_xen}\\" > +echo >>config XEN_REVISION=\\"$r{revision_xen}\\" This is very repetitive. In ts-xen-build, the names of the variables a

[Xen-devel] [PATCH v5] OSSTEST: introduce a raisin build test

2015-05-12 Thread Stefano Stabellini
Signed-off-by: Stefano Stabellini --- Changes in v5: - remove setting DEFAULT_REVISION_RAISIN to master - disable raisin when branch is not xen* - do not overwrite raisin default variables unless both revision_* and tree_* are set of a given component Changes in v4: - introduce enable_raisin in