* [PATCH OSSTEST] ts-xen-build-prep: install nasm
@ 2015-01-31 22:51 Wei Liu
2015-02-02 10:23 ` Ian Campbell
0 siblings, 1 reply; 4+ messages in thread
From: Wei Liu @ 2015-01-31 22:51 UTC (permalink / raw)
To: xen-devel; +Cc: ian.jackson, Wei Liu, ian.campbell
OVMF requires nasm to build.
Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
---
ts-xen-build-prep | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ts-xen-build-prep b/ts-xen-build-prep
index a7d0d03..24a8b25 100755
--- a/ts-xen-build-prep
+++ b/ts-xen-build-prep
@@ -178,7 +178,7 @@ sub prep () {
autoconf automake libtool xsltproc
libxml2-utils libxml2-dev libnl-dev
libdevmapper-dev w3c-dtd-xhtml libxml-xpath-perl
- ccache));
+ ccache nasm));
target_cmd_root($ho, "chmod -R a+r /usr/share/git-core/templates");
# workaround for Debian #595728
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH OSSTEST] ts-xen-build-prep: install nasm
2015-01-31 22:51 [PATCH OSSTEST] ts-xen-build-prep: install nasm Wei Liu
@ 2015-02-02 10:23 ` Ian Campbell
2015-02-02 10:38 ` Wei Liu
0 siblings, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2015-02-02 10:23 UTC (permalink / raw)
To: Wei Liu; +Cc: ian.jackson, xen-devel
On Sat, 2015-01-31 at 22:51 +0000, Wei Liu wrote:
> OVMF requires nasm to build.
I suppose this is a new requirement added by the mainline branch we just
added?
>
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> ---
> ts-xen-build-prep | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ts-xen-build-prep b/ts-xen-build-prep
> index a7d0d03..24a8b25 100755
> --- a/ts-xen-build-prep
> +++ b/ts-xen-build-prep
> @@ -178,7 +178,7 @@ sub prep () {
> autoconf automake libtool xsltproc
> libxml2-utils libxml2-dev libnl-dev
> libdevmapper-dev w3c-dtd-xhtml libxml-xpath-perl
> - ccache));
> + ccache nasm));
>
> target_cmd_root($ho, "chmod -R a+r /usr/share/git-core/templates");
> # workaround for Debian #595728
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH OSSTEST] ts-xen-build-prep: install nasm
2015-02-02 10:23 ` Ian Campbell
@ 2015-02-02 10:38 ` Wei Liu
2015-02-02 14:18 ` Ian Campbell
0 siblings, 1 reply; 4+ messages in thread
From: Wei Liu @ 2015-02-02 10:38 UTC (permalink / raw)
To: Ian Campbell; +Cc: ian.jackson, xen-devel
Ian Campbell <Ian.Campbell@citrix.com> writes:
> On Sat, 2015-01-31 at 22:51 +0000, Wei Liu wrote:
>> OVMF requires nasm to build.
>
> I suppose this is a new requirement added by the mainline branch we just
> added?
>>
Yes. The log in the latest two failed runs suggests so.
I also tested building latest OVMF HEAD it does require nasm to build.
Wei.
>> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
>
>> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
>> ---
>> ts-xen-build-prep | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/ts-xen-build-prep b/ts-xen-build-prep
>> index a7d0d03..24a8b25 100755
>> --- a/ts-xen-build-prep
>> +++ b/ts-xen-build-prep
>> @@ -178,7 +178,7 @@ sub prep () {
>> autoconf automake libtool xsltproc
>> libxml2-utils libxml2-dev libnl-dev
>> libdevmapper-dev w3c-dtd-xhtml libxml-xpath-perl
>> - ccache));
>> + ccache nasm));
>>
>> target_cmd_root($ho, "chmod -R a+r /usr/share/git-core/templates");
>> # workaround for Debian #595728
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH OSSTEST] ts-xen-build-prep: install nasm
2015-02-02 10:38 ` Wei Liu
@ 2015-02-02 14:18 ` Ian Campbell
0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2015-02-02 14:18 UTC (permalink / raw)
To: Wei Liu; +Cc: ian.jackson, xen-devel
On Mon, 2015-02-02 at 10:38 +0000, Wei Liu wrote:
> Ian Campbell <Ian.Campbell@citrix.com> writes:
>
> > On Sat, 2015-01-31 at 22:51 +0000, Wei Liu wrote:
> >> OVMF requires nasm to build.
> >
> > I suppose this is a new requirement added by the mainline branch we just
> > added?
> >>
>
> Yes. The log in the latest two failed runs suggests so.
>
> I also tested building latest OVMF HEAD it does require nasm to build.
Thanks, applied to pretest after discussion with Ian J.
Ian.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-02-02 14:18 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-31 22:51 [PATCH OSSTEST] ts-xen-build-prep: install nasm Wei Liu
2015-02-02 10:23 ` Ian Campbell
2015-02-02 10:38 ` Wei Liu
2015-02-02 14:18 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.