[edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages

Fan, ZhijuX posted 1 patch 4 years, 10 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
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(-)
[edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages
Posted by Fan, ZhijuX 4 years, 10 months ago
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@lists.01.org".
I factoring these out to a new constant, and then setting the constant
to "devel@edk2.groups.io".

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Zhiju.Fan <zhijux.fan@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@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@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@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@lists.01.org for\n"
+    "\n(Please send email to devel@edk2.groups.io for\n"
     " help, attach the following call stack trace.)\n")
 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@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())
-- 
2.14.1.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#40756): https://edk2.groups.io/g/devel/message/40756
Mute This Topic: https://groups.io/mt/31637847/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages
Posted by Laszlo Ersek 4 years, 10 months ago
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@lists.01.org".
> I factoring these out to a new constant, and then setting the constant
> to "devel@edk2.groups.io".

(1) The commit message should be cleaned up: "Factor these out ... and
then set ..."

Furthermore:

> 
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Signed-off-by: Zhiju.Fan <zhijux.fan@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@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@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@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@lists.01.org for\n"
> +    "\n(Please send email to devel@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@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 (#40789): https://edk2.groups.io/g/devel/message/40789
Mute This Topic: https://groups.io/mt/31637847/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages
Posted by Fan, ZhijuX 4 years, 10 months ago
(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@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Laszlo Ersek
> Sent: Thursday, May 16, 2019 9:17 PM
> To: devel@edk2.groups.io; Fan, ZhijuX <zhijux.fan@intel.com>
> Cc: Gao, Liming <liming.gao@intel.com>; Feng, Bob C <bob.c.feng@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@lists.01.org".
> > I factoring these out to a new constant, and then setting the constant
> > to "devel@edk2.groups.io".
> 
> (1) The commit message should be cleaned up: "Factor these out ... and then
> set ..."
> 
> Furthermore:
> 
> >
> > Cc: Bob Feng <bob.c.feng@intel.com>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Signed-off-by: Zhiju.Fan <zhijux.fan@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@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@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@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@lists.01.org for\n"
> > +    "\n(Please send email to devel@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@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/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH] BaseTools:Update mailing list address in BaseTools error messages
Posted by Laszlo Ersek 4 years, 10 months ago
Hi Zhiju,

On 05/17/19 05:07, Fan, ZhijuX wrote:
> (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.

You misunderstood me.

(a) The definition of EDKII_MAIL_ADDR is fine.

(b) The way you consume EDKII_MAIL_ADDR in "GenFds.py", "Trim.py", and "build.py", is also fine.

(c) The way you hardwire the email address "devel@edk2.groups.io" in "StringTable.py" is not fine.

Rather than directly embedding the email address in MSG_SEARCH_FOR_HELP, you should change MSG_SEARCH_FOR_HELP to take a string parameter (%s).

Then all locations in UPT that refer to MSG_SEARCH_FOR_HELP should be updated to substitute EDKII_MAIL_ADDR for the %s parameter of MSG_SEARCH_FOR_HELP.

Let me demonstrate:

---------------------
diff --git a/BaseTools/Source/Python/UPT/InstallPkg.py b/BaseTools/Source/Python/UPT/InstallPkg.py
index 6cb9f2533df5..0249810b3a9e 100644
--- a/BaseTools/Source/Python/UPT/InstallPkg.py
+++ b/BaseTools/Source/Python/UPT/InstallPkg.py
@@ -547,7 +547,7 @@ def Main(Options = None):
                     "\nInstallPkg",
                     CODE_ERROR,
                     ST.ERR_UNKNOWN_FATAL_INSTALL_ERR % Options.PackageFile,
-                    ExtraData=ST.MSG_SEARCH_FOR_HELP,
+                    ExtraData=(ST.MSG_SEARCH_FOR_HELP % EDKII_MAIL_ADDR),
                     RaiseError=False
                     )
         Logger.Quiet(ST.MSG_PYTHON_ON % (python_version(),
diff --git a/BaseTools/Source/Python/UPT/Logger/StringTable.py b/BaseTools/Source/Python/UPT/Logger/StringTable.py
index 59f71d390bef..48321ee0b865 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@lists.01.org for\n"
+    "\n(Please send email to %s for\n"
     " help, attach the following call stack trace.)\n")
 MSG_REMOVE_TEMP_FILE_STARTED   = _("Removing temp files started ... ")
 MSG_REMOVE_TEMP_FILE_DONE   = _("Removing temp files ... Done.")
---------------------

Thanks
Laszlo


>> -----Original Message-----
>> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>> Laszlo Ersek
>> Sent: Thursday, May 16, 2019 9:17 PM
>> To: devel@edk2.groups.io; Fan, ZhijuX <zhijux.fan@intel.com>
>> Cc: Gao, Liming <liming.gao@intel.com>; Feng, Bob C <bob.c.feng@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@lists.01.org".
>>> I factoring these out to a new constant, and then setting the constant
>>> to "devel@edk2.groups.io".
>>
>> (1) The commit message should be cleaned up: "Factor these out ... and then
>> set ..."
>>
>> Furthermore:
>>
>>>
>>> Cc: Bob Feng <bob.c.feng@intel.com>
>>> Cc: Liming Gao <liming.gao@intel.com>
>>> Signed-off-by: Zhiju.Fan <zhijux.fan@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@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@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@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@lists.01.org for\n"
>>> +    "\n(Please send email to devel@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@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 (#40914): https://edk2.groups.io/g/devel/message/40914
Mute This Topic: https://groups.io/mt/31637847/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-