xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Attilio Rao <attilio.rao@citrix.com>
To: Keir Fraser <keir.xen@gmail.com>
Cc: "g@phlegethon.org" <g@phlegethon.org>,
	"Tim (Xen.org)" <tim@xen.org>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: Build failure in OVMF
Date: Thu, 22 Mar 2012 15:23:15 +0000	[thread overview]
Message-ID: <4F6B43E3.4090801@citrix.com> (raw)
In-Reply-To: <CB90E08F.2F1DF%keir.xen@gmail.com>

On 22/03/12 13:58, Keir Fraser wrote:
> On 22/03/2012 13:15, "Tim Deegan"<tim@xen.org>  wrote:
>
>    
>> At 13:07 +0000 on 22 Mar (1332421679), Attilio Rao wrote:
>>      
>>> Actually CONFIG_OVMF should be disabled by default, can you please check
>>> how is in your case?
>>>        
>> whitby:xen-unstable.hg$ grep CONFIG_OVMF Config.mk
>> CONFIG_OVMF ?= $(CONFIG_Linux)
>>
>>    changeset:   25077:6bf50858c3c5
>>    user:        Attilio Rao<attilio.rao@citrix.com>
>>    date:        Thu Mar 22 10:26:03 2012 +0000
>>    summary:     Fetch the OVMF repository from specific git mirror and enable
>> it
>>
>>      
>>> Also, did you apply the OVMF v4 patch for enabling the CONFIG_OVMF on?
>>>        
>> Nope, AFAICT I have no local patches.
>>      
> I applied the OVMF-enable patch this morning, hence Tim's problem. I might
> have applied v3 rather than v4 though? Please check.
>
>    

I actually think you have applied the right patch, there is just a 
situation in OVMF binary building that we need to cope with, my bad for 
not having better checked.

For the moment, please disable OVMF by default building, I'll send you 
more patches and when the upstream TIANOCORE will have fixes in I will 
send an explicit patch to enable it.

Thanks,
Attilio

  parent reply	other threads:[~2012-03-22 15:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-22 12:59 Build failure in OVMF Tim Deegan
2012-03-22 13:07 ` Attilio Rao
2012-03-22 13:15   ` Tim Deegan
2012-03-22 13:58     ` Keir Fraser
2012-03-22 14:08       ` Ian Campbell
2012-03-22 14:48         ` Attilio Rao
2012-03-22 15:23       ` Attilio Rao [this message]
2012-03-22 14:55 ` Tim Deegan
2012-03-22 15:04   ` Ian Campbell
2012-03-22 15:16     ` Tim Deegan
2012-03-26  9:37       ` Ian Campbell
2012-03-22 15:20   ` Attilio Rao
2012-04-02 19:07 ` Jordan Justen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F6B43E3.4090801@citrix.com \
    --to=attilio.rao@citrix.com \
    --cc=g@phlegethon.org \
    --cc=keir.xen@gmail.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).