Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-31 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Justen, Jordan L Sent: Thursday, May 28, 2015 11:14 PM To: edk2-devel@lists.sourceforge.net Cc: Justen, Jordan L; Gao, Liming Subject: [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions For *.asm and *.s, there have been

Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-28 Thread Brian J. Johnson
servers run with a case sensitive file system. > Seconded. > Thanks, > > Andrew Fish > >> Cheers, >> Larry >> >> -Original Message- >> From: Jordan Justen [mailto:jordan.l.jus...@intel.com] >> Sent: Thursday, May 28, 2015 8:14 AM &

Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-28 Thread Jordan Justen
dk2-devel@lists.sourceforge.net > Subject: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm > extensions > > For *.asm and *.s, there have been cases of *.Asm and *.S files, but since > the nasm extensions are new, we don't need to support the upper case &

Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-28 Thread Andrew Fish
t; Cheers, > Larry > > -Original Message- > From: Jordan Justen [mailto:jordan.l.jus...@intel.com] > Sent: Thursday, May 28, 2015 8:14 AM > To: edk2-devel@lists.sourceforge.net > Subject: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm > ext

Re: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-28 Thread Hauch, Larry
Subject: [edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions For *.asm and *.s, there have been cases of *.Asm and *.S files, but since the nasm extensions are new, we don't need to support the upper case extensions. In other words, remove .Nasm and .NASM. Contrib

[edk2] [PATCH v2] BaseTools/Conf: Don't support upper case nasm extensions

2015-05-28 Thread Jordan Justen
For *.asm and *.s, there have been cases of *.Asm and *.S files, but since the nasm extensions are new, we don't need to support the upper case extensions. In other words, remove .Nasm and .NASM. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Liming Gao