[edk2-devel] [PATCH v1 0/1] Resolve regex syntax warnings

Joey Vagedes via groups.io posted 1 patch 4 months, 3 weeks ago
Failed in applying to current master (apply log)
BaseTools/Source/Python/AmlToC/AmlToC.py                     |  2 +-
BaseTools/Source/Python/AutoGen/BuildEngine.py               |  2 +-
BaseTools/Source/Python/AutoGen/GenDepex.py                  |  2 +-
BaseTools/Source/Python/AutoGen/GenMake.py                   |  2 +-
BaseTools/Source/Python/AutoGen/IdfClassObject.py            |  2 +-
BaseTools/Source/Python/AutoGen/ModuleAutoGen.py             |  4 ++--
BaseTools/Source/Python/AutoGen/StrGather.py                 |  2 +-
BaseTools/Source/Python/AutoGen/WorkspaceAutoGen.py          |  2 +-
BaseTools/Source/Python/Common/Expression.py                 | 16 ++++++-------
BaseTools/Source/Python/Common/GlobalData.py                 |  4 ++--
BaseTools/Source/Python/Common/Misc.py                       | 24 ++++++++++----------
BaseTools/Source/Python/Common/ToolDefClassObject.py         |  6 ++---
BaseTools/Source/Python/GenFds/FdfParser.py                  | 10 ++++----
BaseTools/Source/Python/GenFds/GenFds.py                     |  2 +-
BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTable.py | 12 +++++-----
BaseTools/Source/Python/Trim/Trim.py                         | 18 +++++++--------
BaseTools/Source/Python/Workspace/DscBuildData.py            |  8 +++----
BaseTools/Source/Python/Workspace/MetaFileParser.py          |  2 +-
18 files changed, 60 insertions(+), 60 deletions(-)
[edk2-devel] [PATCH v1 0/1] Resolve regex syntax warnings
Posted by Joey Vagedes via groups.io 4 months, 3 weeks ago
Python 3.12 now produces syntax warnings when using an invalid escape
character (\ followed by an unexpected character). This happens
throughout BaseTools due the usage of regular expressions. the re module
in python suggests that when creating regex patterns, to use raw text.
This patch series adds the r prefix to any regex pattern that uses an
invalid escape sequence.

Cc: Rebecca Cran <rebecca@bsdio.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Yuwei Chen <yuwei.chen@intel.com>
Signed-off-by: Joey Vagedes <joey.vagedes@gmail.com>

Joey Vagedes (1):
  BaseTools: Resolve regex syntax warnings

 BaseTools/Source/Python/AmlToC/AmlToC.py                     |  2 +-
 BaseTools/Source/Python/AutoGen/BuildEngine.py               |  2 +-
 BaseTools/Source/Python/AutoGen/GenDepex.py                  |  2 +-
 BaseTools/Source/Python/AutoGen/GenMake.py                   |  2 +-
 BaseTools/Source/Python/AutoGen/IdfClassObject.py            |  2 +-
 BaseTools/Source/Python/AutoGen/ModuleAutoGen.py             |  4 ++--
 BaseTools/Source/Python/AutoGen/StrGather.py                 |  2 +-
 BaseTools/Source/Python/AutoGen/WorkspaceAutoGen.py          |  2 +-
 BaseTools/Source/Python/Common/Expression.py                 | 16 ++++++-------
 BaseTools/Source/Python/Common/GlobalData.py                 |  4 ++--
 BaseTools/Source/Python/Common/Misc.py                       | 24 ++++++++++----------
 BaseTools/Source/Python/Common/ToolDefClassObject.py         |  6 ++---
 BaseTools/Source/Python/GenFds/FdfParser.py                  | 10 ++++----
 BaseTools/Source/Python/GenFds/GenFds.py                     |  2 +-
 BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTable.py | 12 +++++-----
 BaseTools/Source/Python/Trim/Trim.py                         | 18 +++++++--------
 BaseTools/Source/Python/Workspace/DscBuildData.py            |  8 +++----
 BaseTools/Source/Python/Workspace/MetaFileParser.py          |  2 +-
 18 files changed, 60 insertions(+), 60 deletions(-)

-- 
2.43.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112139): https://edk2.groups.io/g/devel/message/112139
Mute This Topic: https://groups.io/mt/103021364/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 0/1] Resolve regex syntax warnings
Posted by Rebecca Cran 4 months, 1 week ago
Queued as https://github.com/tianocore/edk2/pull/5178 .

On 12/6/2023 1:27 PM, Joey Vagedes wrote:
> Python 3.12 now produces syntax warnings when using an invalid escape
> character (\ followed by an unexpected character). This happens
> throughout BaseTools due the usage of regular expressions. the re module
> in python suggests that when creating regex patterns, to use raw text.
> This patch series adds the r prefix to any regex pattern that uses an
> invalid escape sequence.
>
> Cc: Rebecca Cran <rebecca@bsdio.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Yuwei Chen <yuwei.chen@intel.com>
> Signed-off-by: Joey Vagedes <joey.vagedes@gmail.com>
>
> Joey Vagedes (1):
>    BaseTools: Resolve regex syntax warnings
>
>   BaseTools/Source/Python/AmlToC/AmlToC.py                     |  2 +-
>   BaseTools/Source/Python/AutoGen/BuildEngine.py               |  2 +-
>   BaseTools/Source/Python/AutoGen/GenDepex.py                  |  2 +-
>   BaseTools/Source/Python/AutoGen/GenMake.py                   |  2 +-
>   BaseTools/Source/Python/AutoGen/IdfClassObject.py            |  2 +-
>   BaseTools/Source/Python/AutoGen/ModuleAutoGen.py             |  4 ++--
>   BaseTools/Source/Python/AutoGen/StrGather.py                 |  2 +-
>   BaseTools/Source/Python/AutoGen/WorkspaceAutoGen.py          |  2 +-
>   BaseTools/Source/Python/Common/Expression.py                 | 16 ++++++-------
>   BaseTools/Source/Python/Common/GlobalData.py                 |  4 ++--
>   BaseTools/Source/Python/Common/Misc.py                       | 24 ++++++++++----------
>   BaseTools/Source/Python/Common/ToolDefClassObject.py         |  6 ++---
>   BaseTools/Source/Python/GenFds/FdfParser.py                  | 10 ++++----
>   BaseTools/Source/Python/GenFds/GenFds.py                     |  2 +-
>   BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTable.py | 12 +++++-----
>   BaseTools/Source/Python/Trim/Trim.py                         | 18 +++++++--------
>   BaseTools/Source/Python/Workspace/DscBuildData.py            |  8 +++----
>   BaseTools/Source/Python/Workspace/MetaFileParser.py          |  2 +-
>   18 files changed, 60 insertions(+), 60 deletions(-)
>



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112776): https://edk2.groups.io/g/devel/message/112776
Mute This Topic: https://groups.io/mt/103021364/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-