summaryrefslogtreecommitdiffstats
path: root/BaseTools
Commit message (Collapse)AuthorAgeFilesLines
...
* BaseTools: Fix the bug to display the single SKUID infoYonghong Zhu2018-02-241-0/+4
| | | | | | | | | | | when defined SKUID_IDENTIFIER = DEFAULT|TEST in DSC [Defines] section, per spec it means current SKUID is single, the bug is build report print both DEFAULT and TEST info, it should only print TEST. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 8aaa8f7bc0f5dc79bce476e2ffa8a956c4db8bb0)
* BaseTools: Add check for INF statement must be a .inf fileYonghong Zhu2018-02-241-1/+3
| | | | | | | | | | Per FDF spec, INF statement must use a .inf file, we add this error check. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit a87e79d9d64a73f1c8368e2b7f309954271c69a7)
* BaseTools: Add WindowsLike path in front of PATH EnvYonghong Zhu2018-02-241-1/+1
| | | | | | | | | | | | Original BaseTools source build append WindowsLike path to PATH Env, while WINDDK installation has a "build.exe", if user place WINDDK folder to PATH either during WINDDK installation or manually, it will block the BaseTools' build.bat. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit ffcec664ca2d97066f5f47f7e3abd85171f91641)
* BaseTools/Conf: Add VS2017/ARM supportPete Batard2018-02-232-1/+61
| | | | | | | | | | We duplicate the Assembly-Code-File section from build_rule.template because --convert-hex cannot be used with the MSFT ARM assembler. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Pete Batard <pete@akeo.ie> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 0a4c903c5a07a5f0b2d2e9e9ba6929ae6a9dddaa)
* BaseTools/Conf: disable DTC legacy phandle formatArd Biesheuvel2018-02-232-1/+2
| | | | | | | | | | | | | | | | | | By default, the device tree compiler emits phandle properties twice: once called 'phandle' and again called 'linux,phandle'. Given that Linux was updated in early 2010 [0] to accept the former (which is what is specified in the ePAPR and device tree specifications), there is no point in emitting both when compiling device trees for UEFI platforms. [0] 04b954a673dd02f585a2769c4945a43880faa989 "of/flattree: Make the kernel accept ePAPR style phandle information" Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 34a4ddda4d61ede41f575510dc5adcf933d3b6f5)
* BaseTools: Fix VOID* type bugFeng, YunhuaX2018-02-111-0/+2
| | | | | | | | | | | | Code miss UINT32 and UINT64 value type setting in VOID*, like as {UINT32({TRUE})} Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit a35918caae4d0b9bb51d0d4765117d7ca9a4d641)
* BaseTool: correct the generate compress section processFeng, YunhuaX2018-02-111-8/+5
| | | | | | | | | | | | First generate a dummy file with section alignment, then compress the dummy file to generate the compress file Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit ce2818e418020a9cf84f6712cbda4075b0652809)
* BaseTools: Update Expression.py for string comparison and MACRO replace issueYunhua Feng2018-02-113-31/+13
| | | | | | | | | | | | | 1. Fix string comparison incorrect issue, we expected "ABC" is greater than "AAD" since the second char 'B' is greater than 'A'. 2. fix MACRO not replace issue. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 9efe8d604049b1d2f320ae5c40cd925d6504bceb)
* BaseTools: not specified value of MAX_CONCURRENT_THREAD_NUMBERYonghong Zhu2018-02-112-6/+11
| | | | | | | | | | | when MAX_CONCURRENT_THREAD_NUMBER is not specified, tool will automatically detect number of processor threads. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=775 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 2052cb675f1fdbc77f916422fdf5c2da29741169)
* BaseTools: Enhance error handling for unsupported toolchain Flags/PathYonghong Zhu2018-02-111-1/+5
| | | | | | | | | | | | | Case1: Cover the Tool PATH is not exist, eg: build MdeModule under GCC5 toolchain and IPF arch. Case2: Cover the Tool FLAGS is not exist, eg: build OvmfPkg under CLANG35 toolchain and X64 arch. fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=595 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 70d0a7549ef4a5dbdd15c0f78bd2f8ede093f8f4)
* BaseTools: Fixed incorrect Structure Pcd Value.Feng, Bob C2018-02-111-7/+13
| | | | | | | | | | | When structurePCD only has overall value assigned in Dsc under different SKU, the value under default sku is used. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 77204d608be05103a68b470fbbcdeccfc887911f)
* BaseTools: Fixed Build failed issue.BobCF2018-02-111-1/+6
| | | | | | | | | | | If the PCD is not used in DSC file and user set that PCD value from Command line, build will fail. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e1511113debf54ec2aceb20ee8eeecb331acd498)
* BaseTool: Fixed Pcd issues.Feng, Bob C2018-02-114-79/+100
| | | | | | | | | | | | 1. Check variable offset when merging Hii Pcds 2. Fixed the issue of Hii value inherit with default store. 3. Error handling for incorrect structure pcd declare. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 8aa4db4b807ea7de395acd4018a139745327e374)
* BaseTools: Fix COMPRESS alignment incorrect issueFeng, YunhuaX2018-02-111-0/+14
| | | | | | | | | | | | | | | | | | | | | | | Doesn't generate the correct alignment for the leaf section in the compression section. Below FFS rule doesn't work as the expectation. [Rule.Common.PEIM.PE32] FILE PEIM = $(NAMED_GUID) { PEI_DEPEX PEI_DEPEX Optional $(INF_OUTPUT)/$(MODULE_NAME).depex COMPRESS { PE32 PE32 Align=Auto $(INF_OUTPUT)/$(MODULE_NAME).efi UI STRING="$(MODULE_NAME)" Optional VERSION STRING="$(INF_VERSION)" Optional BUILD_NUM=$(BUILD_NUMBER) } } Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 4f735fc8cdd9a7e385d285c771abddfcae8a5e58)
* BaseTools: Report error when GUID format is incorrectFeng, YunhuaX2018-02-111-10/+4
| | | | | | | | | | | | | | | | Flexible GUID format of PCD value support following format, so tool should report error when it is not correct. 1. { GUID("11E13869-1896-4A07-8B21-D8B23DD2A2B4") } 2. { GUID({ 0x11e13869, 0x1896, 0x4a07,{ 0x8b, 0x21, 0xd8, 0xb2, 0x3d, 0xd2, 0xa2, 0xb4 } }) } 3. { GUID(gEfiBlockIoProtocolGuid) } Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 4344a788c472fccd9af559891363374868fb6275)
* BaseTools: Fix flexible PCD DEVICE_PATH parse issueFeng, YunhuaX2018-02-111-1/+2
| | | | | | | | | | | | | When the format of DEVICE_PATH have string, like as: {DEVICE_PATH("BBS(1,"AB",0)")} have string "AB", will get the wrong value. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 8ad5f10a2be2c2c8dde111c0478b02da44a781e2)
* BaseTools: Fix build argument --pcd for flexible format bugsFeng, YunhuaX2018-02-113-7/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | Build argument --pcd flexible format, like as: 1. VOID* type gTokenSpaceGuid.Test=H"{flexible format}" gTokenSpaceGuid.Test=L"string" gTokenSpaceGuid.Test="string" gTokenSpaceGuid.Test=L'string' gTokenSpaceGuid.Test='string' 2. UINT8/UINT16/UINT32/UINT64 type gTokenSpaceGuid.Test=H"{flexible format}" gTokenSpaceGuid.Test=L"string" gTokenSpaceGuid.Test="string" gTokenSpaceGuid.Test=L'string' gTokenSpaceGuid.Test='string' In linux, single quotes need escape gTokenSpaceGuid.Test=L\'string\' gTokenSpaceGuid.Test=\'string\' Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit f9bba77495750bb5f9bc5c5864b7c76fece5ec9f)
* BaseTools: enhance error check for DatumType formatYonghong Zhu2018-02-061-1/+6
| | | | | | | | | | Add a check for DatumType format, eg: VOID *, original Tool will crash but no detail error message which cause confusion to user. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e0db09cd1cd35a1e2ae627e305d903652555aef2)
* BaseTools GNUmakefile: Remove HOST_ARCH in every tool MakefileLiming Gao2018-02-0618-36/+18
| | | | | | | | HOST_ARCH has been moved into the common header.makefile Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit d16e777ac564557f805db05a0110874797b65ceb)
* BaseTools GNUmakefile: Move HOST_ARCH detection into common makefileLiming Gao2018-02-061-2/+26
| | | | | | | | | With this change, enter single tool directory, make can pass. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 1eac82c878f5f65e2cd4eb6a6553dbddeb739c12)
* BaseTools: Update Expression.py for VOID* to support L'a' and 'a'Feng, YunhuaX2018-02-062-3/+20
| | | | | | | | | | | | | | | | | | | | Original VOID* type support L"string" and "string" format, now we also add support for single quote string that without null terminator. Type VOID* support L'a' and 'a', the value transfer to c style value. L'a' --> {0x61, 0x00} L'ab' --> {0x61, 0x00, 0x62, 0x00} 'a' --> {0x61} 'ab' --> {0x61, 0x62} when the value is L'' or '' that not include any character, tool will report error. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 0e6b86731e5792a2fe89b595268a817f0cd989cc)
* BaseTools: StructurePcd array Value support flexible formatFeng, YunhuaX2018-02-061-2/+33
| | | | | | | | | | | | | | | | | | if StructurePcd set item value is array, support flexible format like as: gEfiStructuredPcdPkgTokenSpaceGuid.Test.Array | {flexible format} {flexible format} = {L"ABC"} | {L'ABC'} | {"ABC"} | {UINT8(0x10)} | {UINT16(0x10)} | {UINT32(0x10)} | {UINT64(0x10)} | {DEVICE_PATH("PciRoot(0)/Pci(0,0)")} | {GUID(gPcdPkgTokenSpaceGuid)} | {L"ABC", L'ABC', UINT8(0x10)....} Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit bee0f2f167e6e982e665c851d605cd50e7748e08)
* BaseTools: Fix make PcdValueCommon.c failure on GCCFeng, YunhuaX2018-02-061-2/+1
| | | | | | | | | | | | | | | | | error message: PcdValueCommon.c: In function '__PcdGetPtr': PcdValueCommon.c:315:11: error: variable 'Byte' set but not used [-Werror=unused-but-set-variable] UINT8 Byte; ^ cc1: all warnings being treated as errors Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 693877f8e593e51f38e67108c4db98e56f68e8d8)
* BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommonLiming Gao2018-02-061-10/+0
| | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 83ba30c5a3e4b285bc2be99308cbe5fcfb1d0e16)
* BaseTools: Structure Pcd in CommandLine.Feng, Bob C2018-02-027-98/+233
| | | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 6f49996cedfb198beb374ea0cb89a06a71ef960c)
* BaseTools: Support multiple .h fileFeng, Bob C2018-02-024-7/+9
| | | | | | | | | | for structure Pcd declaration in DEC file. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 81add864f4af238a2dfb702904a6abec12738b9d)
* BaseTool: Enhance error handling.BobCF2018-02-021-49/+67
| | | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 5db9414cc1b9b449c70a974f0aa0d17b41d09629)
* BaseTool: Add comments in PcdValueInit.c.Feng, Bob C2018-02-021-4/+6
| | | | | | | | | | Add Comments for __FLEXIBLE_SIZE () statement. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 75771aebd39ec7c6d383e98085b54c9419d2546e)
* BaseTools: Update BPDG to support L'' and '' format as VPD Pcd ValueYonghong Zhu2018-02-022-17/+21
| | | | | | | | | | Current Pcd value support flexible format, this patch add support for BPDG Tool to support L'' and '' format. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit d5988a8ac9716130a323fb12bbf81c41807c7865)
* BaseTools: Fix the bug to align VPD PCD based on value typeYonghong Zhu2018-02-021-2/+12
| | | | | | | | | | | | Spec required for VOID* VPD Pcd, Ascii string use byte alignment, byte array use 8-byte alignment, unicode string use 2-byte alignment. while when the VPD pcd offset use *, the offset generated in the .map file not follow this rule. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 86737681af34d14dfe088d806d4a5062fdfb3f1f)
* BaseTools: Enhance parse performance by optimize ValueExpressionExYunhua Feng2018-01-315-178/+183
| | | | | | | | | | | | | Optimize ValueExpressionEx function to enhance meta-data file parse performance. Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 35f613d96ce43c7b23cd77aab063424ec4422e0c)
* BaseTools: Fix indentation in CParser.py fileGary Lin2018-01-302-18/+4
| | | | | | | | | | | | Mixing usage of spaces and tabs may confuse the python compiler/interpreter. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Gary Lin <glin@suse.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 877251b4212dedaabefbc3ef6cd637a5b2740d47)
* BaseTools: Fixed incorrect VPD size.BobCF2018-01-301-6/+4
| | | | | | | | | The VPD size is incorrect if that VPD is not used in Module. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Feng Bob C <bob.c.feng@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e827d21da1c21aea0e1cdc9b896aacf988a5bacd)
* BaseTool: Fixed the StructurePcd incorrect value.Feng, Bob C2018-01-301-3/+6
| | | | | | | | | | | If user not set Structure overall value in Dsc, Structure Pcd value would be incorrect. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 9904222db0affb82d2aa0004dda35c3c66b74234)
* BaseTools: Fixed some small issuesFeng, Bob C2018-01-303-4/+10
| | | | | | | | | | | | 1. The structure pcd default value should use the default value under sku. 2. Incorrect VpdOffset value for those un-used in module Vpd 3. Add a checkpoint for Structure Pcd Name Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 8e011d83bbd7006919c00917fbdfaf7a60edae21)
* BaseTools: Fixed build failure for the PCD value initialization.Feng, Bob C2018-01-301-2/+14
| | | | | | | | | | A pcd is initialized under one SKU but is uninitialized under another SKU. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 841d86fe406ed6cf504c62b6838876a65cf8f7a0)
* BaseTools: Add comments for the Structure Pcd definition in PcdValueInit.c fileFeng, Bob C2018-01-303-1/+7
| | | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 6a103440806c33c0aaa401cd443dc786d259397e)
* BaseTool: Combine the HiiPcd value if they link to same VariableFeng, Bob C2018-01-302-3/+35
| | | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng <bob.c.feng@intel.com> Cc: Liming Gao <liming.gao@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e6bf3cfd50c1f4f6c6d4648ee89c950094d85727)
* BaseTools: CommonLib Fix Crash to write the last byteLiming Gao2018-01-301-1/+1
| | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 4faf5340467a9595b1df7f4faf43f78ec59d3157)
* BaseTools: Barf on unknown HOST_ARCH in C MakefileChema Gonzalez2018-01-302-8/+10
| | | | | | | | | | | | | | I was getting `HOST_ARCH` set using the linux arch name ("x86_64"), which is different from the MS one ("X64"). It is not clear anyway we can proceed without valid build variables (`ARCH_INCLUDE`, `BIN_PATH`, `LIB_PATH`, `SYS_BIN_PATH`, and `SYS_LIB_PATH`). Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chema Gonzalez <chemag@gmail.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 9d5aab05540fd7de26894b05544f1efc424ff595)
* BaseTools/tools_def CLANG3x: ignore unknown warning optionsArd Biesheuvel2018-01-301-3/+3
| | | | | | | | | | | | | | | | Ironically, disabling warnings in the OpensslLib library build is causing breakage when using the CLANG35 toolchain to build for ARM: error: unknown warning option '-Werror=maybe-uninitialized'; did you mean '-Werror=uninitialized'? [-Werror,-Wunknown-warning-option] So let's add -Wno-unknown-warning-option to the list of warnings to ignore when using Clang 3.5, and move the same option from the x86 specific list to the shared list for Clang 3.8. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit faf0475b1376bde30c6c5d7fa6f02dfe22801f27)
* BaseTools: Enable MAX_CONCURRENT_THREAD_NUMBER = 0 featureYunhua Feng2018-01-301-2/+6
| | | | | | | | | | | | | when set 'MAX_CONCURRENT_THREAD_NUMBER=0', will auto-detect number of processor threads as MAX_CONCURRENT_THREAD_NUMBER. Fixes: https://bugzilla.tianocore.org/show_bug.cgi?id=775 Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 29af38b0f8f2aa7b08f61a9df38a59dbc15e9302)
* BaseTools: Add DefaultStore section format CheckYonghong Zhu2018-01-242-1/+9
| | | | | | | | | | This patch add DefaultStore section format Check and it use same logic with SKUID section. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 767ddbe87416afe001d4176f5d73e74f5f10f16b)
* BaseTools: update SKUID value to support both integer and Hex numberYonghong Zhu2018-01-242-6/+7
| | | | | | | | | This patch updated Skuid value to support both integer and hex value. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e6b10112b3ee661595f81aec7e1948bb4e8e0153)
* BaseTools: Enhance binary file in [Binaries] section use relative pathYonghong Zhu2018-01-241-0/+2
| | | | | | | | | | Enhance the binary file in Asbuilt inf file [Binaries] section use relative path. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit 5e2c0ecd0bff48fb960ada1d138cbc382d4c1934)
* BaseTools: Add "processing meta-data" string backYonghong Zhu2018-01-241-3/+3
| | | | | | | | | | Previous build tool will display "processing meta-data ..." to let user know the progress. this Patch add this string back. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> (cherry picked from commit e74cea4c7b2ea013a4b08f22c0a4e7a4ad6e69bb)
* BaseTools: Optimizing DscDefaultValue process in BuildReportFrom: Yunhua Feng2018-01-211-2/+2
| | | | | | | | | | | | | DscDefaultValue from Dsc file has been parsed by ValueExpressionEx when Dsc file parse, so only DscDefaultValue from FDF file need ValueExpressionEx parse Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit a676a24656546a7fc722868fb75cd178c003056a)
* BaseTools: Fix GenFds increment build bug that missing cover command ↵Yunhua Feng2018-01-211-3/+3
| | | | | | | | | | | | | | | | | option's change Issue decription: step 1, build platform X64 step 2, build platform IA32 step 3, build platform X64 step 4, check all ffs files for X64, the content still has IA32 in it Cc: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 6735645d9c09d9a391ea96233d3addd3c2b30843)
* BaseTools: Use nasm as the preferred assembly source files for XCODE5 toolLiming Gao2018-01-191-1/+0
| | | | | | | | | | | https://bugzilla.tianocore.org/show_bug.cgi?id=850 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> Cc: Andrew Fish <afish@apple.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit 2583352f2427be24019e97f18905d8d2b1c64872)
* BaseTools: Disable -Wno-unused-const-variable in XCODE5 RELEASE targetLiming Gao2018-01-191-3/+3
| | | | | | | | | Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming.gao@intel.com> Cc: Yonghong Zhu <yonghong.zhu@intel.com> Cc: Andrew Fish <afish@apple.com> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> (cherry picked from commit db408fa3c1b05c867c21e7b05e8db9c7072a3b85)