Re: [edk2] [PATCH 6/6] BaseTools/Conf: Add VS2017/ARM support

2017-12-04 Thread Pete Batard
On 2017.12.04 15:48, Gao, Liming wrote: 1. You add new build for ARM. I remember the reason is that --convert-hex can't be used by trim for ARM ASM file. Right? That is correct. Maybe there's a better option to achieve just that, as duplicating the whole section just to remove an option

Re: [edk2] [PATCH 6/6] BaseTools/Conf: Add VS2017/ARM support

2017-12-04 Thread Gao, Liming
Pete: 1. You add new build for ARM. I remember the reason is that --convert-hex can't be used by trim for ARM ASM file. Right? 2. On tools_def.txt. Why introduce /D_ARM_WINAPI_PARTITION_DESKTOP_SDK_AVAILABLE option? Thanks Liming > -Original Message- > From: Pete Batard

[edk2] [PATCH 6/6] BaseTools/Conf: Add VS2017/ARM support

2017-12-04 Thread Pete Batard
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Pete Batard --- BaseTools/Conf/build_rule.template | 30 BaseTools/Conf/tools_def.template | 28 ++ 2 files changed, 58 insertions(+) diff --git