My original PR has passed CI: https://github.com/tianocore/edk2/pull/2231
Ross On Thu, 23 Dec 2021 at 22:35, Kinney, Michael D <michael.d.kin...@intel.com> wrote: > > Hi Ross, > > This PR is showing CI failures with VS2019 > > https://dev.azure.com/tianocore/11ea4a10-ac9f-4e5f-8b13-7def1f19d478/_apis/build/builds/36559/logs/178 > > 2021-12-23T00:53:44.7544282Z PROGRESS - Running Build DEBUG > 2021-12-23T00:54:36.3840067Z ERROR - Compiler #2059 from C:\Program Files > (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\vcruntime_string.h(67): > syntax error: '(' > 2021-12-23T00:54:36.3841683Z ERROR - Compiler #2146 from C:\Program Files > (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\vcruntime_string.h(67): > syntax error: missing ')' before identifier 'UINTN' > 2021-12-23T00:54:36.3845453Z WARNING - Compiler #4029 from C:\Program Files > (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\vcruntime_string.h(67): > declared formal parameter list different from definition > 2021-12-23T00:54:36.3847199Z ERROR - Compiler #2059 from C:\Program Files > (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\vcruntime_string.h(67): > syntax error: ')' > 2021-12-23T00:54:36.3851318Z ERROR - Compiler #2059 from C:\Program Files > (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\vcruntime_string.h(67): > syntax error: '<parameter-list>' > 2021-12-23T00:54:36.3853972Z WARNING - Compiler #4205 from C:\Program Files > (x86)\Windows Kits\10\include\10.0.22000.0\ucrt\corecrt_memcpy_s.h(45): > nonstandard extension used: static function declaration in function scope > 2021-12-23T00:54:36.3858517Z WARNING - Compiler #4020 from C:\Program Files > (x86)\Windows Kits\10\include\10.0.22000.0\ucrt\corecrt_memcpy_s.h(54): > 'SetMem': too many actual parameters > 2021-12-23T00:54:36.3861595Z WARNING - Compiler #4020 from > d:\a\1\s\MdeModulePkg\Library\BrotliCustomDecompressLib\brotli\c\common\shared_dictionary.c(103): > 'SetMem': too many actual parameters > 2021-12-23T00:54:36.3864836Z WARNING - Compiler #4020 from > d:\a\1\s\MdeModulePkg\Library\BrotliCustomDecompressLib\brotli\c\common\shared_dictionary.c(496): > 'SetMem': too many actual parameters > 2021-12-23T00:54:36.3867350Z ERROR - Compiler #1077 from NMAKE : fatal > '"C:\Program Files (x86)\Microsoft Visual > Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\bin\Hostx86\x86\cl.exe"' : > return code '0x2' > 2021-12-23T00:54:36.3870885Z ERROR - Compiler #7000 from : Failed to > execute command > 2021-12-23T00:54:36.3873160Z ERROR - EDK2 #002 from : Failed to build module > 2021-12-23T00:54:36.3878035Z CRITICAL - Build failed > > Can you please resolve and update the PR. > > Thanks, > > Mike > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ross Burton > > Sent: Thursday, December 23, 2021 2:02 AM > > To: Gao, Liming <gaolim...@byosoft.com.cn> > > Cc: Pedro Falcato <pedro.falc...@gmail.com>; edk2-devel-groups-io > > <devel@edk2.groups.io>; Feng, Bob C <bob.c.f...@intel.com>; > > Chen, Christine <yuwei.c...@intel.com> > > Subject: Re: [edk2-devel] [PATCH] BaseTools/Brotli: update to latest brotli > > submodule > > > > On Thu, 23 Dec 2021 at 00:48, gaoliming <gaolim...@byosoft.com.cn> wrote: > > > Create PR https://github.com/tianocore/edk2/pull/2346 for this patch. > > > > There are now *three* PRs on GitHub to solve this issue, which is a > > build breaker for anyone using a new GCC. > > > > Can someone please merge the fix? > > > > Ross > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#85203): https://edk2.groups.io/g/devel/message/85203 Mute This Topic: https://groups.io/mt/87915616/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-