Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-12 Thread Jordan Justen
name than my ideas. Thanks Liming. -Jordan > > -Original Message- > > From: Andrew Fish [mailto:af...@apple.com] > > Sent: Saturday, November 8, 2014 12:03 PM > > To: Justen, Jordan L > > Cc: edk2-devel@lists.sourceforge.net; Fan, Jeff > > Subject: Re: [edk2] [PAT

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-12 Thread Andrew Fish
2014 12:03 PM > To: Justen, Jordan L > Cc: edk2-devel@lists.sourceforge.net; Fan, Jeff > Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for > IA32/X64 > > >> On Nov 7, 2014, at 7:43 PM, Jordan Justen wrote: >> >> On 2014-11-07 15:53:55,

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-12 Thread Gao, Liming
= S nasm Thanks Liming -Original Message- From: Andrew Fish [mailto:af...@apple.com] Sent: Saturday, November 8, 2014 12:03 PM To: Justen, Jordan L Cc: edk2-devel@lists.sourceforge.net; Fan, Jeff Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64 >

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-08 Thread Andrew Fish
> On Nov 8, 2014, at 2:51 PM, Jordan Justen wrote: > >> >> I’m not saying this is the right solution, but we should ask the >> question should we tie the override to the build_rule.template file >> sections vs. trying to override the contents of these actions. That >> is just a good thing for a

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-08 Thread Jordan Justen
On 2014-11-07 20:03:09, Andrew Fish wrote: > > On Nov 7, 2014, at 7:43 PM, Jordan Justen wrote: > > On 2014-11-07 15:53:55, Andrew Fish wrote: > >> I was having lunch today with Mike Kinney, and Vincent Zimmer > >> (unfortunately no Tanqueray involved) and I think I’ve distilled the > >> issue dow

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-07 Thread Andrew Fish
> On Nov 7, 2014, at 7:43 PM, Jordan Justen wrote: > > On 2014-11-07 15:53:55, Andrew Fish wrote: >> I was having lunch today with Mike Kinney, and Vincent Zimmer >> (unfortunately no Tanqueray involved) and I think I’ve distilled the >> issue down to a new feature that we need from the build sy

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-07 Thread Jordan Justen
On 2014-11-07 15:53:55, Andrew Fish wrote: > I was having lunch today with Mike Kinney, and Vincent Zimmer > (unfortunately no Tanqueray involved) and I think I’ve distilled the > issue down to a new feature that we need from the build system. I > want to be able to have an Xcode (or VC++, or Inte

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-07 Thread Andrew Fish
> On Nov 7, 2014, at 2:53 PM, Bill Paul wrote: > > Of all the gin joints in all the towns in all the world, Andrew Fish had to FYI Tanqueray and Tonic is the official drink of EFI. At least that is what Mark, Ken, and I decided when we were traveling around to all the Itanium(tm) Processor p

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-07 Thread Bill Paul
Of all the gin joints in all the towns in all the world, Andrew Fish had to walk into mine at 11:44:50 on Friday 07 November 2014 and say: > > On Nov 7, 2014, at 10:32 AM, Kinney, Michael D > > wrote: > > > > Andrew, > > > > Adding XCODE as a build rule family is possible. Your observation th

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Kinney, Michael D
@lists.sourceforge.net; Gao, Liming Cc: Fan, Jeff Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64 On 2014-11-06 11:18:16, Zimmer, Vincent wrote: > I agree. I would keep the original .asm and .s, along w/ new .nasm, at > least as part of a transition period of engagement

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Jordan Justen
, Jeff > Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for > IA32/X64 > > Hi > I have concern on removing original .asm and .S file, without a full > validation for .nasm file. > > I am fine on adding .nasm. But can we still keep original .asm

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Laszlo Ersek
usten, Jordan L; edk2-devel@lists.sourceforge.net > Cc: Fan, Jeff > Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for > IA32/X64 > > Hi > I have concern on removing original .asm and .S file, without a full > validation for .nasm file. >

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Zimmer, Vincent
, Jordan L; edk2-devel@lists.sourceforge.net Cc: Fan, Jeff Subject: Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64 Hi I have concern on removing original .asm and .S file, without a full validation for .nasm file. I am fine on adding .nasm. But can we still keep original

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Andrew Fish
> On Nov 6, 2014, at 1:42 AM, Andrew Fish wrote: > >> >> On Nov 6, 2014, at 12:26 AM, Jordan Justen > > wrote: >> >> On 2014-11-05 20:29:46, Andrew Fish wrote: >>> >>>On Nov 5, 2014, at 7:51 PM, Jordan Justen >> > >>>

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-06 Thread Sergey Isakov
On 06.11.2014, at 11:26, Jordan Justen wrote: > Is NASM just plain incompatible with XCODE? I see no problems. It is compatible with key *_XCODE5_X64_NASM_FLAGS = -f macho64--

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Jordan Justen
On 2014-11-05 19:19:26, Andrew Fish wrote: > > > On Nov 5, 2014, at 6:47 PM, Gao, Liming wrote: > > > > Jordan: > > This patch updates module INF with .nasm only, and remove the original > > .asm and .S. Right? > > > > After apply this patch, all developers are required to install nasm > >

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Jordan Justen
On 2014-11-05 19:23:24, Yao, Jiewen wrote: > Hi > I have concern on removing original .asm and .S file, without a full > validation for .nasm file. > > I am fine on adding .nasm. But can we still keep original .asm and > .S file? We should do a 'full validation' at this stage *before* making the

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Jordan Justen
On 2014-11-05 18:47:53, Gao, Liming wrote: > Jordan: > This patch updates module INF with .nasm only, and remove the original .asm > and .S. Right? > > After apply this patch, all developers are required to install nasm > compiler. Yes. -Jordan > -Original Message- > From: Justen

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Yao, Jiewen
Hi I have concern on removing original .asm and .S file, without a full validation for .nasm file. I am fine on adding .nasm. But can we still keep original .asm and .S file? Thank you Yao Jiewen -Original Message- From: Gao, Liming Sent: Thursday, November 06, 2014 10:48 AM To: Juste

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Andrew Fish
> On Nov 5, 2014, at 6:47 PM, Gao, Liming wrote: > > Jordan: > This patch updates module INF with .nasm only, and remove the original .asm > and .S. Right? > > After apply this patch, all developers are required to install nasm > compiler. > We require the option of NOT using nasm with

Re: [edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Gao, Liming
Jordan: This patch updates module INF with .nasm only, and remove the original .asm and .S. Right? After apply this patch, all developers are required to install nasm compiler. Thanks Liming -Original Message- From: Justen, Jordan L Sent: Thursday, November 6, 2014 10:01 AM To: ed

[edk2] [PATCH 000/345] Convert EDK II core packages to NASM for IA32/X64

2014-11-05 Thread Jordan Justen
Nope. I don't plan to actually send out these 345 patches. :) But, these patches are available in git: git://github.com/jljusten/edk2 nasm-edk2-core or https://github.com/jljusten/edk2.git nasm-edk2-core Or, view the branch in a web browser: https://github.com/jljusten/edk2/tree/nasm-edk2-core