Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-20 Thread Gao, Liming
2:52 AM To: Gao, Liming Cc: Dong, Guo; edk2-devel@lists.sourceforge.net; Kinney, Michael D Subject: Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages On Mon, Aug 18, 2014 at 2:25 AM, Gao, Liming wrote: > Jordan: > > The commit message is just Mike description. Are y

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-18 Thread Jordan Justen
: 1. The messages all arrive at about the same time 2. In a threaded email view, the whole series in grouped -Jordan > -Original Message- > From: Jordan Justen [mailto:jljus...@gmail.com] > Sent: Monday, August 18, 2014 3:57 PM > To: Gao, Liming; Dong, Guo > Cc: edk2-devel@l

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-18 Thread Gao, Liming
--- From: Jordan Justen [mailto:jljus...@gmail.com] Sent: Monday, August 18, 2014 3:57 PM To: Gao, Liming; Dong, Guo Cc: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages What is the commit message for this patch? It looks like this was

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-18 Thread Jordan Justen
What is the commit message for this patch? It looks like this was missing the Contributed-under and Signed-off-by too. On Wed, Aug 13, 2014 at 7:13 PM, Gao, Liming wrote: > Hi, all > > Could you help review this patch? It includes the following changes 1-6 > for MdePkg. Why are you including

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-15 Thread Kinney, Michael D
<mailto:edk2-devel@lists.sourceforge.net> Subject: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages Hi, all Could you help review this patch? It includes the following changes 1-6 for MdePkg. The patch is a little big. For new added UNI file, I zip them together. Th

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-14 Thread Kinney, Michael D
Liming, Why was AARHCH64 was removed from VALID_ARCHITECTURES in BaseLib.inf and BaseCpuLib.inf? Thanks, Mike From: Gao, Liming [mailto:liming@intel.com] Sent: Wednesday, August 13, 2014 7:13 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-14 Thread El-Haj-Mahmoud, Samer
@lists.sourceforge.net Subject: Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages Yes. This is a possible solution. But, svn generated patch for UTF16 file is not good readability. This patch adds new UNI files. You can see them from ZIP files. Thanks Liming -Original Message

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-13 Thread Gao, Liming
14, 2014 10:26 AM To: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages Can we change the file encoding type svn property for uni files to make them utf16 instead of binary stream? This way we can actually include uni files in

Re: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages

2014-08-13 Thread El-Haj-Mahmoud, Samer
, 9:17PM To: edk2-devel@lists.sourceforge.net [edk2-devel@lists.sourceforge.net] Subject: [edk2] [Patch 1/2] [MdePkg] INF/DEC file updates to EDK II packages Hi, all Could you help review this patch? It includes the following changes 1-6 for MdePkg. The patch is a little big. For new added UNI