[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [OSSTEST PATCH v2] support XSM/FLASK via Kconfig



On Tue, 2016-01-05 at 23:25 -0600, Doug Goldstein wrote:
> In antcipation of XSM and FLASK migrating to Kconfig add support for
> building them via Kconfig or the existing mechanism.
> 
> Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx>
> ---
> Still untested but visually looks correct.
> 
> Changes since v2:
> - Support Xen versions prior to Kconfig being integrated

Do we not need the gate on the first hunk as well?Â

A non-Kconfig aware would ignore the xen/.config but osstest will collect
it later on and that might confuse folks reading the logs.

Such a Xen also won't have a .gitignore entry for xen/.config, so various
git commands might show this tree as dirty. I don't think osstest would
rely on such though, so maybe this doesn't really matter. I don't think
xen/scripts/scmversion adds -dirty in this case either.

> ---
> Âts-xen-build | 5 +++++
> Â1 file changed, 5 insertions(+)
> 
> diff --git a/ts-xen-build b/ts-xen-build
> index 80b1faa..8dde146 100755
> --- a/ts-xen-build
> +++ b/ts-xen-build
> @@ -55,6 +55,8 @@ sub checkout () {
> Â     echo >>.config KERNELS=''
> ÂEND
> ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ(nonempty($r{enable_xsm}) ? <<END : '').
> +     echo >>xen/.config CONFIG_XSM='${build_xsm}'
> +     echo >>xen/.config CONFIG_FLASK='${build_xsm}'
> Â     echo >>.config XSM_ENABLE='${build_xsm}'
> ÂEND
> ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ(nonempty($r{tree_qemu}) ? <<END : '').
> @@ -126,6 +128,9 @@ END
> ÂEND
> Â#/;
> ÂÂÂÂÂbuildcmd_stamped_logged(9000, 'build', '',<<END,'');
> +ÂÂÂÂÂÂÂÂÂÂÂÂif test -f xen/Kconfig; then
> +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ$make_prefix make -C xen olddefconfig
> +ÂÂÂÂÂÂÂÂÂÂÂÂfi
> ÂÂÂÂÂÂÂÂÂÂÂÂÂ$make_prefix make $makeflags @ARGV
> ÂEND
> Â

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.