[edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages
Fan, ZhijuX
zhijux.fan at intel.com
Fri May 17 03:07:44 UTC 2019
(2)ExtraData=ST.MSG_SEARCH_FOR_HELP % EDKII_MAIL_ADDR
I have tried to solve this problem in this way. ST is the collection of all ExtraData information in UPT,
This collection does not exist in other modules, Other modules have only the collection of error message.
All ExtraData information exists as a string, There is no uniform variable like ST to collect this information.
So I created EDKII_MAIL_ADDR to achieve the same effect.
Any question, please let me know. Thanks.
Best Regards
Fan Zhiju
> -----Original Message-----
> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> Laszlo Ersek
> Sent: Thursday, May 16, 2019 9:17 PM
> To: devel at edk2.groups.io; Fan, ZhijuX <zhijux.fan at intel.com>
> Cc: Gao, Liming <liming.gao at intel.com>; Feng, Bob C <bob.c.feng at intel.com>
> Subject: Re: [edk2-devel] [PATCH] BaseTools:Update mailing list address in
> BaseTools error messages
>
> On 05/16/19 07:11, Fan, ZhijuX wrote:
> > BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1815
> >
> > The edk2 source tree contains four instances of the outdated mailing
> > list address "edk2-devel at lists.01.org".
> > I factoring these out to a new constant, and then setting the constant
> > to "devel at edk2.groups.io".
>
> (1) The commit message should be cleaned up: "Factor these out ... and then
> set ..."
>
> Furthermore:
>
> >
> > Cc: Bob Feng <bob.c.feng at intel.com>
> > Cc: Liming Gao <liming.gao at intel.com>
> > Signed-off-by: Zhiju.Fan <zhijux.fan at intel.com>
> > ---
> > BaseTools/Source/Python/Common/DataType.py | 1 +
> > BaseTools/Source/Python/GenFds/GenFds.py | 2 +-
> > BaseTools/Source/Python/Trim/Trim.py | 2 +-
> > BaseTools/Source/Python/UPT/Logger/StringTable.py | 2 +-
> > BaseTools/Source/Python/build/build.py | 2 +-
> > 5 files changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/BaseTools/Source/Python/Common/DataType.py
> > b/BaseTools/Source/Python/Common/DataType.py
> > index 780711bf8e..7f10533345 100644
> > --- a/BaseTools/Source/Python/Common/DataType.py
> > +++ b/BaseTools/Source/Python/Common/DataType.py
> > @@ -88,6 +88,7 @@ EDK_COMPONENT_TYPE_SAL_RT_DRIVER =
> 'SAL_RT_DRIVER'
> > EDK_COMPONENT_TYPE_APPLICATION = 'APPLICATION'
> > EDK_NAME = 'EDK'
> > EDKII_NAME = 'EDKII'
> > +EDKII_MAIL_ADDR = 'devel at edk2.groups.io'
> >
> > COMPONENT_TO_MODULE_MAP_DICT = {
> > EDK_COMPONENT_TYPE_LIBRARY : SUP_MODULE_BASE,
> > diff --git a/BaseTools/Source/Python/GenFds/GenFds.py
> > b/BaseTools/Source/Python/GenFds/GenFds.py
> > index 21ae9c4d4c..bd70aac9f5 100644
> > --- a/BaseTools/Source/Python/GenFds/GenFds.py
> > +++ b/BaseTools/Source/Python/GenFds/GenFds.py
> > @@ -388,7 +388,7 @@ def GenFdsApi(FdsCommandDict,
> WorkSpaceDataBase=None):
> > "\nPython",
> > CODE_ERROR,
> > "Tools code failure",
> > - ExtraData="Please send email to edk2-devel at lists.01.org for
> help, attaching following call stack trace!\n",
> > + ExtraData="Please send email to %s for help,
> > + attaching following call stack trace!\n" % EDKII_MAIL_ADDR,
> > RaiseError=False
> > )
> > EdkLogger.quiet(traceback.format_exc())
> > diff --git a/BaseTools/Source/Python/Trim/Trim.py
> > b/BaseTools/Source/Python/Trim/Trim.py
> > index 6f29f1a35a..573f33772f 100644
> > --- a/BaseTools/Source/Python/Trim/Trim.py
> > +++ b/BaseTools/Source/Python/Trim/Trim.py
> > @@ -533,7 +533,7 @@ def Main():
> > "\nTrim",
> > CODE_ERROR,
> > "Unknown fatal error when trimming [%s]" % InputFile,
> > - ExtraData="\n(Please send email to edk2-devel at lists.01.org for
> help, attaching following call stack trace!)\n",
> > + ExtraData="\n(Please send email to %s for help,
> > + attaching following call stack trace!)\n" % EDKII_MAIL_ADDR,
> > RaiseError=False
> > )
> > EdkLogger.quiet("(Python %s on %s) " %
> > (platform.python_version(), sys.platform) + traceback.format_exc())
> > diff --git a/BaseTools/Source/Python/UPT/Logger/StringTable.py
> > b/BaseTools/Source/Python/UPT/Logger/StringTable.py
> > index 59f71d390b..c1eda61d05 100644
> > --- a/BaseTools/Source/Python/UPT/Logger/StringTable.py
> > +++ b/BaseTools/Source/Python/UPT/Logger/StringTable.py
> > @@ -317,7 +317,7 @@ MSG_NEW_FILE_NAME_FOR_DIST = _(
> > MSG_UPDATE_PACKAGE_DATABASE = _("Update Distribution Package
> Database ...")
> > MSG_PYTHON_ON = _("(Python %s on %s) ")
> > MSG_SEARCH_FOR_HELP = _(
> > - "\n(Please send email to edk2-devel at lists.01.org for\n"
> > + "\n(Please send email to devel at edk2.groups.io for\n"
> > " help, attach the following call stack trace.)\n")
>
> (2) When I filed the BZ, I didn't realize that this instance of the old list address
> was in a localized string / message.
>
> However, the example of MSG_PYTHON_ON shows that such messages can
> take parameters as well. This means that all code locations referencing
> MSG_SEARCH_FOR_HELP will have to be updated --
> MSG_SEARCH_FOR_HELP should use %s, and all uses of
> MSG_SEARCH_FOR_HELP should pass in EDKII_MAIL_ADDR as a parameter:
>
> $ git grep -l -w MSG_SEARCH_FOR_HELP
>
> Source/Python/UPT/InstallPkg.py
> Source/Python/UPT/InventoryWs.py
> Source/Python/UPT/Logger/StringTable.py
> Source/Python/UPT/MkPkg.py
> Source/Python/UPT/ReplacePkg.py
> Source/Python/UPT/RmPkg.py
> Source/Python/UPT/TestInstall.py
>
> For example,
>
> ExtraData=ST.MSG_SEARCH_FOR_HELP % EDKII_MAIL_ADDR
>
> Thanks
> Laszlo
>
> > MSG_REMOVE_TEMP_FILE_STARTED = _("Removing temp files started ...
> ")
> > MSG_REMOVE_TEMP_FILE_DONE = _("Removing temp files ... Done.")
> > diff --git a/BaseTools/Source/Python/build/build.py
> > b/BaseTools/Source/Python/build/build.py
> > index 7271570d29..0071581c80 100644
> > --- a/BaseTools/Source/Python/build/build.py
> > +++ b/BaseTools/Source/Python/build/build.py
> > @@ -2480,7 +2480,7 @@ def Main():
> > "\nbuild",
> > CODE_ERROR,
> > "Unknown fatal error when processing [%s]" % MetaFile,
> > - ExtraData="\n(Please send email to edk2-devel at lists.01.org for
> help, attaching following call stack trace!)\n",
> > + ExtraData="\n(Please send email to %s for help,
> > + attaching following call stack trace!)\n" % EDKII_MAIL_ADDR,
> > RaiseError=False
> > )
> > EdkLogger.quiet("(Python %s on %s) " %
> > (platform.python_version(), sys.platform) + traceback.format_exc())
> >
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#40853): https://edk2.groups.io/g/devel/message/40853
Mute This Topic: https://groups.io/mt/31637847/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list