Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Bob Feng
ws.C files,So I fixed this bug to >> Change the original logic to support.C files. >> >> >> Any question, please let me know. Thanks. >> >> Best Regards >> Fan Zhiju >> >> >> >> >>> -----Original Message----- >>> From: Le

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Laszlo Ersek
;> Any question, please let me know. Thanks. >> >> Best Regards >> Fan Zhiju >> >> >> >> >>> -Original Message----- >>> From: Leif Lindholm [mailto:leif.lindh...@linaro.org] >>> Sent: Wednesday, May 8, 2019 5:09 PM >>&

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Leif Lindholm
nge the > >> original logic to support.C files. > >> > >> > >> Any question, please let me know. Thanks. > >> > >> Best Regards > >> Fan Zhiju > >> > >> > >> > >> > >>

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Andrew Fish via Groups.Io
gt; >> >> Any question, please let me know. Thanks. >> >> Best Regards >> Fan Zhiju >> >> >> >> >>> -Original Message- >>> From: Leif Lindholm [mailto:leif.lindh...@linaro.org] >>> Sent: Wednesday, May 8, 201

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Leif Lindholm
o: Andrew Fish > > Cc: devel@edk2.groups.io; Fan, ZhijuX ; Gao, Liming > > ; Feng, Bob C > > Subject: Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C > > files with .C suffixes > > > > On Tue, May 07, 2019 at 07:01:26PM -0700, Andrew Fish wrote: >

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Fan, ZhijuX
y 8, 2019 5:09 PM > To: Andrew Fish > Cc: devel@edk2.groups.io; Fan, ZhijuX ; Gao, Liming > ; Feng, Bob C > Subject: Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C > files with .C suffixes > > On Tue, May 07, 2019 at 07:01:26PM -0700, Andrew Fish wrote: >

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-08 Thread Leif Lindholm
> >> Any question, please let me know. Thanks. > >> > >> Best Regards > >> Fan Zhiju > >> > >> -Original Message- > >> From: af...@apple.com [mailto:af...@apple.com] > >> Sent: Tuesday, May 7, 2019 10:31 AM > &g

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-07 Thread Andrew Fish via Groups.Io
; >> Best Regards >> Fan Zhiju >> >> -Original Message- >> From: af...@apple.com [mailto:af...@apple.com] >> Sent: Tuesday, May 7, 2019 10:31 AM >> To: devel@edk2.groups.io; Fan, ZhijuX >> Cc: Gao, Liming ; Feng, Bob C >> Subject: Re:

Re: FW: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-07 Thread Fan, ZhijuX
gt; > > > Any question, please let me know. Thanks. > > > > Best Regards > > Fan Zhiju > > > > -Original Message- > > From: af...@apple.com [mailto:af...@apple.com] > > Sent: Tuesday, May 7, 2019 10:31 AM > > To: devel@edk2.groups.io; Fan,

Re: FW: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-07 Thread Leif Lindholm
2.groups.io; Fan, ZhijuX > Cc: Gao, Liming ; Feng, Bob C > Subject: Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C > files with .C suffixes > > This brings up a question? Do we tests on a file system that is case > sensitive? Is this just lack of a test case?

FW: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-06 Thread Fan, ZhijuX
know. Thanks. Best Regards Fan Zhiju -Original Message- From: af...@apple.com [mailto:af...@apple.com] Sent: Tuesday, May 7, 2019 10:31 AM To: devel@edk2.groups.io; Fan, ZhijuX Cc: Gao, Liming ; Feng, Bob C Subject: Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files wi

Re: [edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-06 Thread Andrew Fish via Groups.Io
This brings up a question? Do we tests on a file system that is case sensitive? Is this just lack of a test case? Thanks, Andrew Fish > On May 6, 2019, at 7:22 PM, Fan, ZhijuX wrote: > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1773 > > Build break if C file suffixes of named .C in

[edk2-devel] [PATCH V2] BaseTools:improve code to support C files with .C suffixes

2019-05-06 Thread Fan, ZhijuX
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1773 Build break if C file suffixes of named .C instead of .c Code not recognize filenames with .C suffixes. This patch adds code to Support both .c file and .C file Cc: Bob Feng Cc: Liming Gao Signed-off-by: Zhiju.Fan --- BaseTools/Source/P