Re: [edk2-devel] [Patch 1/4 V6] BaseTools: Add build option for dependency file generation

["Bob Feng" <bob.c.feng@intel.com>]

I'll create patch to fix this issue.

Thanks,
Bob

-----Original Message-----
From: Kinney, Michael D 
Sent: Tuesday, December 17, 2019 7:30 AM
To: devel@edk2.groups.io; anthony.perard@citrix.com; Feng, Bob C <bob.c.feng@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
Cc: Gao, Liming <liming.gao@intel.com>; Shi, Steven <steven.shi@intel.com>
Subject: RE: [edk2-devel] [Patch 1/4 V6] BaseTools: Add build option for dependency file generation

I recommend you use the ASCII ' character.

We want all source code files to be ASCII and we will be updating CI checkers to enforce this.  Only UNI string files are allowed to use UTF-8 characters.

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Anthony 
> PERARD
> Sent: Monday, December 16, 2019 6:14 AM
> To: devel@edk2.groups.io; Feng, Bob C
> <bob.c.feng@intel.com>
> Cc: Gao, Liming <liming.gao@intel.com>; Shi, Steven 
> <steven.shi@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: Re: [edk2-devel] [Patch 1/4 V6] BaseTools: Add build option 
> for dependency file generation
> 
> On Fri, Dec 06, 2019 at 11:26:55PM +0800, Bob Feng
> wrote:
> >  BaseTools/Conf/tools_def.template | 181
> ++++++++++++++++--------------
> 
> Hi Bob,
> 
> That patch introduces the first two UTF-8 characters (the quote ’ ) in 
> an otherwise all-ascii file. And that prevent our CI from building 
> OVMF.
> Part of the error log:
> 
>     Running edk2 build for OvmfPkgX64
>     build.py...
>      : error 0001: File open failure
>             /home/osstest/build.144860.build-
> amd64/xen/tools/firmware/ovmf-dir-
> remote/Conf/tools_def.txt
>     Traceback (most recent call last):
>       File "/home/osstest/build.144860.build-
> amd64/xen/tools/firmware/ovmf-dir-
> remote/BaseTools/Source/Python/Common/ToolDefClassObjec
> t.py", line 117, in IncludeToolDefFile
>         FileContent = F.readlines()
>       File "/usr/lib/python3.5/encodings/ascii.py",
> line 26, in decode
>         return codecs.ascii_decode(input, self.errors)[0]
>     UnicodeDecodeError: 'ascii' codec can't decode byte
> 0xe2 in position 2289: ordinal not in range(128)
> 
> Full build log can be found here, with ovmf build near the bottom:
>     http://logs.test-
> lab.xenproject.org/osstest/logs/144860/build-
> amd64/6.ts-xen-build.log
> 
> Should we replaces the two offending quotes by proper ascii quotes? Or 
> should we fix IncludeToolDefFile to always use UTF-8 decoding? Or 
> both?
> 
> Thanks,
> 
> --
> Anthony PERARD
> 
>